Triage 24 commits

Author: delan

commits.txt

@@ -395,7 +395,7 @@ https://github.com/servo/servo/pull/40462	(@Taym95, #40462)	WebGLBuffer: follow
     # This change is preparation for fixing #22550 and implementing
     # off-the-main-thread CSS parsing.
     # Testing: This should not change any behavior so is covered by existing tests.
-https://github.com/servo/servo/pull/40453	(@mrobinson, @mukilan, #40453)	net: Use callbacks instead of IPC channels in the `ImageCache` API (#40453)
+-https://github.com/servo/servo/pull/40453	(@mrobinson, @mukilan, #40453)	net: Use callbacks instead of IPC channels in the `ImageCache` API (#40453)
     # Previously, the `ImageCache` would inform consumers of changes to image
     # availability by messaging an IPC channel. This isn't great, because it
     # requires serialization and deserialization. Nowadays, `ImageCache`s are
@@ -476,7 +476,7 @@ https://github.com/servo/servo/pull/40429	(@mrobinson, @mukilan, #40429)	script:
     # [this](https://hg-edge.mozilla.org/mozilla-central/file/tip/testing/webdriver/src/server.rs#l84). There is also no
     # way to override [Dispatcher](https://hg-edge.mozilla.org/mozilla-central/file/tip/testing/webdriver/src/server.rs#l65).
     # A follow up PR which changes wptrunner to gracefully shutdown servo without killing depends on this.
-https://github.com/servo/servo/pull/40445	(@yezhizhen, #40445)	webdriver: Remove session requirement from Shutdown command (#40445)
+-https://github.com/servo/servo/pull/40445	(@yezhizhen, #40445)	webdriver: Remove session requirement from Shutdown command (#40445)
     # Change API endpoint from "/session/{sessionId}/servo/shutdown" to "/servo/shutdown", similar to [Status
     # command](https://w3c.github.io/webdriver/#dfn-status).
     # This will make it much easier to shutdown Servo properly from wptrunner.
@@ -863,7 +863,7 @@ https://github.com/servo/servo/pull/40411	(@simonwuelker, #40411)	xpath: Handle
     # This removes some unneeded lints, especially `#[allow(unsafe_code)]`.
     # Testing: Refactor
     # Part of: #40383
-https://github.com/servo/servo/pull/40393	(@yezhizhen, #40393)	Reduce unnecessary type conversion and share `WebDriverCommandMsg::LoadUrl` handler across platform (#40393)
+-https://github.com/servo/servo/pull/40393	(@yezhizhen, #40393)	Reduce unnecessary type conversion and share `WebDriverCommandMsg::LoadUrl` handler across platform (#40393)
     # - Reduces unnecessary type conversion between `ServoUrl`, `Url`, string slice
     # - Share `WebDriverCommandMsg::LoadUrl` handler across platform
     # Testing: Covered by existing test. Manually tested for OHOS.
@@ -1321,13 +1321,13 @@ https://github.com/servo/servo/pull/40489	(@eerii, #40489)	tools: Remove wrapped
     # `into_string()` copies the whole url into a new string.
     # Signed-off-by: Narfinger
     # Testing: Should not change any functionality.
-https://github.com/servo/servo/pull/40538	(@atbrakhi, #40538)	servoshell: Move `webdriver_receiver` to shared base (#40538)
+-https://github.com/servo/servo/pull/40538	(@atbrakhi, #40538)	servoshell: Move `webdriver_receiver` to shared base (#40538)
     # This PR eliminates duplicate code while maintaining the same functionality across all platforms.. We move
     # `webdriver_receiver` to shared base where `webdriver_sender` lives
     # Testing: No functional changes. Existing tests should test this.
     # Fixes: Part of https://github.com/servo/servo/issues/40530
 -https://github.com/servo/servo/pull/40537  (@Narfinger, #40537)    script_bindings: Small USVString cleanup (#40537)
-https://github.com/servo/servo/pull/40534	(@atbrakhi, #40534)	servoshell: Move screenshot stability tracking to shared base (#40534)
+-https://github.com/servo/servo/pull/40534	(@atbrakhi, #40534)	servoshell: Move screenshot stability tracking to shared base (#40534)
     # Move screenshot stability tracking to shared base. This PR eliminates duplicate code while maintaining the same
     # functionality across all platforms.
     # Testing: No functional changes. Existing tests should test this.
@@ -1877,7 +1877,7 @@ https://github.com/servo/servo/pull/40590	(@mrobinson, #40590)	script: Use `Scri
     # This change is preparation for splitting out the creation of a new
     # `ScriptThread` and the spawning of its first pipeline.
     # Testing: This should not change observable behavior, so is covered by existing tests.
-https://github.com/servo/servo/pull/40595	(@nicoburns, #40595)	Bump html5ever to 0.36.1, stylo to f319793 (#40595)
+-https://github.com/servo/servo/pull/40595	(@nicoburns, #40595)	Bump html5ever to 0.36.1, stylo to f319793 (#40595)
     # The versions of html5ever were bumped due to a breaking change to the `web_atoms` crate initially being published
     # under the wrong version number.
     # Testing: not required as the code is identical, only version numbers have changed.
@@ -2556,7 +2556,7 @@ https://github.com/servo/servo/pull/40593	(@simonwuelker, #40593)	xpath: Handle
     # - html/semantics/embedded-content/media-elements/seeking/seek-to-max-value.htm
     # - html/semantics/embedded-content/media-elements/seeking/seek-to-negative-time.htm
     # Fixes: https://github.com/servo/servo/issues/34496
-https://github.com/servo/servo/pull/40508	(@Taym95, #40508)	Add AbortSignal garbage collection (#40508)
+-https://github.com/servo/servo/pull/40508	(@Taym95, #40508)	Add AbortSignal garbage collection (#40508)
     # Add AbortSignal garbage collection
     # Testing: all AbortSignal should pass.
     # Fixes: #40481
@@ -2659,7 +2659,7 @@ https://github.com/servo/servo/pull/40616	(@TimvdLippe, #40616)	Implement protoc
     # Testing: Refactoring. Existing tests suffice.
 -https://github.com/servo/servo/pull/40642	(@kkoyung, #40642)	wpt: Remove legacy expectation (#40642)
     # These WPT tests on WebCrypto API were either renamed or removed. We can safely remove these legacy test expectations
-https://github.com/servo/servo/pull/40629	(@kongbai1996, #40629)	Add scroll and fling trace. (#40629)
+-https://github.com/servo/servo/pull/40629	(@kongbai1996, #40629)	Add scroll and fling trace. (#40629)
     # Add scroll and fling trace.
     # Testing: There are no related tests available for now; we will add a test item for frame rate statistics on CI later.
     # Fixes: NA
@@ -2854,7 +2854,8 @@ https://github.com/servo/servo/pull/40762	(@simonwuelker, #40762)	script: Don't
     # panicking on a few instead.
     # Testing: This change adds a simple crashtest
     # Fixes: https://github.com/servo/servo/issues/40719
-https://github.com/servo/servo/pull/40768	(@simonwuelker, #40768)	script: Add messages to exceptions thrown in Element::attachShadow (#40768)
++https://github.com/servo/servo/pull/40768	(@simonwuelker, #40768)	script: Add messages to exceptions thrown in Element::attachShadow (#40768)
+    dom; exception messages
     # Part of https://github.com/servo/servo/issues/40756
 https://github.com/servo/servo/pull/40433	(@mrobinson, #40433)	paint: Do trigger animations when a non-animation paint happens (#40433)
     # Otherwise the `RefreshObsever` can get wedged. This does mean that we
@@ -3053,7 +3054,8 @@ https://github.com/servo/servo/pull/40736	(@mrobinson, #40736)	webgl: Ensure all
     # crashes on the CI.
     # Fixes: #40399.
     # Fixes: #40417.
-https://github.com/servo/servo/pull/40725	(@mrobinson, #40725)	script: Use a `WeakRef` for references to `WebGLRenderingContext` (#40725)
++https://github.com/servo/servo/pull/40725	(@mrobinson, #40725)	script: Use a `WeakRef` for references to `WebGLRenderingContext` (#40725)
+    security; fix use-after-free in WebGL
     # Many WebGL objects refer to a `WebGLRenderingContext` and rely on it for
     # messaging to the `WebGLThread`. This poses a problem, because WebGL
     # objects often need to send a message to the `WebGLThread` during their
@@ -3211,7 +3213,8 @@ https://github.com/servo/servo/pull/40712	(@mrobinson, #40712)	libservo: Trigger
     # should cause a box rebuild. This change does that, hopefully fixing
     # flaky tests.
     # Testing: This doesn't have any test changes, but should fix the test situation described in #40419.
-https://github.com/servo/servo/pull/40578	(@rayguo17, #40578)	Media: Display User Agent Controls widget for video element. (#40578)
++https://github.com/servo/servo/pull/40578	(@rayguo17, #40578)	Media: Display User Agent Controls widget for video element. (#40578)
+    layout; UA controls for <video> (was previously supported for <audio>)
     # Add Contents::ReplacedWithWidget for video elements with UA control widget, to allow Traverse into the children
     # of video element during Box Tree Construction, Generate IndependentFormattingContextContent::ReplacedWithWidgets,
     # to store both the `ReplacedContents` and `BlockFormattingContext`. During fragment tree generation process, first
@@ -3499,7 +3502,7 @@ https://github.com/servo/servo/pull/40666	(@arihant2math, @yezhizhen, #40666)	in
     # Testing: It's just refactoring
     # try run: https://github.com/sagudev/servo/actions/runs/19287700927
 >>> 2025-11-22T07:09:34Z
-https://github.com/servo/servo/pull/40784	(@yezhizhen, #40784)	webdriver: Generalize `implicit_wait` (#40784)
+-https://github.com/servo/servo/pull/40784	(@yezhizhen, #40784)	webdriver: Generalize `implicit_wait` (#40784)
     # I wanted to implement [implicit wait](https://w3c.github.io/webdriver/#dfn-implicit-wait-timeout) for other
     # commands, such as send key/element clear/click. But the current trait bound, parameters, and logic are way too
     # tight and specific.
@@ -3508,7 +3511,7 @@ https://github.com/servo/servo/pull/40784	(@yezhizhen, #40784)	webdriver: Genera
     # Partial fix for https://github.com/servo/servo/issues/40744
     # Testing: Refactor covered by existing tests.
     # Fixes:  Contributes to https://github.com/servo/servo/issues/40744
-https://github.com/servo/servo/pull/40804	(@Loirooriol, #40804)	compositing_traits: Drop unnecessary variable mutability (#40804)
+-https://github.com/servo/servo/pull/40804	(@Loirooriol, #40804)	compositing_traits: Drop unnecessary variable mutability (#40804)
     # This avoids a warning when compiling tests.
     # Testing: Not needed, no behavior change
 +https://github.com/servo/servo/pull/40797	(@Loirooriol, @mrobinson, #40797)	layout: Preserve cached layout results of undamaged descendants (#40797)
@@ -3522,7 +3525,8 @@ https://github.com/servo/servo/pull/40804	(@Loirooriol, #40804)	compositing_trai
     # damage. However, intrinsic sizes don't depend on ancestors for the most part, so now we will only clear the cache
     # if the relayout damage is in the element or a descendant.
     # Testing: Not needed, no change in behavior
-https://github.com/servo/servo/pull/40794	(@mrobinson, #40794)	libservo: Have `WebView` take a `RenderingContext` rather than `Servo` (#40794)
++https://github.com/servo/servo/pull/40794	(@mrobinson, #40794)	libservo: Have `WebView` take a `RenderingContext` rather than `Servo` (#40794)
+    api; per-webview rendering contexts (big part of support for multiple windows)
     # This is the last major change before we can try to implement support for
     # multiple windows in Servo. This change makes it so that each `WebView`
     # (via `WebViewBuilder`) takes the `RenderingContext` as an argument,
@@ -3559,7 +3563,7 @@ https://github.com/servo/servo/pull/40653	(@TimvdLippe, #40653)	libservo: Notify
     # or window, so this change just removes it entirely.
     # Testing: This should make the main loop of headless servoshell more efficient,
     # but I do not think this difference is going to be observable in any way.
