Skip to content

script: Pass &mut JSContext in more places#43252

Merged
Gae24 merged 7 commits into
servo:mainfrom
Gae24:temp-cx-cleanup
Mar 14, 2026
Merged

script: Pass &mut JSContext in more places#43252
Gae24 merged 7 commits into
servo:mainfrom
Gae24:temp-cx-cleanup

Conversation

@Gae24

@Gae24 Gae24 commented Mar 13, 2026

Copy link
Copy Markdown
Contributor

Continuation of #43220, removes many temp_cx calls introduced by that.

Testing: No behaviour change, a successful build is enough.
Fixes: #43241 #43240

@Gae24 Gae24 force-pushed the temp-cx-cleanup branch 4 times, most recently from 35b002e to 6efcfc4 Compare March 13, 2026 19:27
Gae24 added 7 commits March 14, 2026 09:29
Signed-off-by: Gae24 <96017547+Gae24@users.noreply.github.com>
Signed-off-by: Gae24 <96017547+Gae24@users.noreply.github.com>
Signed-off-by: Gae24 <96017547+Gae24@users.noreply.github.com>
Signed-off-by: Gae24 <96017547+Gae24@users.noreply.github.com>
Signed-off-by: Gae24 <96017547+Gae24@users.noreply.github.com>
Signed-off-by: Gae24 <96017547+Gae24@users.noreply.github.com>
Signed-off-by: Gae24 <96017547+Gae24@users.noreply.github.com>
@Gae24 Gae24 force-pushed the temp-cx-cleanup branch from 6efcfc4 to be6119b Compare March 14, 2026 08:30
@Gae24 Gae24 marked this pull request as ready for review March 14, 2026 08:37
@Gae24 Gae24 requested a review from gterzian as a code owner March 14, 2026 08:37
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Mar 14, 2026
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Mar 14, 2026
@Gae24 Gae24 added this pull request to the merge queue Mar 14, 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 14, 2026
Merged via the queue into servo:main with commit e8b581a Mar 14, 2026
34 checks passed
@Gae24 Gae24 deleted the temp-cx-cleanup branch March 14, 2026 10:37
@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 14, 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.

Pass &mut JSContext to Element::attach_shadow

3 participants