commit 5135adaf64023656741bec81316a01abcfa0c488 parent 4a5ca0d853a2aef17b689e63ad58616fab42e5f2 Author: David Grogan <dgrogan@chromium.org> Date: Wed, 3 Dec 2025 13:57:00 +0000 Bug 2003252 [wpt PR 56362] - [viewport units] Respond to dynamic changes to :root's scrollbars, a=testonly Automatic update from web-platform-tests [viewport units] Respond to dynamic changes to :root's scrollbars This is modeled on root font size change logic. When :root's unconditional scrollbars change, we clear anything viewport-dependent from matched_properties_cache_ then run kRecalcDescendants. Bug: 354751900 Change-Id: I7c9a5c11a166e5221f187acdf687a55ecb0010e3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7208965 Commit-Queue: David Grogan <dgrogan@chromium.org> Reviewed-by: Anders Hartvoll Ruud <andruud@chromium.org> Cr-Commit-Position: refs/heads/main@{#1552029} -- wpt-commits: d7ca0083bcca580a60357662d784f53f31f7d295 wpt-pr: 56362 Diffstat:
5 files changed, 13 insertions(+), 31 deletions(-)
diff --git a/testing/web-platform/tests/css/css-mixins/contents-nested-declarations-fallback.html b/testing/web-platform/tests/css/css-mixins/contents-nested-declarations-fallback.html @@ -9,7 +9,7 @@ <body> <style> - @mixin --m1(@contents) { + @mixin --m1() { @contents { color: green; } @@ -28,7 +28,7 @@ <style> - @mixin --m2(@contents) { + @mixin --m2() { @contents { color: red; } diff --git a/testing/web-platform/tests/css/css-mixins/contents-nested-declarations.html b/testing/web-platform/tests/css/css-mixins/contents-nested-declarations.html @@ -9,7 +9,7 @@ <body> <style> - @mixin --m1(@contents) { + @mixin --m1() { @contents; } #e1::after { @@ -28,7 +28,7 @@ <style> - @mixin --m2(@contents) { + @mixin --m2() { @contents; } /* Should match <div id=e2> with the specificity of :where(#e2) (zero), diff --git a/testing/web-platform/tests/css/css-mixins/contents-rule.html b/testing/web-platform/tests/css/css-mixins/contents-rule.html @@ -9,7 +9,7 @@ <body> <style> - @mixin --m1(@contents) { + @mixin --m1() { @contents; } #e1 { @@ -27,7 +27,7 @@ <style> - @mixin --m2(@contents) { + @mixin --m2() { @contents } #e2 { @@ -45,7 +45,7 @@ <style> - @mixin --m3(@contents) { + @mixin --m3() { &.a { @contents { color: blue; } } @@ -65,7 +65,7 @@ <style> - @mixin --m4(@contents) { + @mixin --m4() { &.c { @contents { color: green; } } @@ -85,25 +85,7 @@ <style> - @mixin --m5() { - @contents { color: red !important; } - color: green; - } - #e5 { - @apply --m5 { color: red !important; } - } - </style> - <div id="e5">This text should be green.</div> - <script> - test(() => { - let target = document.getElementById('e5'); - assert_equals(getComputedStyle(target).color, 'rgb(0, 128, 0)'); - }, '@contents is ignored if there is no @contents parameter'); - </script> - - - <style> - @mixin --m6(@contents) { + @mixin --m6() { @contents { } color: green; } @@ -121,7 +103,7 @@ <style> - @mixin --m7(@contents) { + @mixin --m7() { @contents; color: green; } diff --git a/testing/web-platform/tests/css/css-mixins/mixin-cssom.tentative.html b/testing/web-platform/tests/css/css-mixins/mixin-cssom.tentative.html @@ -45,7 +45,7 @@ <style id=style3> - @mixin --m2(@contents) { + @mixin --m2() { @contents } </style> @@ -53,7 +53,7 @@ test(() => { let ss = style3.sheet; assert_equals(ss.cssRules[0].cssText, -`@mixin --m2(@contents) { +`@mixin --m2() { @contents; }`); }, 'serialization of @mixin with @contents'); diff --git a/testing/web-platform/tests/css/css-values/viewport-units-scrollbars-dynamic-001.html b/testing/web-platform/tests/css/css-values/viewport-units-scrollbars-dynamic-001.html @@ -11,7 +11,7 @@ <script> document.body.offsetTop; - document.documentElement.style.overflowY = 'scroll'; + document.documentElement.style.overflow = 'scroll'; </script> </body>