Skip to content

devtools: Allow registering environment actor from debugger.js#43166

Merged
atbrakhi merged 1 commit intoservo:mainfrom
eerii:register-environment-actor
Mar 11, 2026
Merged

devtools: Allow registering environment actor from debugger.js#43166
atbrakhi merged 1 commit intoservo:mainfrom
eerii:register-environment-actor

Conversation

@eerii
Copy link
Copy Markdown
Member

@eerii eerii commented Mar 11, 2026

This functionality will be used in a follow up patch to implement the getEnvironment message.

Testing: Check mach test-devtools and manual test
Part of: #36027

This functionality will be used in a follow up patch to implement the
getEnvironment message.

Co-authored-by: atbrakhi <atbrakhi@igalia.com>
Signed-off-by: eri <eri@igalia.com>
@eerii eerii requested a review from gterzian as a code owner March 11, 2026 08:50
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Mar 11, 2026
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Mar 11, 2026
@atbrakhi atbrakhi added this pull request to the merge queue Mar 11, 2026
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Mar 11, 2026
Merged via the queue into servo:main with commit 948859b Mar 11, 2026
33 checks passed
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Mar 11, 2026
@eerii eerii deleted the register-environment-actor branch March 12, 2026 08:19
github-merge-queue Bot pushed a commit that referenced this pull request Mar 12, 2026
Initial work to support scopes in the debugger. They don't show at the
moment since we are not processing variables yet. Depends on #43166.

Testing: Ran `mach test-devtools` and manual testing.
Part of: #36027

Signed-off-by: eri <eri@igalia.com>
Co-authored-by: atbrakhi <atbrakhi@igalia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants