Tests:
authorsullivan <sullivan@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 29 Jan 2004 02:16:33 +0000 (02:16 +0000)
committersullivan <sullivan@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 29 Jan 2004 02:16:33 +0000 (02:16 +0000)
commit1f0466fcdfd9828459ca7281df4c79a4b24b0126
tree5a464a9cb0c7db43a2de7d27af3e368c86184e96
parent1b7e29b571d0380595c05c7a8811459ee704edaa
Tests:

        Reviewed by Dave.

        * Blot/blot-tasks.txt:
        Mentioned NSTextView's (and therefore Mail's) option-Escape behavior here
        since we will probably have to implement it also; clarified a comment.

WebCore:

        Reviewed by Dave.

        * khtml/rendering/render_text.cpp:
        (RenderText::paintObject):
        Increment s in the while() clause so it is incremented even
        after a continue statement. This fixes an infinite loop I ran
        into in the printing code path at certain scaled print sizes
        on certain pages.

WebKit:

        More header/footer work: refactored the header/footer code so it could
        be easily reused by other WebDocument classes; used it from WebImageView
        and WebTextView; removed the page count parameters because it's possible
        (though currently nasty, see 3543078) to determine this in the client.

        Reviewed by Dave.

        * Misc.subproj/WebNSPrintOperationExtras.h Added.
        * Misc.subproj/WebNSPrintOperationExtras.m Added.
        (-[NSPrintOperation _web_pageSetupScaleFactor]):
        new convenience method.

        * WebView.subproj/WebUIDelegatePrivate.h:
        Removed page index and page count parameters from delegate methods.

        * WebView.subproj/WebViewPrivate.h:
        New private category for header/footer printing methods so that different
        WebDocument methods can share almost all of the code.

        * WebView.subproj/WebView.m:
        (-[WebView _headerHeight]):
        (-[WebView _footerHeight]):
        (-[WebView _drawHeaderInRect:]):
        (-[WebView _drawFooterInRect:]):
        (-[WebView _adjustPrintingMarginsForHeaderAndFooter]):
        (-[WebView _drawHeaderAndFooter]):
        Moved all of these methods here, formerly in WebHTMLView. Removed the
        page index and page count parameters.

        * WebView.subproj/WebHTMLView.m:
        Removed all the header/footer code that's now in WebView.m, and the
        method that's now -[NSPrintOperation _web_pageSetupScaleFactor]
        (-[WebHTMLView _setPrinting:minimumPageWidth:maximumPageWidth:adjustViewSize:]):
        call methods differently that have now been moved
        (-[WebHTMLView _scaleFactorForPrintOperation:]):
        ditto
        (-[WebHTMLView knowsPageRange:]):
        ditto
        (-[WebHTMLView drawPageBorderWithSize:]):
        now just turns around and calls -[WebView _drawHeaderAndFooter]

        * WebView.subproj/WebImageView.m:
        (-[WebImageView drawPageBorderWithSize:]):
        new method, just calls -[WebView _drawHeaderAndFooter]
        (-[WebImageView beginDocument]):
        now calls -[WebView _adjustPrintMarginsForHeaderAndFooter], also moved in file.
        (-[WebImageView endDocument]):
        just moved in file.

        * WebView.subproj/WebTextView.m:
        (-[WebTextView drawPageBorderWithSize:]):
        new method, just calls -[WebView _drawHeaderAndFooter]
        (-[WebTextView knowsPageRange:]):
        overridden to call -[WebView _adjustPrintMarginsForHeaderAndFooter]

        * WebKit.pbproj/project.pbxproj:
        updated for added files

WebBrowser:

        Some header/footer cleanup.

        Reviewed by Dave.

        * BrowserWebViewPrinting.m:
        (-[BrowserWebView webView:drawHeaderInRect:]):
        Removed obsolete page index and count parameters.
        Use displayName rather than currentTitle because it works for all
        document types (currentTitle didn't work for plain text pages).
        (-[BrowserWebView webView:drawFooterInRect:]):
        Removed obsolete page index and count parameters.
        Did some AppKit hackery to get the total document page count in a way that
        works for any print operation. Now handles the case where the page count
        is unknown (by printing "Page <x>" instead of "Page <x> of <n>"). Also added
        a 1 pixel offset to the footer's baseline to work around AppKit bug where
        the descenders in the footer were getting slightly clipped at some scale
        factors.

        * English.lproj/Localizable.strings:
        updated for these changes

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@6004 268f45cc-cd09-0410-ab3c-d52691b4dbfc
12 files changed:
WebCore/ChangeLog-2005-08-23
WebCore/khtml/rendering/render_text.cpp
WebKit/ChangeLog
WebKit/Misc.subproj/WebNSPrintOperationExtras.h [new file with mode: 0644]
WebKit/Misc.subproj/WebNSPrintOperationExtras.m [new file with mode: 0644]
WebKit/WebKit.pbproj/project.pbxproj
WebKit/WebView.subproj/WebHTMLView.m
WebKit/WebView.subproj/WebImageView.m
WebKit/WebView.subproj/WebTextView.m
WebKit/WebView.subproj/WebUIDelegatePrivate.h
WebKit/WebView.subproj/WebView.m
WebKit/WebView.subproj/WebViewPrivate.h