Skip to content

Fix issues discovered when using web server device in flutter tools #1588

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

annagrin
Copy link
Contributor

@annagrin annagrin commented Apr 18, 2022

Support for resolving dart: uris is incomplete and slows downs integration due to dependency on an internal library.

  • Remove dependency on package:_fe_analyzer_shared and incorrect support for resolving dart: uris.

Flutter tool crashes in dev_handler

  • Fix a crash in dev handler on null DevTools address.

Bogus warning about unresolved dart:web_sql library that is no longer used.

  • Remove unused dart:web_sql SDK library from the SDK libraries list.

Broken breakpoints due to script information being cleared on page (re)load.

  • Made extension debugger match the behavior of web kit debugger on Page.frameStartedLoading and Debugger.globalObjectCleared.
    Webkit debugger has the correct behavior since the scripts are not re-added on page reload or hot restart.

Closes: #1581
Closes: #1575
Closes: #1579
Helps: flutter/flutter#101639
Helps: flutter/flutter#101224
Helps: flutter/flutter#102159

@annagrin annagrin requested review from nshahan, elliette and grouma April 19, 2022 00:02
Copy link
Contributor

@elliette elliette left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with few small changes

@annagrin annagrin merged commit 8c814f9 into dart-lang:master Apr 19, 2022
@mit-mit
Copy link
Member

mit-mit commented Apr 20, 2022

@annagrin we need this published before we can roll to Flutter, right?

@annagrin
Copy link
Contributor Author

@mit-mit that's right, it has been published in dwds-v14.0.0 and rolled to flutter in flutter/flutter#103220

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants