Skip to content

Investigate/Remove usage of sync WebDriver APIs #50144

Closed

Description

@jonahwilliams

The flutter_driver web implementation currently relies on sync sockets: https://github.com/flutter/flutter/blob/master/packages/flutter_driver/lib/src/driver/web_driver.dart#L12

We probably shouldn't be using these, since it can hang the tool or test script to block on a socket connection. It seems like the async versions of web_driver incur additional overhead on closing, but that could be investigated as a bug on its own.

  • @angjieli did you ever discover the reason for the 15 second delay?

Activity

  1. added
    toolAffects the "flutter" command-line tool. See also t: labels.
    t: flutter driver"flutter driver", flutter_drive, or a driver test
    on Feb 4, 2020
  2. angjieli commented on Feb 5, 2020

    @angjieli
    Contributor

    It seems like async web_driver uses HttpClient which has a property called idleTimeout defaults to 15 seconds.
    https://github.com/dart-lang/sdk/blob/master/sdk/lib/_http/http.dart#L1423

  3. angjieli commented on Feb 5, 2020

    @angjieli
    Contributor

    Other than the issue mentioned above, using the async webdriver is more likely to break for non-Chrome driver when calling fromExistingSession() while working perfectly fine with sync webdriver.

  4. added
    P3Issues that are less important to the Flutter project
    triaged-toolTriaged by Flutter Tool team
    on Oct 16, 2023
  5. matanlurey commented on Jun 27, 2025

    @matanlurey
    Contributor

    Closed by #50835.

  6. github-actions commented on Jul 11, 2025

    @github-actions

    This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.

  7. locked as resolved and limited conversation to collaborators on Jul 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    P3Issues that are less important to the Flutter projectc: proposalA detailed proposal for a change to Fluttert: flutter driver"flutter driver", flutter_drive, or a driver testteam-toolOwned by Flutter Tool teamtoolAffects the "flutter" command-line tool. See also t: labels.triaged-toolTriaged by Flutter Tool team

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions