Skip to content

Conversation

bkonyi
Copy link
Collaborator

@bkonyi bkonyi commented Oct 7, 2025

This is needed to access the DTD instance spawned by DDS.

bkonyi added 3 commits October 7, 2025 17:15
This is needed to access the DTD instance spawned by DDS.
This is needed to access the DTD instance spawned by DDS.
Copy link

github-actions bot commented Oct 7, 2025

Package publishing

Package Version Status Publish tag (post-merge)
package:dwds 26.0.0 ready to publish dwds-v26.0.0
package:frontend_server_client 4.0.1-wip WIP (no publish necessary)
package:webdev 3.8.0-wip WIP (no publish necessary)

Documentation at https://github.com/dart-lang/ecosystem/wiki/Publishing-automation.

@bkonyi
Copy link
Collaborator Author

bkonyi commented Oct 7, 2025

cc @gspencergoog

Copy link
Contributor

@srujzs srujzs left a comment

Choose a reason for hiding this comment

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

LGTM!

@auto-submit auto-submit bot merged commit 186bfe7 into main Oct 9, 2025
57 checks passed
bkonyi added a commit that referenced this pull request Oct 9, 2025
This is needed to access the DTD instance spawned by DDS.
bkonyi added a commit that referenced this pull request Oct 9, 2025
* [DWDS] Don't send PauseInterrupted event during a hot reload (#2695)

Fixes dart-lang/sdk#61560

We rely on a pause within a hot reload to pause execution so that we can reregister breakpoints. However, the existing pause mechanism always sends a PauseInterrupted event, which then triggers the client to think this is a normal pause event and not an internal detail. Instead, we should have the ChromeProxyService signal to the debugger that this is an "internal pause" and therefore it should not send a regular pause event and should use a completer to signal the pause is done.

Tests are refactored and updated to correctly check for the events when reregistering breakpoints. Specifically, it checks no other events besides the expected ones are sent.

* [ DWDS ] Expose `dtdUri` via `DebugConnection` (#2694)

This is needed to access the DTD instance spawned by DDS.

---------

Co-authored-by: Srujan Gaddam <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants