Closed
Bug 326363
Opened 19 years ago
Closed 19 years ago
Attempting to print in trunk builds shows error "An unknown error occurred while printing"
Categories
(Core :: Printing: Output, defect)
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: nguyenhm16, Assigned: mark)
References
Details
(Keywords: smoketest)
Attachments
(1 file)
1.06 KB,
patch
|
kherron+mozilla
:
review+
roc
:
superreview+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.9a1) Gecko/20060203 Camino/1.0+
Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.9a1) Gecko/20060207 Camino/1.0+
When printing, I get the following sheet:
Printer Error
An unknown error occurred while printing
Works on 1.0rc1, 20060203 nightly
Reproducible: Always
Steps to Reproduce:
1. Go to any page
2. File->Print... or CMD-P
Actual Results:
You get the above-mentioned error sheet
Expected Results:
Print dialog box should come up
Comment 1•19 years ago
|
||
Just to verify...
This works fine on RC1, but *not* on the build used to file this bug?
cl
Comment 2•19 years ago
|
||
What printer do you have? Does it happen if you print to PDF? Is there any console output? Have you generated a debug log from CUPS? What color socks did you put on this morning?
Reporter | ||
Comment 3•19 years ago
|
||
Chris: Yes, this works fine on RC1, as well as the 20060203 nightly (am running it right now, hitting CMD-P brings up the print dialog as expected).
Simon: I have a Samsung ML1750. I don't even get a chance to try to print to PDF, the print dialog never shows up, just the error sheet, nothing on the console. I don't think it even gets as far as CUPS. Blue socks, as usual.
On a lark, I just tried it on the latest FireFox nightly and I get the same error:
Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.9a1) Gecko/20060207 Firefox/1.6a1
Tried the same thing on FireFox 1.5.0.1 and it works.
Looks like it's not Camino specific.
Updated•19 years ago
|
Product: Camino → Core
Summary: Can't print → Attempting to print in trunk builds shows error "An unknown error occurred while printing"
Version: unspecified → Trunk
Assignee | ||
Comment 4•19 years ago
|
||
Confirming in 0207 Firefox trunk build. Can you try a few trunk builds and figure out when this went sour?
Status: UNCONFIRMED → NEW
Ever confirmed: true
Assignee | ||
Updated•19 years ago
|
Assignee: mikepinkerton → printing
Reporter | ||
Comment 5•19 years ago
|
||
On FireFox, it worked with this build:
Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.9a1) Gecko/20060204 Firefox/1.6a1
But stopped working with this build:
Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.9a1) Gecko/20060205 Firefox/1.6a1
Assignee | ||
Comment 7•19 years ago
|
||
We don't save per-printer settings on the Mac.
Assignee: printing → mark
Status: NEW → ASSIGNED
Attachment #211145 -
Flags: superreview?(roc)
Attachment #211145 -
Flags: review?
Assignee | ||
Updated•19 years ago
|
Attachment #211145 -
Flags: review? → review?(kherron+mozilla)
Attachment #211145 -
Flags: superreview?(roc) → superreview+
Comment 8•19 years ago
|
||
Comment on attachment 211145 [details] [diff] [review]
Don't call GetDefaultPrinterName on Mac
Just to be clear, The old code (before bug 324072) would have initialized retval with the name of the last printer used. The old code will not do that because the code to load that information was removed from |InitPrintSettingsFromPrefs|. Assuming that's okay, this patch looks fine.
Attachment #211145 -
Flags: review?(kherron+mozilla) → review+
Assignee | ||
Comment 9•19 years ago
|
||
I see. Functionally, this seems fine, but I'll take another look to clean up the assertions.
Comment 10•19 years ago
|
||
*** Bug 327028 has been marked as a duplicate of this bug. ***
Comment 11•19 years ago
|
||
I'm probably going to back out the patch for bug 324072 and write a slightly different patch that can be accepted on the 1.8 branch. I'd like to avoid breaking printing on MacOSX with the new patch, but I don't understand exactly what's wrong with the current code. Mark, could you explain more specifically how the current code (including the patch for bug 324072) is failing?
Assignee | ||
Comment 12•19 years ago
|
||
We don't deal with named printers on the Mac, we only send things to "the printer." No per-printer prefs are ever saved. We have no printer enumerator, and GetDefaultPrinterName always fails.
Comment 13•19 years ago
|
||
*** Bug 327492 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 14•19 years ago
|
||
Kenneth, what's the story here? I don't want to leave this broken for too much longer, so if your revised solution to bug 324072 is going to take a while, I'll revise the patch on this bug to at least provisionally get trunk printing working again on the Mac.
Comment 15•19 years ago
|
||
The patch from bug 324072 is backed out. Could someone retest this and resolve the bug if it's fixed?
Comment 16•19 years ago
|
||
(In reply to comment #15)
> The patch from bug 324072 is backed out. Could someone retest this and resolve
> the bug if it's fixed?
>
In tinderbox-build, This problem is solved.
Mac OS X 10.3.9
Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.9a1) Gecko/20060304 Firefox/1.6a1
Assignee | ||
Comment 17•19 years ago
|
||
This is better now on the trunk after the backout of bug 324072. Thanks, Kenneth.
Clearing bogus dependency.
Comment 18•19 years ago
|
||
In Camino, it still reproduces...
Mac OS X 10.3.9
Camino which built from source of trunk.
Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; ja-JP; rv:1.9a1) Gecko/20060305 Camino/1.2+
Comment 19•19 years ago
|
||
(In reply to comment #18)
> In Camino, it still reproduces...
The one camino-specific change was backed out with the rest of it. I don't have a Mac build environment so unfortunately I can't troubleshoot this.
Assignee | ||
Comment 20•19 years ago
|
||
This bug is fixed. Camino (and other Cocoa widget-based products) still won't print due to bug 329455.
You need to log in
before you can comment on or make changes to this bug.
Description
•