Closed Bug 1716537 Opened 3 years ago Closed 3 years ago

Same process frames nested in remote frames are invisible when printing to PDF

Categories

(Core :: Printing: Output, defect)

Product:
Core
Component:
Printing: Output
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
91 Branch
Project Flags:
Fission Milestone M8
Tracking Flags:
Tracking Status
firefox-esr78 --- unaffected
firefox89 --- unaffected
firefox90 --- unaffected
firefox91 --- fixed

People

(Reporter: jdescottes, Assigned: emilio)

References

(Regression)

Details

(Keywords: regression)

Attachments

(2 files)

Bug 1716537 - Propagate print setting updates to remote sub-subdocuments as well. r=mattwoodrow
(deleted), text/x-phabricator-request
Details
Bug 1716537 - Allow updating viewport dimensions on paginated subdocuments. r=mattwoodrow
(deleted), text/x-phabricator-request
Details

STRs:

AR: Print is missing all the content from "same process frames nested in a remote frame"
ER: Print should be the same as without fission

The test page is:

<!doctype html>
<style>
  body { margin: 0 }
  iframe { width: 100%; height: 100%; }
</style>
<iframe frameborder=0 scrolling=no src="https://bug1695806-crossframe.glitch.me/"></iframe>

The remote frame is:

<!doctype html>
<iframe scrolling=no frameborder=0 srcdoc='from nested srcdoc iframe'></iframe>
<iframe scrolling=no frameborder=0 src='data:text/html,from nested data uri iframe'></iframe>
<iframe scrolling=no frameborder=0 src='/sameorigin.html'></iframe>
<iframe scrolling=no frameborder=0 src='https://bug1695806-otherorigin.glitch.me/'></iframe>
From the remote frame itself

(both sameorigin.html and bug1695806-otherorigin.glitch.me contain just some text)

When printing with Fission enabled, all the content from the first 3 iframes is missing. So it seems that "same-process" iframes nested in a remote frame are not visible.

Blocks: 1695806

Emilio: looks like there is slight regression after Bug 1699837, any idea what could have caused this?

Flags: needinfo?(emilio)

Set release status flags based on info from the regressing bug 1699837

status-firefox89: --- → unaffected
status-firefox90: --- → unaffected
status-firefox91: --- → affected
status-firefox-esr78: --- → unaffected

Sounds like a corner case that can wait for M8. Also, emilio is on pto right now, so redirecting to Frank to find an alternate assignee for this.

Fission Milestone: --- → M8
Flags: needinfo?(emilio) → needinfo?(fgriffith)

I don't wanna lose track of this still :)

Flags: needinfo?(emilio)

Heard from Emilo and he stated that he will still own this. Will assign.

Flags: needinfo?(fgriffith)
Assignee: nobody → emilio
Severity: -- → S3

As those don't have the same incremental reflow issues as root paginated
documents, and we do need this for remote iframes to update their
viewport.

Depends on D119103

Flags: needinfo?(emilio)
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/c6561b266910 Propagate print setting updates to remote sub-subdocuments as well. r=mattwoodrow https://hg.mozilla.org/integration/autoland/rev/0b5bfe85b344 Allow updating viewport dimensions on paginated subdocuments. r=mattwoodrow
Flags: needinfo?(emilio)
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/7086ac4e5818 Propagate print setting updates to remote sub-subdocuments as well. r=mattwoodrow https://hg.mozilla.org/integration/autoland/rev/ddc93b2746df Allow updating viewport dimensions on paginated subdocuments. r=mattwoodrow
Flags: needinfo?(emilio)
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/3226bc23c713 Propagate print setting updates to remote sub-subdocuments as well. r=mattwoodrow https://hg.mozilla.org/integration/autoland/rev/aa3e3f80bd68 Allow updating viewport dimensions on paginated subdocuments. r=mattwoodrow
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/690e37acebe8 Propagate print setting updates to remote sub-subdocuments as well. r=mattwoodrow https://hg.mozilla.org/integration/autoland/rev/db705d5bbd3e Allow updating viewport dimensions on paginated subdocuments. r=mattwoodrow

Third time's the charm.

Flags: needinfo?(emilio)

https://hg.mozilla.org/mozilla-central/rev/690e37acebe8
https://hg.mozilla.org/mozilla-central/rev/db705d5bbd3e

Status: NEW → RESOLVED
Closed: 3 years ago
status-firefox91: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → 91 Branch
Flags: qe-verify+
Has Regression Range: --- → yes
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: