Changeset 154711 in webkit

Timestamp:

Aug 27, 2013 3:23:16 PM (11 years ago)

Author:

Luciano Miguel Wolf

Message:

100% cpu usage for "transition: opacity" animation
​https://bugs.webkit.org/show_bug.cgi?id=120012

Reviewed by Noam Rosenthal.

Schedule animation timer after scheduling a layer flush. This way it
won't overwrite animation timer with "0", thus avoiding 100% cpu usage.

• WebProcess/WebPage/CoordinatedGraphics/CoordinatedLayerTreeHost.cpp:

(WebKit::CoordinatedLayerTreeHost::scheduleAnimation):

Location:

trunk/Source/WebKit2

Files:

• ChangeLog (modified) (1 diff)
• WebProcess/WebPage/CoordinatedGraphics/CoordinatedLayerTreeHost.cpp (modified) (1 diff)

trunk/Source/WebKit2/ChangeLog

@@ +1 @@
+2013-08-27  Luciano Wolf  <luciano.wolf@openbossa.org>
+
+        100% cpu usage for "transition: opacity" animation
+        https://bugs.webkit.org/show_bug.cgi?id=120012
+
+        Reviewed by Noam Rosenthal.
+
+        Schedule animation timer after scheduling a layer flush. This way it
+        won't overwrite animation timer with "0", thus avoiding 100% cpu usage.
+
+        * WebProcess/WebPage/CoordinatedGraphics/CoordinatedLayerTreeHost.cpp:
+        (WebKit::CoordinatedLayerTreeHost::scheduleAnimation):
+
 2013-08-27  Csaba Osztrogonác  <ossy@webkit.org>
 

trunk/Source/WebKit2/WebProcess/WebPage/CoordinatedGraphics/CoordinatedLayerTreeHost.cpp

@@ -397 +397 @@
         return;
 
+    scheduleLayerFlush();
     m_layerFlushTimer.startOneShot(m_coordinator->nextAnimationServiceTime());
-    scheduleLayerFlush();
 }
 #endif