Closed
Bug 393747
Opened 17 years ago
Closed 17 years ago
Support nsISupports in print preview observer
Categories
(Toolkit :: Printing, defect)
Toolkit
Printing
Tracking
()
RESOLVED
FIXED
mozilla1.9alpha8
People
(Reporter: philor, Assigned: philor)
References
()
Details
Attachments
(1 file)
|
(deleted),
patch
|
Gavin
:
review+
|
Details | Diff | Splinter Review |
Given the lockstep order of the interface id values, and even the variable names and indentation, you'd get the impression that there was One True Observer In JavaScript QI function that everyone copies, but apparently print preview missed the memo, using a single long line to hide the fact that it doesn't support nsISupports like all the other kids do.
Attachment #278275 -
Flags: review?(gavin.sharp)
|
||
Updated•17 years ago
|
Attachment #278275 -
Flags: review?(gavin.sharp) → review+
|
Assignee | |
Comment 1•17 years ago
|
||
toolkit/components/printing/content/printUtils.js 1.29
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9 M8
You need to log in
before you can comment on or make changes to this bug.
Description
•