Title: [133470] branches/safari-536.28-branch/Source/WebCore
Revision
133470
Author
lforsch...@apple.com
Date
2012-11-05 07:06:17 -0800 (Mon, 05 Nov 2012)

Log Message

Merged r124919.  <rdar://problem/12516354>

Modified Paths

Diff

Modified: branches/safari-536.28-branch/Source/WebCore/ChangeLog (133469 => 133470)


--- branches/safari-536.28-branch/Source/WebCore/ChangeLog	2012-11-05 15:02:29 UTC (rev 133469)
+++ branches/safari-536.28-branch/Source/WebCore/ChangeLog	2012-11-05 15:06:17 UTC (rev 133470)
@@ -1,5 +1,24 @@
 2012-11-05  Lucas Forschler  <lforsch...@apple.com>
 
+        Merge r124919
+
+    2012-08-07  Adrienne Walker  <e...@google.com>
+
+            50% fixed position coverage slow scroll heuristic is incorrect when invalidations aren't clipped
+            https://bugs.webkit.org/show_bug.cgi?id=92011
+
+            Reviewed by Simon Fraser.
+
+            The heuristic in scrollContentsFastPath to slow scroll by invalidating
+            the entire frame if fixed position elements cover 50% of the frame
+            takes away the ability of ports to make their own decisions about how
+            to best handle invalidations. Therefore, remove this heuristic.
+
+            * page/FrameView.cpp:
+            (WebCore::FrameView::scrollContentsFastPath):
+
+2012-11-05  Lucas Forschler  <lforsch...@apple.com>
+
         Merge r124914
 
     2012-08-07  Abhishek Arya  <infe...@chromium.org>
@@ -206358,3 +206377,4 @@
 .
 .
 .
+.

Modified: branches/safari-536.28-branch/Source/WebCore/page/FrameView.cpp (133469 => 133470)


--- branches/safari-536.28-branch/Source/WebCore/page/FrameView.cpp	2012-11-05 15:02:29 UTC (rev 133469)
+++ branches/safari-536.28-branch/Source/WebCore/page/FrameView.cpp	2012-11-05 15:06:17 UTC (rev 133470)
@@ -1494,10 +1494,6 @@
             regionToUpdate.unite(updateRect);
     }
 
-    // The area to be painted by fixed objects exceeds 50% of the area of the view, we cannot use the fast path.
-    if (regionToUpdate.totalArea() > (clipRect.width() * clipRect.height() * 0.5))
-        return false;
-
     // 1) scroll
     hostWindow()->scroll(scrollDelta, rectToScroll, clipRect);
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to