Skip to content

devtools: implement clearBreakpoint#42154

Merged
jdm merged 2 commits into
servo:mainfrom
atbrakhi:clear_breakpoints
Jan 27, 2026
Merged

devtools: implement clearBreakpoint#42154
jdm merged 2 commits into
servo:mainfrom
atbrakhi:clear_breakpoints

Conversation

@atbrakhi
Copy link
Copy Markdown
Member

@atbrakhi atbrakhi commented Jan 26, 2026

Add an event listener for clearBreakpoint to debugger.js and the necessary glue to access it from the devtools crate.

Testing: ./mach test-devtools and manual testing.
Fixes: Part of: #36027

@atbrakhi atbrakhi requested a review from gterzian as a code owner January 26, 2026 09:28
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Jan 26, 2026
@atbrakhi atbrakhi requested a review from eerii January 26, 2026 09:29
Add an event listener for `clearBreakpoint` to `debugger.js` and the necessary
glue to access it from the `devtools` crate.

Signed-off-by: atbrakhi <[email protected]>
Co-authored-by: eri <[email protected]>
Comment thread resources/debugger.js Outdated
Comment thread resources/debugger.js
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Jan 26, 2026
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Jan 26, 2026
@atbrakhi atbrakhi requested a review from jdm January 27, 2026 14:32
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Jan 27, 2026
@jdm jdm added this pull request to the merge queue Jan 27, 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 Jan 27, 2026
Merged via the queue into servo:main with commit e28a0f6 Jan 27, 2026
32 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 Jan 27, 2026
@eerii eerii mentioned this pull request Mar 4, 2026
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.

4 participants