-https://github.com/servo/servo/pull/40792	(@mrobinson, #40792)	servoshell: Move `Minibrowser` to headed window (#40792)
+-https://github.com/servo/servo/pull/40792	(@mrobinson, #40792)	servoshell: Move `Minibrowser` to headed window (#40792)
     # Instead of storing the `Minibrowser` egui struct in the
     # `RunningAppState` which is used for both headed and headless windows,
     # move it to the headed window implementation. This addresses a lot of
@@ -3579,7 +3583,7 @@ https://github.com/servo/servo/pull/40788	(@mukilan, @mrobinson, #40788)	composi
     # `PainterId` or `WebViewId`.
     # This change is to enable support for per-`WebView`s  `RenderingContext`s.
     # Testing: This change shouldn't change behavior, so existing WPT tests should cover it.
-https://github.com/servo/servo/pull/40766	(@atbrakhi, #40766)	servoshell: Share WebView management between desktop and EGL platforms (#40766)
+-https://github.com/servo/servo/pull/40766	(@atbrakhi, #40766)	servoshell: Share WebView management between desktop and EGL platforms (#40766)
     # Desktop and EGL implementations were maintaining identical WebView collection logic (HashMap, creation order
     # tracking, focus management).
     # Extracted this into a shared WebViewCollection struct in RunningAppStateBase. Both platforms now use the common
@@ -3631,7 +3635,8 @@ https://github.com/servo/servo/pull/40647	(@simonwuelker, #40647)	fonts: Compute
     #         }
     #     The hackfest
     # ```
-https://github.com/servo/servo/pull/40821	(@nicoburns, #40821)	Upgrade Taffy to v0.9.2 (#40821)
++https://github.com/servo/servo/pull/40821	(@nicoburns, #40821)	Upgrade Taffy to v0.9.2 (#40821)
+    crash; fix panic in grid layout
     # Upgrades the patch version of Taffy (release notes: https://github.com/DioxusLabs/taffy/releases/tag/v0.9.2)
     # Testing: WPT tests
     # Fixes https://github.com/servo/servo/issues/40730
@@ -3662,14 +3667,14 @@ https://github.com/servo/servo/pull/40817	(@jdm, #40817)	testing: Remove ahem.cs
     # holdover from this. This change renames this struct to `Gui` to better
     # reflect what it does.
     # Testing: This just renames some data structures, so should not need new tests.
-https://github.com/servo/servo/pull/40787	(@eerii, #40787)	devtools: Add Environment actor boilerplate and Object actor encode (#40787)
+-https://github.com/servo/servo/pull/40787	(@eerii, #40787)	devtools: Add Environment actor boilerplate and Object actor encode (#40787)
     # Prequisite for implementing pausing in the debugger. The `EnvironmentActor` will be used in the future from the
     # `FrameActor`, we need the struct defined to start to implement it.
     # The `ObjectActorMsg` is not only used by the `EnvoironmentActor`, but also by the `ThreadActor` when replying to
     # `interrupt`.
     # Testing: Deferred until the pause is fully implemented.
     # Fixes: Part of #36027.
-https://github.com/servo/servo/pull/40786	(@eerii, #40786)	devtools: Pause actor boilerplate (#40786)
+-https://github.com/servo/servo/pull/40786	(@eerii, #40786)	devtools: Pause actor boilerplate (#40786)
     # Prequisite for implementing pausing in the debugger. This will be part of the `ThreadActor` response to `interrupt`. I
     # decided to split the change in multiple patches since it is quite big. Continuation of #38824.
     # Testing: Deferred until the pause is fully implemented.
@@ -3689,7 +3694,7 @@ https://github.com/servo/servo/pull/40811	(@jdm, #40811)	script: Improve spec-co
     # Fixes: #40808
 +https://github.com/servo/servo/pull/40692	(@jdm, #40692)	Run SVG tests in CI. (#40692)
     # Now that we have rudimentary SVG support, we should probably run the testsuite.
