[webkit-reviews] review requested: [Bug 26584] [Qt] Printing extra information : [Attachment 31643] A better patch

[bugzilla-daemon at webkit.org]

Jakob Truelsen <antialize at gmail.com> has asked	for review:
Bug 26584: [Qt] Printing extra information
https://bugs.webkit.org/show_bug.cgi?id=26584

Attachment 31643: A better patch
https://bugs.webkit.org/attachment.cgi?id=31643&action=review

------- Additional Comments from Jakob Truelsen <antialize at gmail.com>
As for using an printcontext in the countpages method: It is my impression that
you need a print rendertree, to predict the number of pages.  It would of cause
make good sense to use the same render three both for counting and for
printing. But this would as I seed it require changes to existing API.	In my
experience building the printcontext does not take long. 

If anyone know of a better way to achieve then please let my know. I'm willing
to implement it.