Title: [264153] releases/WebKitGTK/webkit-2.28/Source/WTF
- Revision
- 264153
- Author
- carlo...@webkit.org
- Date
- 2020-07-09 00:01:10 -0700 (Thu, 09 Jul 2020)
Log Message
Merge r264015 - [GTK] WebProcess hangs when browsing GitHub
https://bugs.webkit.org/show_bug.cgi?id=213970
Reviewed by Sergio Villar Senin.
Use a lower priority for LayerFlushTimer and DisplayRefreshMonitorTimer. We were using a very high priority for
drawing with the idea of keeping a good rendering performance without being affected by other timers. The
problem is that animations can be controlled by timers, so we need to ensure that MainThreadSharedTimer has
higher priority than drawing.
* wtf/glib/RunLoopSourcePriority.h: Use 110 for LayerFlushTimer and DisplayRefreshMonitorTimer.
Modified Paths
Diff
Modified: releases/WebKitGTK/webkit-2.28/Source/WTF/ChangeLog (264152 => 264153)
--- releases/WebKitGTK/webkit-2.28/Source/WTF/ChangeLog 2020-07-09 06:30:53 UTC (rev 264152)
+++ releases/WebKitGTK/webkit-2.28/Source/WTF/ChangeLog 2020-07-09 07:01:10 UTC (rev 264153)
@@ -1,3 +1,17 @@
+2020-07-07 Carlos Garcia Campos <cgar...@igalia.com>
+
+ [GTK] WebProcess hangs when browsing GitHub
+ https://bugs.webkit.org/show_bug.cgi?id=213970
+
+ Reviewed by Sergio Villar Senin.
+
+ Use a lower priority for LayerFlushTimer and DisplayRefreshMonitorTimer. We were using a very high priority for
+ drawing with the idea of keeping a good rendering performance without being affected by other timers. The
+ problem is that animations can be controlled by timers, so we need to ensure that MainThreadSharedTimer has
+ higher priority than drawing.
+
+ * wtf/glib/RunLoopSourcePriority.h: Use 110 for LayerFlushTimer and DisplayRefreshMonitorTimer.
+
2020-04-10 Alicia Boya García <ab...@igalia.com>
[WTF] DataMutex: Add runUnlocked()
Modified: releases/WebKitGTK/webkit-2.28/Source/WTF/wtf/glib/RunLoopSourcePriority.h (264152 => 264153)
--- releases/WebKitGTK/webkit-2.28/Source/WTF/wtf/glib/RunLoopSourcePriority.h 2020-07-09 06:30:53 UTC (rev 264152)
+++ releases/WebKitGTK/webkit-2.28/Source/WTF/wtf/glib/RunLoopSourcePriority.h 2020-07-09 07:01:10 UTC (rev 264153)
@@ -35,7 +35,7 @@
// G_PRIORITY_HIGH = -100
// We don't use those macros here to avoid having to include glib header only
// for this. But we should take into account that GLib uses G_PRIORITY_DEFAULT
-// for timeout sourcea and G_PRIORITY_DEFAULT_IDLE for idle sources.
+// for timeout sources and G_PRIORITY_DEFAULT_IDLE for idle sources.
// Changes in these priorities can have a huge impact in performance, and in
// the correctness too, so be careful when changing them.
enum RunLoopSourcePriority {
@@ -64,10 +64,10 @@
CompositingThreadUpdateTimer = 110,
// Layer flush.
- LayerFlushTimer = -100,
+ LayerFlushTimer = 110,
// DisplayRefreshMonitor timer, should have the same value as the LayerFlushTimer.
- DisplayRefreshMonitorTimer = -100,
+ DisplayRefreshMonitorTimer = 110,
// Rendering timer in the main thread when accelerated compositing is not used.
NonAcceleratedDrawingTimer = 100,
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes
