2010-04-30 Shinichiro Hamaji <hamaji@chromium.org>
authorhamaji@chromium.org <hamaji@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 30 Apr 2010 09:14:28 +0000 (09:14 +0000)
committerhamaji@chromium.org <hamaji@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 30 Apr 2010 09:14:28 +0000 (09:14 +0000)
        Reviewed by Darin Adler.

        Unnecessary PrintContext::end() calls
        https://bugs.webkit.org/show_bug.cgi?id=38247

        Refactoring only, so no new tests.

        * page/PrintContext.cpp:
        (WebCore::PrintContext::pageNumberForElement):
        (WebCore::PrintContext::numberOfPages):

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

WebCore/ChangeLog
WebCore/page/PrintContext.cpp

index 3d03d8b..46418f9 100644 (file)
@@ -1,3 +1,16 @@
+2010-04-30  Shinichiro Hamaji  <hamaji@chromium.org>
+
+        Reviewed by Darin Adler.
+
+        Unnecessary PrintContext::end() calls
+        https://bugs.webkit.org/show_bug.cgi?id=38247
+
+        Refactoring only, so no new tests.
+
+        * page/PrintContext.cpp:
+        (WebCore::PrintContext::pageNumberForElement):
+        (WebCore::PrintContext::numberOfPages):
+
 2010-04-30  Kent Tamura  <tkent@chromium.org>
 
         Reviewed by Adele Peterson.
index 868f8e1..ffde0be 100644 (file)
@@ -193,10 +193,9 @@ int PrintContext::pageNumberForElement(Element* element, const FloatSize& pageSi
     for (; pageNumber < printContext.pageCount(); pageNumber++) {
         const IntRect& page = printContext.pageRect(pageNumber);
         if (page.x() <= left && left < page.right() && page.y() <= top && top < page.bottom())
-            break;
+            return pageNumber;
     }
-    printContext.end();
-    return (pageNumber < printContext.pageCount() ? pageNumber : -1);
+    return -1;
 }
 
 int PrintContext::numberOfPages(Frame* frame, const FloatSize& pageSizeInPixels)
@@ -207,7 +206,6 @@ int PrintContext::numberOfPages(Frame* frame, const FloatSize& pageSizeInPixels)
     PrintContext printContext(frame);
     printContext.begin(pageRect.width());
     printContext.computePageRectsWithPageSize(pageSizeInPixels, false);
-    printContext.end();
     return printContext.pageCount();
 }