-https://github.com/servo/servo/pull/40806	(@yezhizhen, #40806)	openxr: Fix DirectX 11 build warning and reduce visiblity (#40806)
+-https://github.com/servo/servo/pull/40806	(@yezhizhen, #40806)	openxr: Fix DirectX 11 build warning and reduce visiblity (#40806)
     # This mod is only compiled for Win therefore hasn't been captured by Lint.
 -https://github.com/servo/servo/pull/40809	(@kkoyung, #40809)	script: Rename `ExportedKey::Raw` to `ExportedKey::Bytes` (#40809)
     # The enum variant `ExportedKey::Raw(Vec)` is actually used when a key is exported in "raw", "spki" or "pkcs8"
@@ -3722,7 +3727,7 @@ https://github.com/servo/servo/pull/40806	(@yezhizhen, #40806)	openxr: Fix Direc
     # a rebase manually by commenting `@dependabot rebase`.
     # [//]: # (dependabot-automerge-start)
     # [//]: # (dependabot-automerge-end)
-https://github.com/servo/servo/pull/40791	(@eerii, #40791)	devtools: Frame actor boilerplate (#40791)
+-https://github.com/servo/servo/pull/40791	(@eerii, #40791)	devtools: Frame actor boilerplate (#40791)
     # Prequisite for implementing pausing in the debugger. This will be part of the `ThreadActor` response to
     # `interrupt`. Continuation of #38824.
     # Depends on #40787 for `EnvironmentActor*`.
@@ -3965,7 +3970,7 @@ https://github.com/servo/servo/pull/40301	(@jdm, @uthmaniv, #40301)	fonts: Add W
     # changes means that this fix will only have to happen in one place
     # instead of two.
     # Testing: This should not change behavior and thus is covered by existing tests.
-https://github.com/servo/servo/pull/40812	(@jdm, #40812)	script: Replace strings for performance entry types with dedicated enum. (#40812)
+-https://github.com/servo/servo/pull/40812	(@jdm, #40812)	script: Replace strings for performance entry types with dedicated enum. (#40812)
     # This both reduces unnecessary string comparisons and fixes several cases where two string arguments were passed
     # in the wrong order.
     # Depends on https://github.com/servo/servo/pull/40811.
@@ -4009,7 +4014,8 @@ https://github.com/servo/servo/pull/40852	(@delan, #40852)	ci: Use new queue API
     # [4](https://github.com/servo/servo/actions/runs/19624564171),
     # [5](https://github.com/servo/servo/actions/runs/19624566167),
     # [6](https://github.com/servo/servo/actions/runs/19624568092)
-https://github.com/servo/servo/pull/40842	(@Taym95, #40842)	Implement brotli support for CompressionStream and DecompressionStream (#40842)
++https://github.com/servo/servo/pull/40842	(@Taym95, #40842)	Implement brotli support for CompressionStream and DecompressionStream (#40842)
+    dom; new CompressionStream("brotli") and new DecompressionStream("brotli")
     # Implement brotli support for CompressionStream and DecompressionStream
     # Testing: brotli test should pass
     # Fixes: #40831
@@ -4343,7 +4349,7 @@ https://github.com/servo/servo/pull/40875	(@tharkum, #40875)	html: Use WeakRef f
     # Addresses part of https://github.com/servo/servo/issues/34573
 -https://github.com/servo/servo/pull/40874	(@yezhizhen, #40874)	Fix compilation warning for `background_hang_monitor` (#40874)
     # Fix warnings exposed by 0a0a20a9c6b7983ed96fb86715ada5b9f6d61f2e
-https://github.com/servo/servo/pull/40872	(@yezhizhen, #40872)	Remove unnecessary clones in various places (#40872)
+-https://github.com/servo/servo/pull/40872	(@yezhizhen, #40872)	Remove unnecessary clones in various places (#40872)
     # Testing: This should not change anything. Still, things have to build.
 >>> 2025-11-27T06:15:41Z
 -https://github.com/servo/servo/pull/40910	(@dependabot[bot], @dependabot[bot], #40910)	build(deps): bump tracing-attributes from 0.1.30 to 0.1.31 (#40910)