Title: [87654] trunk/Source/WebKit2
Revision
87654
Author
[email protected]
Date
2011-05-29 15:14:49 -0700 (Sun, 29 May 2011)

Log Message

REGRESSION (r87637): nytimes.com freezes after going back to it
<rdar://problem/9521928>

Reviewed by Anders Carlsson.

* WebProcess/WebPage/DrawingAreaImpl.cpp:
(WebKit::DrawingAreaImpl::forceRepaint):
Add call to setShouldNotifyAfterNextScheduledLayerFlush(false) to prevent a
redundant call to layerHostDidFlushLayers which would cause the layer state
to get out of sync with the UIProcess.

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (87653 => 87654)


--- trunk/Source/WebKit2/ChangeLog	2011-05-29 21:56:53 UTC (rev 87653)
+++ trunk/Source/WebKit2/ChangeLog	2011-05-29 22:14:49 UTC (rev 87654)
@@ -1,3 +1,16 @@
+2011-05-29  Sam Weinig  <[email protected]>
+
+        Reviewed by Anders Carlsson.
+
+        REGRESSION (r87637): nytimes.com freezes after going back to it
+        <rdar://problem/9521928>
+
+        * WebProcess/WebPage/DrawingAreaImpl.cpp:
+        (WebKit::DrawingAreaImpl::forceRepaint):
+        Add call to setShouldNotifyAfterNextScheduledLayerFlush(false) to prevent a
+        redundant call to layerHostDidFlushLayers which would cause the layer state
+        to get out of sync with the UIProcess.
+
 2011-05-29  Jer Noble  <[email protected]>
 
         Reviewed by Simon Fraser.

Modified: trunk/Source/WebKit2/WebProcess/WebPage/DrawingAreaImpl.cpp (87653 => 87654)


--- trunk/Source/WebKit2/WebProcess/WebPage/DrawingAreaImpl.cpp	2011-05-29 21:56:53 UTC (rev 87653)
+++ trunk/Source/WebKit2/WebProcess/WebPage/DrawingAreaImpl.cpp	2011-05-29 22:14:49 UTC (rev 87654)
@@ -161,8 +161,12 @@
         // Consider refactoring and renaming it.
         if (m_compositingAccordingToProxyMessages)
             m_layerTreeHost->forceRepaint();
-        else
+        else {
+            // Call setShouldNotifyAfterNextScheduledLayerFlush(false) here to 
+            // prevent layerHostDidFlushLayers() from being called a second time.
+            m_layerTreeHost->setShouldNotifyAfterNextScheduledLayerFlush(false);
             layerHostDidFlushLayers();
+        }
         if (!m_layerTreeHost->participatesInDisplay())
             return;
     }
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to