Title: [152165] trunk/Source/WebKit2
- Revision
- 152165
- Author
- [email protected]
- Date
- 2013-06-28 07:49:24 -0700 (Fri, 28 Jun 2013)
Log Message
[WK2][GTK] 100% CPU usage rendering a page with animations
https://bugs.webkit.org/show_bug.cgi?id=117703
Reviewed by Martin Robinson.
Unconditionally update the value of m_lastFlushTime in
::flushAndRenderLayers. By doing that we avoid a neverending
sequence of layer flushes that happened when m_lastFlushTime==0
and flushPendingLayerChanges() was false. In that situation the
value m_lastFlushTime was never updated so a new flush was always
rescheduled.
* WebProcess/WebPage/gtk/LayerTreeHostGtk.cpp:
(WebKit::LayerTreeHostGtk::flushAndRenderLayers):
Modified Paths
Diff
Modified: trunk/Source/WebKit2/ChangeLog (152164 => 152165)
--- trunk/Source/WebKit2/ChangeLog 2013-06-28 14:29:58 UTC (rev 152164)
+++ trunk/Source/WebKit2/ChangeLog 2013-06-28 14:49:24 UTC (rev 152165)
@@ -1,3 +1,20 @@
+2013-06-28 Sergio Villar Senin <[email protected]>
+
+ [WK2][GTK] 100% CPU usage rendering a page with animations
+ https://bugs.webkit.org/show_bug.cgi?id=117703
+
+ Reviewed by Martin Robinson.
+
+ Unconditionally update the value of m_lastFlushTime in
+ ::flushAndRenderLayers. By doing that we avoid a neverending
+ sequence of layer flushes that happened when m_lastFlushTime==0
+ and flushPendingLayerChanges() was false. In that situation the
+ value m_lastFlushTime was never updated so a new flush was always
+ rescheduled.
+
+ * WebProcess/WebPage/gtk/LayerTreeHostGtk.cpp:
+ (WebKit::LayerTreeHostGtk::flushAndRenderLayers):
+
2013-06-28 Andres Gomez <[email protected]>
[GTK] MiniBrowser doesn't solve the saving location when there are no XDG settings
Modified: trunk/Source/WebKit2/WebProcess/WebPage/gtk/LayerTreeHostGtk.cpp (152164 => 152165)
--- trunk/Source/WebKit2/WebProcess/WebPage/gtk/LayerTreeHostGtk.cpp 2013-06-28 14:29:58 UTC (rev 152164)
+++ trunk/Source/WebKit2/WebProcess/WebPage/gtk/LayerTreeHostGtk.cpp 2013-06-28 14:49:24 UTC (rev 152165)
@@ -360,10 +360,10 @@
if (!context || !context->makeContextCurrent())
return;
+ m_lastFlushTime = currentTime();
if (!flushPendingLayerChanges())
return;
- m_lastFlushTime = currentTime();
// Our model is very simple. We always composite and render the tree immediately after updating it.
compositeLayersToContext();
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
