361844 - Printing cleanup

Status: RESOLVED FIXED

(Core :: Printing: Output, defect)

Description

[Eli Friedman]

Patch coming up shortly.

Changes:

1. Less interdependence between nsPrintEngine and nsDocumentViewer.  nsPrintEngine now manages its own presentation/destroys its own presentation.  This has the side-effect of making exiting print preview faster than it used to be in the common case of a page with no frames or plugins.  Also, the EndObservingDocument call is now always near the Destroy call for presshells.

2. Allow subdocuments without display presshells to get printed.  There actually wasn't any fundamental reason they weren't working before; it's just that the nsPrintObject code to get the document failed when there wasn't a presshell.

3. Minor bugfix to allow the print preview navigation buttons to work while printing.  Caused by an odd check in PrintPreviewNavigate().

4. Extremely minor bug with printing selection when the HTML body tag isn't at the origin of the document (not normally visible because the body is almost always at the origin). The bug was actually caused by extra code which incorrectly attempted to compensate for the location of the body relative to its parent.

5. Getting rid of extra/duplicate member variables.

6. Getting rid of useless method parameters.

7. Other misc cleanup.

Comment 1

[Eli Friedman]

Attachment: Patch

Comment 2

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Comment on attachment 246577 [details] [diff] [review]
Patch

Sure would be nice if GetDisplayTitleAndURL and GetDocumentTitleAndURL etc used string classes instead of malloc'd character pointers!

This is a bit of a rubber-stamp because really you're the only person who knows this code. At some stage --- maybe when you're "done" with fixing this up --- it would be really really great if you could write some documentation for the print engine.

Comment 3

[Eli Friedman]

I think I'm going to try and figure out why exactly this fixes bug 361941 before I check this in; I'm worried there might be some deeper event problems.

Comment 4

[Eli Friedman]

Attachment: Updated to trunk

I'm going to check this in now.

I decided that for the moment the event stuff is not a big deal; the only thing it breaks is using the arrow keys to scroll the document in print preview, as far as I can tell, so I'm not going to worry about it for the moment.  I'll make sure to keep the printing stuff moving so that it gets fixed in the near future.

Comment 5

[Olli Pettay [:smaug][[e-mail]]]

(In reply to comment #3)
> I think I'm going to try and figure out why exactly this fixes bug 361941
> before I check this in; I'm worried there might be some deeper event problems.
> 

See https://bugzilla.mozilla.org/show_bug.cgi?id=361941#c3