Open Bug 1649367 Opened 11 months ago Updated 4 months ago

Explicitly use InProcess version of IsRootContentDocument for frame visibility

Categories

(Core :: Layout, enhancement)

enhancement

Tracking

()

ASSIGNED

People

(Reporter: hiro, Assigned: emilio)

Details

Attachments

(1 file)

There are two nsPresContext::GetToplevelContentDocumentPresContext calls in PresShell::AssumeAllFramesVisible and in PresShell::ScheduleApproximateFrameVisibilityUpdateNow, it won't work properly in OOP iframes.

I think this has come up before: bug 1554832. I think it might be working as intended right now, but I'll have to look in detail.

Fission Milestone: --- → M7

Emilio, do you know if this is fixed now?

Flags: needinfo?(emilio)

This doesn't change behavior, and is consistent with the changes made in
bug 1554832 to this code.

Assignee: nobody → emilio
Status: NEW → ASSIGNED

Yes, I think this is fixed, but let's be explicit on which document we care about.

Flags: needinfo?(emilio)

Changing the name of the bug, as the original issue has now been fixed.

No longer blocks: rendering-fission
Fission Milestone: M7 → ---
Summary: Make the optimization for invisible animated images (GIF) in fission → Explicitly use InProcess version of IsRootContentDocument for frame visibility
You need to log in before you can comment on or make changes to this bug.