Closed Bug 1962764 Opened 16 days ago Closed 12 days ago

[wpt-sync] Sync PR 52179 - [reading-flow] Can render with carousel

Categories

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

Product:
Core
Component:
CSS Parsing and Computation
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
140 Branch
Tracking Flags:
Tracking Status
firefox140 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

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

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

Di Zhang <dizhangg@chromium.org> wrote:

[reading-flow] Can render with carousel

Setting CSS property reading-flow on a carousel scroller is failing
DCHECKs for both the focus navigation and the accessibility tree. To
fix this use case,

  1. Relax the OwnedByReadingFlow to be element instead of HTMLElement.
    This is because pseudo-element can be reading flow item scope owners.
  2. Change Element::ReadingFlowChildren to add out of flow reading flow
    items in LayoutTreeBuilderTraversal instead of FlatTreeTraversal.
    This will add scroll marker and related pseudo elements.

Change-Id: Icbb523c4f93aff316a8f8bfcbf867e1ec781a1d7
Bug: 398280418, 412401428
Reviewed-on: https://chromium-review.googlesource.com/6479432
WPT-Export-Revision: 631ad07c70af562a32e62f26584ec944a76080e3

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

CI Results

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

Total 1 tests and 1 subtests

Status Summary

Firefox

OK : 1
FAIL: 1

Chrome

OK : 1
PASS: 1

Safari

OK : 1
FAIL: 1

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/1d6f342c957c [wpt PR 52179] - [reading-flow] Can render with carousel, a=testonly https://hg.mozilla.org/integration/autoland/rev/7db9ae69539d [wpt PR 52179] - Update wpt metadata, a=testonly

https://hg.mozilla.org/mozilla-central/rev/1d6f342c957c
https://hg.mozilla.org/mozilla-central/rev/7db9ae69539d

Status: NEW → RESOLVED
Closed: 12 days ago
status-firefox140: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 140 Branch
You need to log in before you can comment on or make changes to this bug.