Closed Bug 1679320 Opened 4 years ago Closed 4 years ago

hit testing is wrong (or delayed) with an element on top of a cross origin iframe

Categories

(Core :: Layout, defect, P2)

defect

Tracking

()

RESOLVED DUPLICATE of bug 1676466
Fission Milestone M6c

People

(Reporter: heycam, Unassigned)

References

(Blocks 2 open bugs)

Details

Attachments

(2 files)

STR:

  1. Load https://www.macrumors.com/.
  2. Scroll down to an entry that embeds a tweet where the embedding iframe is behind the "Show Full Article" button.
  3. Mouse over the "Show Full Article" button.
  4. Click the button.

Expected results:

  1. When mousing over the button, it is immediately highlighted and the cursor changes to a hand.
  2. When clicking the button, the full macrumors.com article is opened.

Actual results:

  1. When mousing over the button, it is highlighted only after a ~0.5s delay and the cursor does not change.
  2. When clicking the button, the tweet is opened.

The title of the entry in question is "Reliable Leaker Suggests Redesigned MacBooks in 2021 Will Include Both Apple Silicon and Intel Models". Oddly my daily use nightly does just show the tweet text and is not able to reproduce the issue. But on my local build, it shows embedded tweet image, then is able to reproduce the issue. I am attaching a screenshot of the entry. (I took it without fission, since with enabling fission, the tweet image were not rendered in the screenshot)

Looks like the text tweet case on my daily use nightly, the tweet content is not iframe-ed at all. I don't know where the difference comes from.

Attached file A possible test case (deleted) —

Probably this is a minimum test case, there is an OOP iframe inside a containing block and there is a a position:absolute element inside the containing block and the element has an <a>.

CCing Timothy. (I have totally no idea how we treat this kind of overlapping elements in APZ)

Timothy, this is really bad, can you fix this please?

Fission Milestone: --- → M6c
Flags: needinfo?(tnikkel)
Severity: -- → S2
Priority: -- → P2
Blocks: 1519412

Redirecting need info to Botond as I believe he is currently working on fission hit testing issues.

Flags: needinfo?(tnikkel) → needinfo?(botond)

I believe Miko is already looking into similar hit testing issues for Fission (thanks Miko).

Flags: needinfo?(botond) → needinfo?(mikokm)

Botond said he'll look into this.

Flags: needinfo?(botond)

I am ~100% sure this is a dup of bug 1676466. (I hadn't noticed the bug before)

Thanks Hiro! I'll mark bug 1676466 as a dependency, and we'll re-test once that is fixed.

Depends on: 1676466
Flags: needinfo?(mikokm)
Flags: needinfo?(botond)

Miko, can you test this with your patch for bug 1676466 to see if it fixes this?

Flags: needinfo?(mikokm)

(In reply to Neha Kochar [:neha] from comment #11)

Miko, can you test this with your patch for bug 1676466 to see if it fixes this?

It does. :)

Status: NEW → RESOLVED
Closed: 4 years ago
Flags: needinfo?(mikokm)
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: