Support margin-trim on block containers. [chromium/src : main]
0 views
Skip to first unread message
Morten Stenshorne (Gerrit)
Feb 5, 2026, 6:25:19 AM (8 days ago) Feb 5
to Rune Lillesveen, Ian Kilpatrick, Menard, Alexis, Chromium Metrics Reviews, chromium...@chromium.org, apavlo...@chromium.org, asvitkine...@chromium.org, blink-re...@chromium.org, blink-re...@chromium.org, blink-revi...@chromium.org, blink-rev...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org, zol...@webkit.org
Attention needed from Ian Kilpatrick and Rune Lillesveen
New activity on the change![]( "Open in Gerrit")
Related details
Attention is currently required from:
- Ian Kilpatrick
- Rune Lillesveen
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Ian Kilpatrick (Gerrit)
Feb 5, 2026, 7:22:21 AM (8 days ago) Feb 5
to Morten Stenshorne, Rune Lillesveen, Menard, Alexis, Chromium Metrics Reviews, chromium...@chromium.org, apavlo...@chromium.org, asvitkine...@chromium.org, blink-re...@chromium.org, blink-re...@chromium.org, blink-revi...@chromium.org, blink-rev...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org, zol...@webkit.org
Attention needed from Morten Stenshorne and Rune Lillesveen
Ian Kilpatrick voted and added 1 comment![]( "Open in Gerrit")
Votes added by Ian Kilpatrick
| Code-Review | +1 |
1 comment
File third_party/blink/renderer/core/css/css_properties.json5
Line 4224, Patchset 8 (Latest):
invalidate: ["layout", "paint"],Ian Kilpatrick . unresolved
arguably only layout?
Related details
Attention is currently required from:
- Morten Stenshorne
- Rune Lillesveen
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
No-Unresolved-Comments
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Rune Lillesveen (Gerrit)
Feb 5, 2026, 7:30:43 AM (8 days ago) Feb 5
to Morten Stenshorne, Ian Kilpatrick, Rune Lillesveen, Menard, Alexis, Chromium Metrics Reviews, chromium...@chromium.org, apavlo...@chromium.org, asvitkine...@chromium.org, blink-re...@chromium.org, blink-re...@chromium.org, blink-revi...@chromium.org, blink-rev...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org, zol...@webkit.org
Attention needed from Morten Stenshorne
Rune Lillesveen added 1 comment![]( "Open in Gerrit")
File third_party/blink/renderer/core/css/css_margin_trim_value.h
Line 16, Patchset 8 (Latest):
class CSSMarginTrimValue : public CSSValue {Rune Lillesveen . unresolved
We typically use CSSValuePair instead of creating new CSSValue classes and do the mapping to e.g. a mask for ApplyValue().
Related details
Attention is currently required from:
- Morten Stenshorne
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
No-Unresolved-Comments
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Morten Stenshorne (Gerrit)
Feb 5, 2026, 8:10:24 AM (8 days ago) Feb 5
to Ian Kilpatrick, Rune Lillesveen, Menard, Alexis, Chromium Metrics Reviews, chromium...@chromium.org, apavlo...@chromium.org, asvitkine...@chromium.org, blink-re...@chromium.org, blink-re...@chromium.org, blink-revi...@chromium.org, blink-rev...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org, zol...@webkit.org
Attention needed from Rune Lillesveen
Morten Stenshorne added 2 comments![]( "Open in Gerrit")
File third_party/blink/renderer/core/css/css_margin_trim_value.h
Line 16, Patchset 8:
class CSSMarginTrimValue : public CSSValue {Rune Lillesveen . unresolved
We typically use CSSValuePair instead of creating new CSSValue classes and do the mapping to e.g. a mask for ApplyValue().
Morten Stenshorne
In patch set 6 I created a CSSValueList.
https://chromium-review.googlesource.com/c/chromium/src/+/7544283/6
... which worked fine except for serialization.
So I ended up with this, so that both getPropertyValue() and getComputedStyle() would return "block-start block-end" as "block". There are tests for this.
Why would CSSValuePair be any better than CSSValueList? This property takes between one and four values.
File third_party/blink/renderer/core/css/css_properties.json5
Line 4224, Patchset 8:
invalidate: ["layout", "paint"],Ian Kilpatrick . resolved
Morten Stenshornearguably only layout?
Done
Related details
Attention is currently required from:
- Rune Lillesveen
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
No-Unresolved-Comments
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Morten Stenshorne (Gerrit)
Feb 5, 2026, 8:11:54 AM (8 days ago) Feb 5
to Ian Kilpatrick, Rune Lillesveen, Menard, Alexis, Chromium Metrics Reviews, chromium...@chromium.org, apavlo...@chromium.org, asvitkine...@chromium.org, blink-re...@chromium.org, blink-re...@chromium.org, blink-revi...@chromium.org, blink-rev...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org, zol...@webkit.org
Attention needed from Rune Lillesveen
Morten Stenshorne added 1 comment![]( "Open in Gerrit")
File third_party/blink/renderer/core/css/css_margin_trim_value.h
Line 16, Patchset 8:
class CSSMarginTrimValue : public CSSValue {Rune Lillesveen . unresolved
Morten StenshorneWe typically use CSSValuePair instead of creating new CSSValue classes and do the mapping to e.g. a mask for ApplyValue().
In patch set 6 I created a CSSValueList.
https://chromium-review.googlesource.com/c/chromium/src/+/7544283/6... which worked fine except for serialization.
So I ended up with this, so that both getPropertyValue() and getComputedStyle() would return "block-start block-end" as "block". There are tests for this.
Why would CSSValuePair be any better than CSSValueList? This property takes between one and four values.
Morten Stenshorne
Should I rather go back to patch set 6, and simplify the values in MarginTrim::ParseSingleValue()?
Related details
Attention is currently required from:
- Rune Lillesveen
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
No-Unresolved-Comments
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Rune Lillesveen (Gerrit)
Feb 5, 2026, 8:23:15 AM (8 days ago) Feb 5
to Morten Stenshorne, Ian Kilpatrick, Rune Lillesveen, Menard, Alexis, Chromium Metrics Reviews, chromium...@chromium.org, apavlo...@chromium.org, asvitkine...@chromium.org, blink-re...@chromium.org, blink-re...@chromium.org, blink-revi...@chromium.org, blink-rev...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org, zol...@webkit.org
Attention needed from Morten Stenshorne
Rune Lillesveen added 1 comment![]( "Open in Gerrit")
File third_party/blink/renderer/core/css/css_margin_trim_value.h
Line 16, Patchset 8:
class CSSMarginTrimValue : public CSSValue {Rune Lillesveen . unresolved
Morten StenshorneWe typically use CSSValuePair instead of creating new CSSValue classes and do the mapping to e.g. a mask for ApplyValue().
Morten StenshorneIn patch set 6 I created a CSSValueList.
https://chromium-review.googlesource.com/c/chromium/src/+/7544283/6... which worked fine except for serialization.
So I ended up with this, so that both getPropertyValue() and getComputedStyle() would return "block-start block-end" as "block". There are tests for this.
Why would CSSValuePair be any better than CSSValueList? This property takes between one and four values.
Should I rather go back to patch set 6, and simplify the values in MarginTrim::ParseSingleValue()?
Rune Lillesveen
Why would CSSValuePair be any better than CSSValueList? This property takes between one and four values.
I thought it took two, so yeah, a list would be the right choice then.
Should I rather go back to patch set 6, and simplify the values in MarginTrim::ParseSingleValue()?
That's what would match how we typically do it, at least for re-ordering.
Related details
Attention is currently required from:
- Morten Stenshorne
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
No-Unresolved-Comments
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Morten Stenshorne (Gerrit)
Feb 5, 2026, 8:49:28 AM (8 days ago) Feb 5
to Ian Kilpatrick, Rune Lillesveen, Menard, Alexis, Chromium Metrics Reviews, chromium...@chromium.org, apavlo...@chromium.org, asvitkine...@chromium.org, blink-re...@chromium.org, blink-re...@chromium.org, blink-revi...@chromium.org, blink-rev...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org, zol...@webkit.org
Attention needed from Ian Kilpatrick and Rune Lillesveen
Morten Stenshorne added 1 comment![]( "Open in Gerrit")
File third_party/blink/renderer/core/css/css_margin_trim_value.h
Line 16, Patchset 8:
class CSSMarginTrimValue : public CSSValue {Rune Lillesveen . resolved
Morten StenshorneWe typically use CSSValuePair instead of creating new CSSValue classes and do the mapping to e.g. a mask for ApplyValue().
Morten StenshorneIn patch set 6 I created a CSSValueList.
https://chromium-review.googlesource.com/c/chromium/src/+/7544283/6... which worked fine except for serialization.
So I ended up with this, so that both getPropertyValue() and getComputedStyle() would return "block-start block-end" as "block". There are tests for this.
Why would CSSValuePair be any better than CSSValueList? This property takes between one and four values.
Rune LillesveenShould I rather go back to patch set 6, and simplify the values in MarginTrim::ParseSingleValue()?
Why would CSSValuePair be any better than CSSValueList? This property takes between one and four values.
I thought it took two, so yeah, a list would be the right choice then.
Should I rather go back to patch set 6, and simplify the values in MarginTrim::ParseSingleValue()?
That's what would match how we typically do it, at least for re-ordering.
Attention is currently required from:
- Ian Kilpatrick
- Rune Lillesveen
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Rune Lillesveen (Gerrit)
Feb 5, 2026, 9:39:49 AM (8 days ago) Feb 5
to Morten Stenshorne, Ian Kilpatrick, Rune Lillesveen, Menard, Alexis, Chromium Metrics Reviews, chromium...@chromium.org, apavlo...@chromium.org, asvitkine...@chromium.org, blink-re...@chromium.org, blink-re...@chromium.org, blink-revi...@chromium.org, blink-rev...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org, zol...@webkit.org
Attention needed from Ian Kilpatrick and Morten Stenshorne
Rune Lillesveen added 2 comments![]( "Open in Gerrit")
File third_party/blink/renderer/core/css/css_properties.json5
Line 4222, Patchset 10 (Latest):
typedom_types: ["Keyword"],Rune Lillesveen . unresolved
How did this in combination with ConvertFlags and all the keywords above not crash StylePropertyMapTest_CSSKeywordValuesTest?
Related details
Attention is currently required from:
- Ian Kilpatrick
- Morten Stenshorne
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
No-Unresolved-Comments
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Morten Stenshorne (Gerrit)
Feb 5, 2026, 11:54:31 AM (8 days ago) Feb 5
to Ian Kilpatrick, Rune Lillesveen, Menard, Alexis, Chromium Metrics Reviews, chromium...@chromium.org, apavlo...@chromium.org, asvitkine...@chromium.org, blink-re...@chromium.org, blink-re...@chromium.org, blink-revi...@chromium.org, blink-rev...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org, zol...@webkit.org
Attention needed from Ian Kilpatrick and Rune Lillesveen
Morten Stenshorne added 2 comments![]( "Open in Gerrit")
File third_party/blink/renderer/core/css/css_properties.json5
How did this in combination with ConvertFlags and all the keywords above not crash StylePropertyMapTest_CSSKeywordValuesTest?
Morten Stenshorne
Oh, wait. It appears that I re-added `keywords`. This will cause a crash. Removed again.
Line 4223, Patchset 10:
computable: false,Rune Lillesveen . resolved
Morten StenshorneWhy is this false?
Removed. Doesn't have any effect?
Related details
Attention is currently required from:
- Ian Kilpatrick
- Rune Lillesveen
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Rune Lillesveen (Gerrit)
Feb 6, 2026, 3:05:24 AM (7 days ago) Feb 6
to Morten Stenshorne, Ian Kilpatrick, Rune Lillesveen, Menard, Alexis, Chromium Metrics Reviews, chromium...@chromium.org, apavlo...@chromium.org, asvitkine...@chromium.org, blink-re...@chromium.org, blink-re...@chromium.org, blink-revi...@chromium.org, blink-rev...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org, zol...@webkit.org
Attention needed from Ian Kilpatrick and Morten Stenshorne
Rune Lillesveen added 1 comment![]( "Open in Gerrit")
File third_party/blink/renderer/core/css/css_properties.json5
Line 4223, Patchset 10:
computable: false,Rune Lillesveen . resolved
Morten StenshorneWhy is this false?
Removed. Doesn't have any effect?
Attention is currently required from:
- Ian Kilpatrick
- Morten Stenshorne
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Ian Kilpatrick (Gerrit)
Feb 6, 2026, 4:22:53 AM (7 days ago) Feb 6
to Morten Stenshorne, Rune Lillesveen, Menard, Alexis, Chromium Metrics Reviews, chromium...@chromium.org, apavlo...@chromium.org, asvitkine...@chromium.org, blink-re...@chromium.org, blink-re...@chromium.org, blink-revi...@chromium.org, blink-rev...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org, zol...@webkit.org
Attention needed from Morten Stenshorne
Ian Kilpatrick voted Code-Review+1![]( "Open in Gerrit")
Attention is currently required from:
- Morten Stenshorne
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Morten Stenshorne (Gerrit)
Feb 6, 2026, 4:23:56 AM (7 days ago) Feb 6
to Ian Kilpatrick, Rune Lillesveen, Menard, Alexis, Chromium Metrics Reviews, chromium...@chromium.org, apavlo...@chromium.org, asvitkine...@chromium.org, blink-re...@chromium.org, blink-re...@chromium.org, blink-revi...@chromium.org, blink-rev...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org, zol...@webkit.org
Attention needed from Rune Lillesveen
Morten Stenshorne added 1 comment![]( "Open in Gerrit")
File third_party/blink/renderer/core/css/css_properties.json5
Line 4223, Patchset 10:
computable: false,Rune Lillesveen . resolved
Morten StenshorneWhy is this false?
Rune LillesveenRemoved. Doesn't have any effect?
Seems to have had an effect given the latest run?
Morten Stenshorne
It sure did.
There was also a parser mistake (assuming that everything is keywords) that I've now fixed.
Related details
Attention is currently required from:
- Rune Lillesveen
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Rune Lillesveen (Gerrit)
Feb 6, 2026, 4:25:40 AM (7 days ago) Feb 6
to Morten Stenshorne, Rune Lillesveen, Ian Kilpatrick, Menard, Alexis, Chromium Metrics Reviews, chromium...@chromium.org, apavlo...@chromium.org, asvitkine...@chromium.org, blink-re...@chromium.org, blink-re...@chromium.org, blink-revi...@chromium.org, blink-rev...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org, zol...@webkit.org
Attention needed from Morten Stenshorne
Rune Lillesveen voted Code-Review+1![]( "Open in Gerrit")
Attention is currently required from:
- Morten Stenshorne
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Morten Stenshorne (Gerrit)
Feb 6, 2026, 5:12:22 AM (7 days ago) Feb 6
to Rune Lillesveen, Ian Kilpatrick, Menard, Alexis, Chromium Metrics Reviews, chromium...@chromium.org, apavlo...@chromium.org, asvitkine...@chromium.org, blink-re...@chromium.org, blink-re...@chromium.org, blink-revi...@chromium.org, blink-rev...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org, zol...@webkit.org
Morten Stenshorne voted Commit-Queue+2![]( "Open in Gerrit")
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Chromium LUCI CQ (Gerrit)
Feb 6, 2026, 6:16:20 AM (7 days ago) Feb 6
to Morten Stenshorne, Rune Lillesveen, Ian Kilpatrick, Menard, Alexis, Chromium Metrics Reviews, chromium...@chromium.org, apavlo...@chromium.org, asvitkine...@chromium.org, blink-re...@chromium.org, blink-re...@chromium.org, blink-revi...@chromium.org, blink-rev...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org, zol...@webkit.org
Chromium LUCI CQ submitted the change![]( "Open in Gerrit")
Change information
Commit message:
Support margin-trim on block containers.
This adds parsing support for the CSS margin-trim property, and layout
support for block containers. Still no support for flex, grid or
multicol containers.
See https://drafts.csswg.org/css-box-4/#margin-trim
Also note that in our implementation margin-trim doesn't affect the
resolved value of margin properties, at least for now. There are
existing tests that expect this, but not clear whether it's correct.
See https://github.com/w3c/csswg-drafts/issues/11506
There are still some failures in css-box/parsing/margin-trim.html , but
not sure if they are valid. Canonical order should match the order in
the spec, which is "block" before "inline", and "start" before "end".
Bug: 40886857
Change-Id: Ie00d463b5f42fb09eac2958bc7ed7cb2ae28df2b
Skip-Clang-Tidy-Checks: google-readability-casting
Reviewed-by: Ian Kilpatrick <ikilp...@chromium.org>
Reviewed-by: Rune Lillesveen <fut...@chromium.org>
Commit-Queue: Morten Stenshorne <mste...@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1580724}
Files:
- M third_party/blink/public/mojom/use_counter/metrics/css_property_id.mojom
- M third_party/blink/renderer/core/css/css_identifier_value_mappings.h
- M third_party/blink/renderer/core/css/css_properties.json5
- M third_party/blink/renderer/core/css/css_property_equality.cc
- M third_party/blink/renderer/core/css/properties/longhands/longhands_custom.cc
- M third_party/blink/renderer/core/layout/block_layout_algorithm.cc
- M third_party/blink/renderer/core/layout/block_layout_algorithm.h
- M third_party/blink/renderer/core/layout/constraint_space.h
- M third_party/blink/renderer/core/layout/constraint_space_builder.h
- M third_party/blink/renderer/core/layout/geometry/margin_strut.cc
- M third_party/blink/renderer/core/layout/geometry/margin_strut.h
- M third_party/blink/renderer/core/layout/layout_algorithm.h
- M third_party/blink/renderer/core/layout/layout_result.h
- M third_party/blink/renderer/core/style/computed_style_constants.h
- M third_party/blink/renderer/platform/runtime_enabled_features.json5
- M third_party/blink/web_tests/TestExpectations
- D third_party/blink/web_tests/external/wpt/css/css-box/inheritance-expected.txt
- D third_party/blink/web_tests/external/wpt/css/css-box/margin-trim/block-container-block-end-last-child-with-border-expected.txt
- D third_party/blink/web_tests/external/wpt/css/css-box/margin-trim/block-container-block-end-nested-last-child-with-border-expected.txt
- A third_party/blink/web_tests/external/wpt/css/css-box/margin-trim/block-container-block-end-self-collapsing-and-border.html
- A third_party/blink/web_tests/external/wpt/css/css-box/margin-trim/block-container-block-end-self-collapsing-block-start-margin-nested.html
- A third_party/blink/web_tests/external/wpt/css/css-box/margin-trim/block-container-block-end-self-collapsing-block-start-margin.html
- D third_party/blink/web_tests/external/wpt/css/css-box/margin-trim/block-container-block-end-self-collapsing-children-nested-at-bottom-expected.txt
- D third_party/blink/web_tests/external/wpt/css/css-box/margin-trim/block-container-block-end-self-collapsing-children-nested-margin-trim-expected.txt
- D third_party/blink/web_tests/external/wpt/css/css-box/margin-trim/block-container-block-end-self-collapsing-children-offsets-expected.txt
- D third_party/blink/web_tests/external/wpt/css/css-box/margin-trim/block-container-block-end-self-collapsing-children-offsets-nested-multiple-times-expected.txt
- D third_party/blink/web_tests/external/wpt/css/css-box/margin-trim/block-container-block-end-self-collapsing-children-offsets-nested-once-expected.txt
- D third_party/blink/web_tests/external/wpt/css/css-box/margin-trim/block-container-block-end-self-collapsing-children-offsets-vert-lr-expected.txt
- A third_party/blink/web_tests/external/wpt/css/css-box/margin-trim/block-container-block-end-start-margin.html
- A third_party/blink/web_tests/external/wpt/css/css-box/margin-trim/block-container-block-start-003.html
- A third_party/blink/web_tests/external/wpt/css/css-box/margin-trim/block-container-block-start-004.html
- D third_party/blink/web_tests/external/wpt/css/css-box/parsing/margin-trim-computed-expected.txt
- M third_party/blink/web_tests/external/wpt/css/css-box/parsing/margin-trim-expected.txt
- M third_party/blink/web_tests/external/wpt/css/css-box/parsing/margin-trim.html
- M third_party/blink/web_tests/fast/css/getComputedStyle/computed-style-listing-expected.txt
- M third_party/blink/web_tests/fast/css/getComputedStyle/computed-style-without-renderer-listing-expected.txt
- M third_party/blink/web_tests/svg/css/getComputedStyle-listing-expected.txt
- M third_party/blink/web_tests/webexposed/css-properties-as-js-properties-expected.txt
- M third_party/blink/web_tests/webexposed/css-property-listing-expected.txt
- M tools/metrics/histograms/metadata/blink/enums.xml
Change size: L
Delta: 40 files changed, 446 insertions(+), 165 deletions(-)
Branch: refs/heads/main
Submit Requirements:
Code-Review: +1 by Ian Kilpatrick, +1 by Rune Lillesveen
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Blink W3C Test Autoroller (Gerrit)
Feb 6, 2026, 7:20:57 AM (7 days ago) Feb 6
to Chromium LUCI CQ, Morten Stenshorne, Rune Lillesveen, Ian Kilpatrick, Menard, Alexis, Chromium Metrics Reviews, chromium...@chromium.org, apavlo...@chromium.org, asvitkine...@chromium.org, blink-re...@chromium.org, blink-re...@chromium.org, blink-revi...@chromium.org, blink-rev...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, jmedle...@chromium.org, kinuko...@chromium.org, zol...@webkit.org
Message from Blink W3C Test Autoroller![]( "Open in Gerrit")
The WPT PR for this CL has been merged upstream! https://github.com/web-platform-tests/wpt/pull/57608
Related details
Attention set is empty
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Reply all
Reply to author
Forward
0 new messages