Closed Bug 1592963 Opened 5 years ago Closed 4 years ago

Zooming a page with an out of process iframe renders incorrectly

Categories

(Core :: Layout, defect, P3)

defect

Tracking

()

RESOLVED FIXED
Fission Milestone M5

People

(Reporter: enndeakin, Unassigned)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

Steps:

  1. Enable fission (fission.autostart preference)
  2. Make sure View -> Zoom -> Zoom Text Only is unchecked so that full zooming is done.
  3. Open data:text/html,<html><body>Outer Text<iframe src='http://www.example.com'></iframe></body></html>
  4. Zoom the page.

Expected: the main page and the iframe to zoom properly
Actual: the iframe resizes but the page leaves rendering artifacts of the scrollbar behind.

Probably nsDocumentViewer::PropagateToPresContextsHelper needs to be fixed.

Priority: -- → P3
Fission Milestone: --- → M5

Trying to reproduce, I get this, which I think is correct.

Neil, could you confirm that this has apparently been fixed already?

Flags: needinfo?(enndeakin)

It looks like the issue doesn't occur anymore. It might have been fixed by the work done for bug 1590485 and friends. Don't know whether the fix was intentional or not though.

I'll mark as fixed for now.

Status: NEW → RESOLVED
Closed: 4 years ago
Flags: needinfo?(enndeakin)
Resolution: --- → FIXED

It seems it's intentional, there is a browser test for fission. Awesome! I am adding bug 1590485 in dependency list.

Depends on: 1590485

(In reply to Hiroyuki Ikezoe (:hiro) from comment #4)

It seems it's intentional, there is a browser test for fission. Awesome! I am adding bug 1590485 in dependency list.

(In reply to Neil Deakin from comment #3)

It looks like the issue doesn't occur anymore. It might have been fixed by the work done for bug 1590485 and friends. Don't know whether the fix was intentional or not though.

I'll mark as fixed for now.

So this is still sorta an issue, but only for delay-loaded iframes: https://bugzilla.mozilla.org/show_bug.cgi?id=1612068 (that fission test should be preffed off in tree right now as a result)

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: