iframes do not print since bug 485275 landed

RESOLVED FIXED

Status

()

defect
RESOLVED FIXED
10 years ago
9 months ago

People

(Reporter: tnikkel, Assigned: tnikkel)

Tracking

({fixed1.9.1})

1.9.1 Branch
Points:
---
Bug Flags:
blocking1.9.1 +
in-testsuite ?

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Assignee)

Description

10 years ago
Posted patch patchSplinter Review
Flags: blocking1.9.1?
Attachment #378279 - Flags: review?(roc)
Attachment #378279 - Flags: approval1.9.1?
(Assignee)

Updated

10 years ago
Assignee: nobody → tnikkel
Attachment #378279 - Flags: review?(roc)
Attachment #378279 - Flags: review+
Attachment #378279 - Flags: approval1.9.1?
Attachment #378279 - Flags: approval1.9.1+
This clearly blocks.

I wish we could write a test for this but I suspect we currently can't due to the way printing iframes works ("badly").
Flags: blocking1.9.1? → blocking1.9.1+
Keywords: checkin-needed
Whiteboard: [needs landing]
(Assignee)

Comment 2

10 years ago
Posted patch 1.9.1 patchSplinter Review
Attachment #378288 - Flags: review+
actually a reftest-print reftest with iframes probably would just work...
http://hg.mozilla.org/mozilla-central/rev/409416c625bc
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Flags: in-testsuite?
Keywords: checkin-needed
Resolution: --- → FIXED
Whiteboard: [needs landing] → [needs 191 landing]
Keywords: checkin-needed
(Assignee)

Comment 5

10 years ago
(In reply to comment #3)
> actually a reftest-print reftest with iframes probably would just work...

Apparently not. I tried using a page with an image as the ref, and the test was just an iframe of the ref. This worked fine when manually print-previewing them. But when running them as reftests the test would not draw the the image by the time the reftest harness took the snapshot (the iframe border at least was getting drawn when I allowed it to have a border). I tried doing some hacky things combining reftest-print and reftest-wait and removing the reftest-wait part when the iframe document had loaded, but I couldn't get it to work. Not surprising given the reftest readme:

"The underlying layout support for this mode isn't really complete; it doesn't use exactly the same codepath as real print preview/print. In particular, scripting and frames are likely to cause problems;"

Perhaps a test of print previewing a page with an iframe could be added to the basic functionality tests that are performed for releases.

Updated

9 months ago
Product: Core → Core Graveyard
Component: Layout: HTML Frames → Layout: Images
Product: Core Graveyard → Core
You need to log in before you can comment on or make changes to this bug.