Closed Bug 1732038 Opened 4 years ago Closed 4 years ago

[wpt-sync] Sync PR 30837 - [@layer] Fix cascade ordering of inline style and layered style

Categories

(Core :: CSS Parsing and Computation, task, P4)

task

Tracking

()

RESOLVED FIXED
95 Branch
Tracking Status
firefox95 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 30837 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/30837
Details from upstream follow.

Xiaocheng Hu <xiaochengh@chromium.org> wrote:

[@layer] Fix cascade ordering of inline style and layered style

This patch adds a new flag to MatchResult and CascadePriority to
indicate if a declaration is from inline style or other style sheets, so
that we can compare inline style and other layered style correctly.

Without this flag, currently we are treating inline style as regular
declarations in the implicit outer layer, and therefore mistakenly
override it with layered style rules in other style sheets.

Bug: 1095765
Change-Id: I10dbafcf9721fc5b8a5b11edeaeab48c82df86d4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3167095
Commit-Queue: Xiaocheng Hu \<xiaochengh@chromium.org>
Reviewed-by: Rune Lillesveen \<futhark@chromium.org>
Reviewed-by: Anders Hartvoll Ruud \<andruud@chromium.org>
Cr-Commit-Position: refs/heads/main@{#923216}

Component: web-platform-tests → CSS Parsing and Computation
Product: Testing → Core

CI Results

Ran 7 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 2 tests and 2 subtests

Status Summary

Firefox

OK : 2
PASS: 6

Chrome

OK : 2
PASS: 5
FAIL: 1

Safari

OK : 2
PASS: 5
FAIL: 1

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/448715b83303 [wpt PR 30837] - [@layer] Fix cascade ordering of inline style and layered style, a=testonly
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/6f7ebd034fb3 [wpt PR 30837] - [@layer] Fix cascade ordering of inline style and layered style, a=testonly
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 95 Branch
You need to log in before you can comment on or make changes to this bug.