Skip to content

Conversation

@eerii
Copy link
Contributor

@eerii eerii commented Nov 21, 2025

Prequisite for implementing pausing in the debugger. This will be part of the ThreadActor response to interrupt. Continuation of #38824.

Depends on #40787 for EnvironmentActor*.

Testing: Deferred until the pause is fully implemented.
Fixes: Part of #36027.

@codecov-commenter
Copy link

⚠️ JUnit XML file not found

The CLI was unable to find any JUnit XML files to upload.
For more help, visit our troubleshooting guide.

@eerii eerii force-pushed the devtools-frame branch 2 times, most recently from 8062781 to bbb1c81 Compare November 21, 2025 12:00
Signed-off-by: eri <eri@igalia.com>
@eerii
Copy link
Contributor Author

eerii commented Nov 22, 2025

Rebased and removed the dead code allow from the environment actor. The one here and the one in pause will be removed with the changes in the thread actor.

@eerii eerii marked this pull request as ready for review November 22, 2025 19:43
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Nov 22, 2025
* License, v. 2.0. If a copy of the MPL was not distributed with this
* file, You can obtain one at https://mozilla.org/MPL/2.0/. */

// TODO: Remove once the actor is used.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
// TODO: Remove once the actor is used.

@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Nov 23, 2025
Signed-off-by: eri <eri@igalia.com>
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Nov 23, 2025
@jdm jdm added this pull request to the merge queue Nov 23, 2025
@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 Nov 23, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 23, 2025
@servo-highfive servo-highfive added S-tests-failed The changes caused existing tests to fail. and removed S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. labels Nov 23, 2025
@yezhizhen yezhizhen added this pull request to the merge queue Nov 24, 2025
@servo-highfive servo-highfive added S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. and removed S-tests-failed The changes caused existing tests to fail. labels Nov 24, 2025
Merged via the queue into servo:main with commit 879a69d Nov 24, 2025
41 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 Nov 24, 2025
delan added a commit that referenced this pull request Nov 24, 2025
{"fail_fast": false, "matrix": [{"name": "Windows", "workflow": "windows", "wpt": false, "profile": "release", "unit_tests": false, "build_libservo": false, "bencher": false, "coverage": false, "build_args": "", "wpt_args": "", "number_of_wpt_chunks": 20}, {"name": "Linux", "workflow": "linux", "wpt": false, "profile": "release", "unit_tests": false, "build_libservo": false, "bencher": false, "coverage": false, "build_args": "", "wpt_args": "", "number_of_wpt_chunks": 20}, {"name": "Lint", "workflow": "lint", "wpt": false, "profile": "release", "unit_tests": false, "build_libservo": false, "bencher": false, "coverage": false, "build_args": "", "wpt_args": "", "number_of_wpt_chunks": 20}]}
@eerii eerii deleted the devtools-frame branch November 26, 2025 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-awaiting-review There is new code that needs to be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants