Closed
Bug 290214
Opened 19 years ago
Closed 19 years ago
SetPMPrintSettings releases page format instead of print settings
Categories
(Core Graveyard :: GFX: Mac, defect, P2)
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla1.8beta3
People
(Reporter: tony, Assigned: jaas)
Details
(Whiteboard: patch)
Attachments
(1 file)
645 bytes,
patch
|
mikepinkerton
:
review+
sfraser_bugs
:
superreview+
brendan
:
approval1.8b3+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Macintosh; U; PPC Mac OS X; en-ca) AppleWebKit/125.5.7 (KHTML, like Gecko) Safari/125.12 Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.7.6) Gecko/20050317 Firefox/1.0.2 I tried implementing nsPrinterEnumerator for Mac OS X and discovered that nsPrintSettingsX:: SetPMPrintSettings releases mPageFormat instead of mPrintSettings. When PMSessionPageSetupDialog is called, it returns kPMInvalidPageFormat. Reproducible: Always Steps to Reproduce: 1. Obtain an nsPrintSettingsX object. 2. Call SetPMPrintSettings to set the print settings. 3. Call up the Page Setup dialog. Actual Results: Nothing happens. Expected Results: The Page Setup dialog should appear.
Reporter | ||
Comment 1•19 years ago
|
||
Reporter | ||
Updated•19 years ago
|
Attachment #180611 -
Flags: review?(pinkerton)
Comment 2•19 years ago
|
||
Comment on attachment 180611 [details] [diff] [review] Patch to fix the error described. good catch, r=pink
Attachment #180611 -
Flags: review?(pinkerton) → review+
Comment 3•19 years ago
|
||
We should get this in.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: blocking1.8b3?
Priority: -- → P2
Whiteboard: patch
Target Milestone: --- → mozilla1.8beta3
Updated•19 years ago
|
Attachment #180611 -
Flags: superreview+
Updated•19 years ago
|
Attachment #180611 -
Flags: approval1.8b3?
Comment 4•19 years ago
|
||
Comment on attachment 180611 [details] [diff] [review] Patch to fix the error described. a=me for 1.8b3. /be
Attachment #180611 -
Flags: approval1.8b3? → approval1.8b3+
landed on trunk
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Updated•19 years ago
|
Flags: blocking1.8b3?
Updated•16 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•