Set main thread QoS policies after IPC initialization.
authorakling@apple.com <akling@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 18 Jun 2014 15:09:26 +0000 (15:09 +0000)
committerakling@apple.com <akling@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 18 Jun 2014 15:09:26 +0000 (15:09 +0000)
<https://webkit.org/b/134014>

Something was flipping the QoS level back to "unspecified" after
setting it in the ChildProcess initialization code. Pending a better
understanding of what really happens, move the code to a later stage,
after IPC channels are up and running. Now the priority sticks.

Reviewed by Anders Carlsson.

* NetworkProcess/NetworkProcess.cpp:
(WebKit::NetworkProcess::initializeNetworkProcess):
* Shared/ChildProcess.cpp:
(WebKit::ChildProcess::initialize):
* WebProcess/WebProcess.cpp:
(WebKit::WebProcess::initializeWebProcess):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@170100 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit2/ChangeLog
Source/WebKit2/NetworkProcess/NetworkProcess.cpp
Source/WebKit2/Shared/ChildProcess.cpp
Source/WebKit2/WebProcess/WebProcess.cpp

index cf04776..4079135 100644 (file)
@@ -1,3 +1,22 @@
+2014-06-18  Andreas Kling  <akling@apple.com>
+
+        Set main thread QoS policies after IPC initialization.
+        <https://webkit.org/b/134014>
+
+        Something was flipping the QoS level back to "unspecified" after
+        setting it in the ChildProcess initialization code. Pending a better
+        understanding of what really happens, move the code to a later stage,
+        after IPC channels are up and running. Now the priority sticks.
+
+        Reviewed by Anders Carlsson.
+
+        * NetworkProcess/NetworkProcess.cpp:
+        (WebKit::NetworkProcess::initializeNetworkProcess):
+        * Shared/ChildProcess.cpp:
+        (WebKit::ChildProcess::initialize):
+        * WebProcess/WebProcess.cpp:
+        (WebKit::WebProcess::initializeWebProcess):
+
 2014-06-17  Dan Bernstein  <mitz@apple.com>
 
         [Cocoa] WKWebView's _privateBrowsingEnabled property is unused
index ed8332a..5eb545e 100644 (file)
@@ -157,6 +157,8 @@ void NetworkProcess::initializeNetworkProcess(const NetworkProcessCreationParame
 {
     platformInitializeNetworkProcess(parameters);
 
+    WTF::setCurrentThreadIsUserInitiated();
+
     memoryPressureHandler().setLowMemoryHandler(lowMemoryHandler);
     memoryPressureHandler().install();
 
index dad43cd..d5693e9 100644 (file)
@@ -61,8 +61,6 @@ void ChildProcess::initialize(const ChildProcessInitializationParameters& parame
 {
     platformInitialize();
 
-    WTF::setCurrentThreadIsUserInitiated();
-
     initializeProcess(parameters);
     initializeProcessName(parameters);
 
index 694222d..98798b7 100644 (file)
@@ -270,6 +270,8 @@ void WebProcess::initializeWebProcess(const WebProcessCreationParameters& parame
 
     platformInitializeWebProcess(parameters, decoder);
 
+    WTF::setCurrentThreadIsUserInitiated();
+
     memoryPressureHandler().install();
 
     RefPtr<API::Object> injectedBundleInitializationUserData;