Title: [149086] trunk/Source/WebKit2
Revision
149086
Author
[email protected]
Date
2013-04-24 18:32:47 -0700 (Wed, 24 Apr 2013)

Log Message

Page Overlays shouldn't waste time flushing layers if we're not going to use it
https://bugs.webkit.org/show_bug.cgi?id=115145

Reviewed by Simon Fraser.

Don't flush the page overlay layer when it's installed - it starts out
with paintContents=false, so we don't really need a CALayer yet. We'll
flush later if need be.

* WebProcess/WebPage/mac/TiledCoreAnimationDrawingArea.mm:
(WebKit::TiledCoreAnimationDrawingArea::didInstallPageOverlay):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (149085 => 149086)


--- trunk/Source/WebKit2/ChangeLog	2013-04-25 01:32:07 UTC (rev 149085)
+++ trunk/Source/WebKit2/ChangeLog	2013-04-25 01:32:47 UTC (rev 149086)
@@ -1,3 +1,17 @@
+2013-04-24  Tim Horton  <[email protected]>
+
+        Page Overlays shouldn't waste time flushing layers if we're not going to use it
+        https://bugs.webkit.org/show_bug.cgi?id=115145
+
+        Reviewed by Simon Fraser.
+
+        Don't flush the page overlay layer when it's installed - it starts out
+        with paintContents=false, so we don't really need a CALayer yet. We'll
+        flush later if need be.
+
+        * WebProcess/WebPage/mac/TiledCoreAnimationDrawingArea.mm:
+        (WebKit::TiledCoreAnimationDrawingArea::didInstallPageOverlay):
+
 2013-04-24  Jer Noble  <[email protected]>
 
         Further unreviewed build fix; CFNetwork.framework is the one which Lion

Modified: trunk/Source/WebKit2/WebProcess/WebPage/mac/TiledCoreAnimationDrawingArea.mm (149085 => 149086)


--- trunk/Source/WebKit2/WebProcess/WebPage/mac/TiledCoreAnimationDrawingArea.mm	2013-04-25 01:32:07 UTC (rev 149085)
+++ trunk/Source/WebKit2/WebProcess/WebPage/mac/TiledCoreAnimationDrawingArea.mm	2013-04-25 01:32:47 UTC (rev 149086)
@@ -194,7 +194,6 @@
     m_webPage->corePage()->scrollingCoordinator()->setForceMainThreadScrollLayerPositionUpdates(true);
 
     createPageOverlayLayer(pageOverlay);
-    scheduleCompositingLayerFlush();
 }
 
 void TiledCoreAnimationDrawingArea::didUninstallPageOverlay(PageOverlay* pageOverlay)
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to