-
Notifications
You must be signed in to change notification settings - Fork 29.8k
[ Tool ] Handle DartDevelopmentServiceException when launching web applications
#180905
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
[ Tool ] Handle DartDevelopmentServiceException when launching web applications
#180905
Conversation
…applications DWDS can throw `DartDvelopmentServiceException` if DDS fails to connect to DWDS due to the target application shutting down immediately after launch. This change adds logic to catch this exception and exit gracefully. Fixes flutter#178151
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code Review
This pull request introduces handling for DartDevelopmentServiceException when launching web applications, preventing crashes when the application shuts down during DDS startup. This is achieved by adding a new catch block to gracefully exit. A regression test is included to verify this new behavior. Additionally, error logging for several other exception types has been refactored to use error.toString() for consistency and safety. My review includes a suggestion to refactor the new catch block to reduce code duplication.
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
jyameo
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
…applications (flutter#180905) DWDS can throw `DartDvelopmentServiceException` if DDS fails to connect to DWDS due to the target application shutting down immediately after launch. This change adds logic to catch this exception and exit gracefully. Fixes flutter#178151 --------- Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
…applications (flutter#180905) DWDS can throw `DartDvelopmentServiceException` if DDS fails to connect to DWDS due to the target application shutting down immediately after launch. This change adds logic to catch this exception and exit gracefully. Fixes flutter#178151 --------- Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
…ing web applications (flutter/flutter#180905)
…ing web applications (flutter/flutter#180905)
…ing web applications (flutter/flutter#180905)
…ching web applications (flutter/flutter#180905)
…ing web applications (flutter/flutter#180905)
…ing web applications (flutter/flutter#180905)
…ing web applications (flutter/flutter#180905)
…ing web applications (flutter/flutter#180905)
…ing web applications (flutter/flutter#180905)
…ing web applications (flutter/flutter#180905)
…ing web applications (flutter/flutter#180905)
…ing web applications (flutter/flutter#180905)
…applications (flutter#180905) DWDS can throw `DartDvelopmentServiceException` if DDS fails to connect to DWDS due to the target application shutting down immediately after launch. This change adds logic to catch this exception and exit gracefully. Fixes flutter#178151 --------- Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
…ing web applications (flutter/flutter#180905)
…ing web applications (flutter/flutter#180905)
…ing web applications (flutter/flutter#180905)
Manual roll requested by tarrinneal@google.com flutter/flutter@b45a73b...5a2067b 2026-01-13 engine-flutter-autoroll@skia.org Roll Packages from e57e7f4 to eb9e1dc (6 revisions) (flutter/flutter#180907) 2026-01-13 ahmedsameha1@gmail.com Make sure that a FocusScope doesn't crash in 0x0 environment (flutter/flutter#180715) 2026-01-13 bkonyi@google.com [ Tool ] Handle `DartDevelopmentServiceException` when launching web applications (flutter/flutter#180905) 2026-01-13 116356835+AbdeMohlbi@users.noreply.github.com Removes `RequiresApi 23` (flutter/flutter#180629) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages Please CC stuartmorgan@google.com,tarrinneal@google.com on the revert to ensure that a human is aware of the problem. To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
…ing web applications (flutter/flutter#180905)
…ing web applications (flutter/flutter#180905)
Manual roll requested by tarrinneal@google.com flutter/flutter@b45a73b...48c2475 2026-01-13 ahmedsameha1@gmail.com Make sure that a FocusScope doesn't crash in 0x0 environment (flutter/flutter#180715) 2026-01-13 bkonyi@google.com [ Tool ] Handle `DartDevelopmentServiceException` when launching web applications (flutter/flutter#180905) 2026-01-13 116356835+AbdeMohlbi@users.noreply.github.com Removes `RequiresApi 23` (flutter/flutter#180629) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages Please CC stuartmorgan@google.com,tarrinneal@google.com on the revert to ensure that a human is aware of the problem. To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
DWDS can throw
DartDvelopmentServiceExceptionif DDS fails to connect to DWDS due to the target application shutting down immediately after launch.This change adds logic to catch this exception and exit gracefully.
Fixes #178151