Attachment 369116 Details for Bug 197167 – Patch

[patch] Patch

bug-197167-20190506114558.patch (text/plain), 2.38 KB, created by Zan Dobersek on 2019-05-06 02:45:59 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Zan Dobersek

Created: 2019-05-06 02:45:59 PDT

Size: 2.38 KB

patch

obsolete

>Subversion Revision: 244961
>diff --git a/Source/WTF/ChangeLog b/Source/WTF/ChangeLog
>index 8da671ede38b89b43d1d1a714ce7ab6090cfe38b..2be760f9af2a3b518c2276d6f32a550d85df1c70 100644
>--- a/Source/WTF/ChangeLog
>+++ b/Source/WTF/ChangeLog
>@@ -1,3 +1,22 @@
>+2019-05-06  Zan Dobersek  <zdobersek@igalia.com>
>+
>+        [GLib] Rework WPE RunLoopSourcePriority values
>+        https://bugs.webkit.org/show_bug.cgi?id=197167
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        The GLib priorities for the WPE port were initially aligned on the -70
>+        value, theory being that this would help avoid any default-priority
>+        GSources spawned in different dependency libraries affecting our
>+        scheduling. Today it seems that extra caution might not be really
>+        necessary.
>+
>+        This change aligns the base priority value with GLib's default priority
>+        value of 0. We maintain the relativity of priority values by effectively
>+        increasing each priority by 70.
>+
>+        * wtf/glib/RunLoopSourcePriority.h:
>+
> 2019-05-03  Sihui Liu  <sihui_liu@apple.com>
> 
>         Add assertion to check whether shm files have maximum FileProtection of CompleteUnlessOpen
>diff --git a/Source/WTF/wtf/glib/RunLoopSourcePriority.h b/Source/WTF/wtf/glib/RunLoopSourcePriority.h
>index 7127a4e759d8c8007ed11db6ab6183d15957ff03..24cf535be764166f780f0a60a5fa9a1eb790d01f 100644
>--- a/Source/WTF/wtf/glib/RunLoopSourcePriority.h
>+++ b/Source/WTF/wtf/glib/RunLoopSourcePriority.h
>@@ -85,26 +85,26 @@ enum RunLoopSourcePriority {
> #elif PLATFORM(WPE)
> 
> enum RunLoopSourcePriority {
>-    RunLoopDispatcher = -70,
>-    RunLoopTimer = -70,
>+    RunLoopDispatcher = 0,
>+    RunLoopTimer = 0,
> 
>-    MainThreadDispatcherTimer = -60,
>+    MainThreadDispatcherTimer = 10,
> 
>-    MemoryPressureHandlerTimer = -80,
>+    MemoryPressureHandlerTimer = -10,
> 
>-    JavascriptTimer = -60,
>-    MainThreadSharedTimer = -60,
>+    JavascriptTimer = 10,
>+    MainThreadSharedTimer = 10,
> 
>-    LayerFlushTimer = -70,
>-    DisplayRefreshMonitorTimer = -70,
>+    LayerFlushTimer = 0,
>+    DisplayRefreshMonitorTimer = 0,
> 
>-    CompositingThreadUpdateTimer = -70,
>+    CompositingThreadUpdateTimer = 0,
> 
>-    ReleaseUnusedResourcesTimer = -70,
>+    ReleaseUnusedResourcesTimer = 0,
> 
>-    AsyncIONetwork = -60,
>-    DiskCacheRead = -60,
>-    DiskCacheWrite = -50
>+    AsyncIONetwork = 10,
>+    DiskCacheRead = 10,
>+    DiskCacheWrite = 20
> };
> 
> #endif

Actions: View | Formatted Diff | Diff

Attachments on bug 197167: 367949 | 369116 | 369120