[iOS] REGRESSION (r162663): Webpages have strange artifacts or corruption around...
authoraestes@apple.com <aestes@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 29 Jan 2014 03:06:01 +0000 (03:06 +0000)
committeraestes@apple.com <aestes@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 29 Jan 2014 03:06:01 +0000 (03:06 +0000)
https://bugs.webkit.org/show_bug.cgi?id=127823

Reviewed by Daniel Bates.

Original patch by Simon Fraser.

* platform/ScrollView.cpp: Specify LegacyIOSDocumentViewRect when calling visibleContentRect().

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

Source/WebCore/ChangeLog
Source/WebCore/platform/ScrollView.cpp

index 16968a0..ebb999e 100644 (file)
@@ -1,3 +1,14 @@
+2014-01-28  Andy Estes  <aestes@apple.com>
+
+        [iOS] REGRESSION (r162663): Webpages have strange artifacts or corruption around tile boundaries
+        https://bugs.webkit.org/show_bug.cgi?id=127823
+
+        Reviewed by Daniel Bates.
+
+        Original patch by Simon Fraser.
+
+        * platform/ScrollView.cpp: Specify LegacyIOSDocumentViewRect when calling visibleContentRect().
+
 2014-01-28  Zoltan Horvath  <zoltan@webkit.org>
 
         [CSS Shapes] Adjust inset sizing syntax to the latest specification
index 7d692ca..2e81d22 100644 (file)
@@ -382,7 +382,7 @@ int ScrollView::scrollSize(ScrollbarOrientation orientation) const
 {
     // If no scrollbars are present, it does not indicate content is not be scrollable.
     if (!m_horizontalScrollbar && !m_verticalScrollbar && !prohibitsScrolling()) {
-        IntSize scrollSize = m_contentsSize - visibleContentRect().size();
+        IntSize scrollSize = m_contentsSize - visibleContentRect(LegacyIOSDocumentVisibleRect).size();
         scrollSize.clampNegativeToZero();
         return orientation == HorizontalScrollbar ? scrollSize.width() : scrollSize.height();
     }
@@ -1025,7 +1025,7 @@ void ScrollView::repaintContentRectangle(const IntRect& rect, bool now)
 {
     IntRect paintRect = rect;
     if (clipsRepaints() && !paintsEntireContents())
-        paintRect.intersect(visibleContentRect());
+        paintRect.intersect(visibleContentRect(LegacyIOSDocumentVisibleRect));
     if (paintRect.isEmpty())
         return;
 
@@ -1142,7 +1142,7 @@ void ScrollView::paint(GraphicsContext* context, const IntRect& rect)
 
     IntRect documentDirtyRect = rect;
     if (!paintsEntireContents()) {
-        IntRect visibleAreaWithoutScrollbars(location(), visibleContentRect().size());
+        IntRect visibleAreaWithoutScrollbars(location(), visibleContentRect(LegacyIOSDocumentVisibleRect).size());
         documentDirtyRect.intersect(visibleAreaWithoutScrollbars);
     }
 
@@ -1156,7 +1156,7 @@ void ScrollView::paint(GraphicsContext* context, const IntRect& rect)
             context->translate(-scrollX(), -scrollY());
             documentDirtyRect.moveBy(scrollPosition());
 
-            context->clip(visibleContentRect());
+            context->clip(visibleContentRect(LegacyIOSDocumentVisibleRect));
         }
 
         paintContents(context, documentDirtyRect);