Closed Bug 1570149 Opened 6 years ago Closed 6 years ago

[wpt-sync] Sync PR 18182 - [LayoutNG] Fix hit-testing list markers

Categories

(Core :: DOM: CSS Object Model, task, P4)

Product:
Core
Component:
DOM: CSS Object Model
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla70
Tracking Flags:
Tracking Status
firefox70 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

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

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

Koji Ishii <kojii@chromium.org> wrote:

[LayoutNG] Fix hit-testing list markers

This patch fixes hit-testing list markers to include when the
marker content was hit.

In legacy, markers are LayoutBox with special painters, but
LayoutNG creates an anonymous box with an anonymous text or
an anonymous image, and that hit-testing may find the content
inside the marker. This patch checks parent of the object.

Bug: 988414
Change-Id: Ifff61452a791e88f45d44bc45f00ead31070df0c

Reviewed-on: https://chromium-review.googlesource.com/1726571
WPT-Export-Revision: e202c2f25185fa1bacd14b1b2e8d1881c865f15d

Component: web-platform-tests → DOM: CSS Object Model
Product: Testing → Core
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/a2afb2a844bd [wpt PR 18182] - [LayoutNG] Fix hit-testing list markers, a=testonly
Ran 1 tests and 6 subtests OK : 1 PASS : 6

https://hg.mozilla.org/mozilla-central/rev/a2afb2a844bd

Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox70: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla70
You need to log in before you can comment on or make changes to this bug.