fix print preview for 130078

RESOLVED FIXED

Status

()

Core
Print Preview
RESOLVED FIXED
8 years ago
7 years ago

People

(Reporter: tnikkel, Unassigned)

Tracking

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Comment hidden (empty)
(Reporter)

Comment 1

8 years ago
Created attachment 462521 [details] [diff] [review]
patch

This is roc's patch that I have scavenged.
Attachment #462521 - Flags: review?(Olli.Pettay)

Comment 2

8 years ago
Comment on attachment 462521 [details] [diff] [review]
patch

I'm atm on OSX so couldn't test printpreview. But the changes look
reasonable.
Attachment #462521 - Flags: review?(Olli.Pettay) → review+
(Reporter)

Comment 3

8 years ago
I've done basic tests of print and print preview on Linux and Windows with this patch + 130078.
(Reporter)

Updated

8 years ago
Depends on: 585496
(Reporter)

Comment 5

8 years ago
Is it worth spending any time trying to figure out why this doesn't work without 130078, or should we just land this at the same time as 130078?
Blocks: 579621

Updated

8 years ago
Depends on: 585697

Updated

8 years ago
No longer depends on: 585697
I'm using an hourly build with 130078 relanded, and I can print preview pages just fine.

Is this still a bug for other people?
(Reporter)

Comment 8

8 years ago
This bug landed at the same time as bug 130078. I haven't got around to marking all the bugs that landed with 130078 fixed yet.
(Reporter)

Comment 9

8 years ago
http://hg.mozilla.org/mozilla-central/rev/0886ad6e6aaa
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED

Updated

7 years ago
Depends on: 652914
You need to log in before you can comment on or make changes to this bug.