Title: [94587] trunk/Source/WebCore
Revision
94587
Author
[email protected]
Date
2011-09-06 12:03:26 -0700 (Tue, 06 Sep 2011)

Log Message

[chromium] REGRESSION(94353): requestAnimationFrame not throttled in compositing path
https://bugs.webkit.org/show_bug.cgi?id=67621

CCSingleThreadProxy should not perform layout when called
via the compositeImmediately path. Doing so makes it look
like frame rate is unbounded.

Reviewed by James Robinson.

* platform/graphics/chromium/cc/CCSingleThreadProxy.cpp:
(WebCore::CCSingleThreadProxy::compositeAndReadback):
(WebCore::CCSingleThreadProxy::compositeImmediately):
(WebCore::CCSingleThreadProxy::commitIfNeeded):
* platform/graphics/chromium/cc/CCSingleThreadProxy.h:

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (94586 => 94587)


--- trunk/Source/WebCore/ChangeLog	2011-09-06 19:00:51 UTC (rev 94586)
+++ trunk/Source/WebCore/ChangeLog	2011-09-06 19:03:26 UTC (rev 94587)
@@ -1,3 +1,20 @@
+2011-09-06  Nat Duca  <[email protected]>
+
+        [chromium] REGRESSION(94353): requestAnimationFrame not throttled in compositing path
+        https://bugs.webkit.org/show_bug.cgi?id=67621
+
+        CCSingleThreadProxy should not perform layout when called
+        via the compositeImmediately path. Doing so makes it look
+        like frame rate is unbounded.
+
+        Reviewed by James Robinson.
+
+        * platform/graphics/chromium/cc/CCSingleThreadProxy.cpp:
+        (WebCore::CCSingleThreadProxy::compositeAndReadback):
+        (WebCore::CCSingleThreadProxy::compositeImmediately):
+        (WebCore::CCSingleThreadProxy::commitIfNeeded):
+        * platform/graphics/chromium/cc/CCSingleThreadProxy.h:
+
 2011-09-06  Ryosuke Niwa  <[email protected]>
 
         REGRESSION(r94274): FormManagerTest.PreviewForm and FillFormNonEmptyField fail on chromium

Modified: trunk/Source/WebCore/platform/graphics/chromium/cc/CCSingleThreadProxy.cpp (94586 => 94587)


--- trunk/Source/WebCore/platform/graphics/chromium/cc/CCSingleThreadProxy.cpp	2011-09-06 19:00:51 UTC (rev 94586)
+++ trunk/Source/WebCore/platform/graphics/chromium/cc/CCSingleThreadProxy.cpp	2011-09-06 19:03:26 UTC (rev 94587)
@@ -90,7 +90,6 @@
 
     commitIfNeeded();
 
-
     if (!doComposite())
         return false;
 
@@ -232,8 +231,6 @@
 void CCSingleThreadProxy::commitIfNeeded()
 {
     // Update
-    double frameBeginTime = currentTime();
-    m_layerTreeHost->animateAndLayout(frameBeginTime);
     m_layerTreeHost->updateLayers();
 
     // Commit
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to