Skip to content

script: Pass &mut JSContext to DOM apis that call Element::create#43108

Merged
TimvdLippe merged 2 commits into
servo:mainfrom
Gae24:element-create-js
Mar 9, 2026
Merged

script: Pass &mut JSContext to DOM apis that call Element::create#43108
TimvdLippe merged 2 commits into
servo:mainfrom
Gae24:element-create-js

Conversation

@Gae24

@Gae24 Gae24 commented Mar 9, 2026

Copy link
Copy Markdown
Contributor

Testing: No behaviour change, a successful build is enough.
Part of #40600

Signed-off-by: Gae24 <96017547+Gae24@users.noreply.github.com>
@Gae24 Gae24 requested a review from gterzian as a code owner March 9, 2026 09:47
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Mar 9, 2026
@Gae24 Gae24 changed the title script: start passing &mut JSContext to DOM apis that call Element::create script: Pass &mut JSContext to DOM apis that call Element::create Mar 9, 2026
…lection`

Signed-off-by: Gae24 <96017547+Gae24@users.noreply.github.com>
@Gae24 Gae24 force-pushed the element-create-js branch from 53d82e8 to 1f25d23 Compare March 9, 2026 09:59
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Mar 9, 2026
@TimvdLippe TimvdLippe enabled auto-merge March 9, 2026 10:05
@TimvdLippe TimvdLippe added this pull request to the merge queue Mar 9, 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 9, 2026
Merged via the queue into servo:main with commit fa86bbe Mar 9, 2026
30 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 Mar 9, 2026
github-merge-queue Bot pushed a commit that referenced this pull request Mar 10, 2026
…`Node::clone` (#43130)

Continuation of #43108, two new `temp_cx()` calls were required:
- inside `maybe_clone_an_option_into_selectedcontent` since it's part of
a markup5ever trait
- inside `serialize_and_cache_subtree` replacing a `CanGc::note()` call,
propagating it inside reflow code will require even more effort.

Testing: No behaviour change, a successful build is enough.
Part of #40600

---------

Signed-off-by: Gae24 <96017547+Gae24@users.noreply.github.com>
@Gae24 Gae24 deleted the element-create-js branch March 24, 2026 16:21
offline-ant pushed a commit to offline-ant/havi that referenced this pull request Jun 4, 2026
…servo#43108)

Testing: No behaviour change, a successful build is enough.
Part of servo#40600

---------

Signed-off-by: Gae24 <96017547+Gae24@users.noreply.github.com>
(cherry picked from commit fa86bbe)
offline-ant pushed a commit to offline-ant/havi that referenced this pull request Jun 4, 2026
…`Node::clone` (servo#43130)

Continuation of servo#43108, two new `temp_cx()` calls were required:
- inside `maybe_clone_an_option_into_selectedcontent` since it's part of
a markup5ever trait
- inside `serialize_and_cache_subtree` replacing a `CanGc::note()` call,
propagating it inside reflow code will require even more effort.

Testing: No behaviour change, a successful build is enough.
Part of servo#40600

---------

Signed-off-by: Gae24 <96017547+Gae24@users.noreply.github.com>
(cherry picked from commit 4ef22ed)
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