Closed Bug 703197 Opened 14 years ago Closed 14 years ago

Printing a display:none iframe should work

Categories

(Core :: Printing: Setup, defect)

9 Branch
x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla11

People

(Reporter: bzbarsky, Assigned: bzbarsky)

References

()

Details

Attachments

(1 file)

Loading the attached url should pop up a print dialog and let you print.
Attachment #575117 - Flags: review? → review?(bugs)
Attachment #575117 - Flags: review?(bugs) → review+
Flags: in-testsuite?
Whiteboard: [need review]
Target Milestone: --- → mozilla11
Comment on attachment 575117 [details] [diff] [review] Make window.print() work in display:none iframes. >- nsCOMPtr<nsIPresShell> presShell; >- docShell->GetPresShell(getter_AddRefs(presShell)); >- if (!presShell || !mDocument || !mDeviceContext) { >+ if (!mDocument || !mDeviceContext) { > PR_PL(("Can't Print without pres shell, document etc")); Is that log message still correct, after you've removed the check for presShell?
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
> Is that log message still correct, after you've removed the check for presShell? Nope. Good catch. I'll fix that.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: