<rdar://problem/5478690> Regression: printing: footer only appears on the...
authoradachan <adachan@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 13 Sep 2007 01:53:01 +0000 (01:53 +0000)
committeradachan <adachan@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 13 Sep 2007 01:53:01 +0000 (01:53 +0000)
        Reviewed by Adam.

        * WebFrame.cpp:
        (WebFrame::spoolPages): the footer rect is relative to the top left of the current page.  So
        instead of passing pageRect.bottom() as the top of the footer rect (which is relative to the
        document), we should pass in headerHeight plus the height of the pageRect.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@25525 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebKit/win/ChangeLog
WebKit/win/WebFrame.cpp

index dc1aa31ba5745204df12b3538e26368ce5d53b71..c992716c5cfbfa8c6d23a208aea9876f2aed12b6 100644 (file)
@@ -1,3 +1,14 @@
+2007-09-12  Ada Chan  <adachan@apple.com>
+
+        <rdar://problem/5478690> Regression: printing: footer only appears on the first page of print
+
+        Reviewed by Adam.
+
+        * WebFrame.cpp:
+        (WebFrame::spoolPages): the footer rect is relative to the top left of the current page.  So 
+        instead of passing pageRect.bottom() as the top of the footer rect (which is relative to the 
+        document), we should pass in headerHeight plus the height of the pageRect.
+
 2007-09-12  Anders Carlsson   <andersca@apple.com>
 
         Reviewed by Darin (reluctantly).
index c87b0372b232d11b01913d255306936c32ec4ade..a8d2ae70286046fc3d3208d304928119d4b162bc 100644 (file)
@@ -2576,7 +2576,8 @@ HRESULT STDMETHODCALLTYPE WebFrame::spoolPages(
             }
 
             if (footerHeight) {
-                RECT footerRect = {x, pageRect.bottom(), x+pageRect.width(), pageRect.bottom()+(int)footerHeight};
+                y = (int)headerHeight+pageRect.height();
+                RECT footerRect = {x, y, x+pageRect.width(), y+(int)footerHeight};
                 ui2->drawFooterInRect(d->webView, &footerRect, (OLE_HANDLE)(LONG64)pctx, ii+1, pageCount);
             }
         }