Skip to content

devtools: Default implementation for handle_message#41076

Merged
mukilan merged 1 commit intoservo:mainfrom
eerii:devtools-default-handle-message
Dec 5, 2025
Merged

devtools: Default implementation for handle_message#41076
mukilan merged 1 commit intoservo:mainfrom
eerii:devtools-default-handle-message

Conversation

@eerii
Copy link
Copy Markdown
Member

@eerii eerii commented Dec 5, 2025

Some cleanup to avoid having to specify an empty handle_message method if it is not implemented or not necessary. Removed the TODOs from Environment and Pause since they don't respond to any message.

Testing: This patch doesn't change behaviour.

@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Dec 5, 2025
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Dec 5, 2025
@mukilan mukilan added this pull request to the merge queue Dec 5, 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 Dec 5, 2025
Merged via the queue into servo:main with commit 0358619 Dec 5, 2025
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 Dec 5, 2025
@eerii eerii deleted the devtools-default-handle-message branch December 5, 2025 19:57
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