Skip to content

Script: Use &mut JSContext in BroadcastChannel::PostMessage#42417

Merged
TimvdLippe merged 2 commits intoservo:mainfrom
BryanSmith00:broadcastchannel-postmessage
Feb 7, 2026
Merged

Script: Use &mut JSContext in BroadcastChannel::PostMessage#42417
TimvdLippe merged 2 commits intoservo:mainfrom
BryanSmith00:broadcastchannel-postmessage

Conversation

@BryanSmith00
Copy link
Copy Markdown
Contributor

Replace crate::script_runtime::JSContext with js::context::JSContext in BroadcastChannel::PostMessage and update interface binding.

Testing: Builds and runs locally.
Part of: #42347

@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Feb 6, 2026
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Feb 7, 2026
@TimvdLippe TimvdLippe enabled auto-merge February 7, 2026 07:10
@TimvdLippe
Copy link
Copy Markdown
Contributor

Please run ./mach fmt to reformat the files

auto-merge was automatically disabled February 7, 2026 19:19

Head branch was pushed to by a user without write access

@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Feb 7, 2026
@TimvdLippe TimvdLippe enabled auto-merge February 7, 2026 19:46
@TimvdLippe TimvdLippe added this pull request to the merge queue Feb 7, 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 Feb 7, 2026
Merged via the queue into servo:main with commit a6a3b1a Feb 7, 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 Feb 7, 2026
@BryanSmith00 BryanSmith00 deleted the broadcastchannel-postmessage branch February 7, 2026 23:05
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.

3 participants