Closed Bug 1867567 Opened 2 years ago Closed 2 years ago

[wpt-sync] Sync PR 43457 - Mark the outmost dirty style containment scope only if it has changed

Categories

(Core :: Layout: Generated Content, Lists, and Counters, task, P4)

task

Tracking

()

RESOLVED FIXED
122 Branch
Tracking Status
firefox122 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Daniil Sakhapov <sakhapov@chromium.org> wrote:

Mark the outmost dirty style containment scope only if it has changed

During the creation of the style containment scope the ancestors are
created on the fly and are marked as outmost dirty in such cases.
But that's not correct as if sometimes if nothing is changed in the new
ancestor, we don't need to update the outmost dirty scope, as it can
lead to some quotes being updated outside the container update.

Bug: 1505164
Change-Id: Ic267a863201223d1b3c669ae5216c1eef7dddff7
Reviewed-on: https://chromium-review.googlesource.com/5075697
WPT-Export-Revision: fb9a88e1347b6ba86ec282fcdf220a7cfade7f93

Component: web-platform-tests → Layout: Generated Content, Lists, and Counters
Product: Testing → Core

CI Results

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

Total 1 tests

Status Summary

Firefox

PASS: 1

Chrome

PASS: 1

Safari

PASS: 1

Links

GitHub PR Head
GitHub PR Base

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/d6f4f8253e2f [wpt PR 43457] - Mark the outmost dirty style containment scope only if it has changed, a=testonly
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/b58480f03761 [wpt PR 43457] - Mark the outmost dirty style containment scope only if it has changed, a=testonly
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 122 Branch
You need to log in before you can comment on or make changes to this bug.