Title: [189587] trunk/Source/WebKit2
Revision
189587
Author
[email protected]
Date
2015-09-10 13:13:52 -0700 (Thu, 10 Sep 2015)

Log Message

sendProcessWillSuspendImminently uses a wrong message flag
https://bugs.webkit.org/show_bug.cgi?id=148995

Reviewed by Alexey Proskuryakov.

Remove use of flag IPC::InterruptWaitingIfSyncMessageArrives as it does not make
sense when sending a sync message.

* UIProcess/Network/NetworkProcessProxy.cpp:
(WebKit::NetworkProcessProxy::sendProcessWillSuspendImminently):
* UIProcess/WebProcessProxy.cpp:
(WebKit::WebProcessProxy::sendProcessWillSuspendImminently):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (189586 => 189587)


--- trunk/Source/WebKit2/ChangeLog	2015-09-10 19:49:36 UTC (rev 189586)
+++ trunk/Source/WebKit2/ChangeLog	2015-09-10 20:13:52 UTC (rev 189587)
@@ -1,3 +1,18 @@
+2015-09-10  Daniel Bates  <[email protected]>
+
+        sendProcessWillSuspendImminently uses a wrong message flag
+        https://bugs.webkit.org/show_bug.cgi?id=148995
+
+        Reviewed by Alexey Proskuryakov.
+
+        Remove use of flag IPC::InterruptWaitingIfSyncMessageArrives as it does not make
+        sense when sending a sync message.
+
+        * UIProcess/Network/NetworkProcessProxy.cpp:
+        (WebKit::NetworkProcessProxy::sendProcessWillSuspendImminently):
+        * UIProcess/WebProcessProxy.cpp:
+        (WebKit::WebProcessProxy::sendProcessWillSuspendImminently):
+
 2015-09-10  Joseph Pecoraro  <[email protected]>
 
         Unreviewed build fix after r189572.

Modified: trunk/Source/WebKit2/UIProcess/Network/NetworkProcessProxy.cpp (189586 => 189587)


--- trunk/Source/WebKit2/UIProcess/Network/NetworkProcessProxy.cpp	2015-09-10 19:49:36 UTC (rev 189586)
+++ trunk/Source/WebKit2/UIProcess/Network/NetworkProcessProxy.cpp	2015-09-10 20:13:52 UTC (rev 189587)
@@ -335,8 +335,7 @@
         return;
 
     bool handled = false;
-    sendSync(Messages::NetworkProcess::ProcessWillSuspendImminently(), Messages::NetworkProcess::ProcessWillSuspendImminently::Reply(handled),
-        0, std::chrono::seconds(1), IPC::InterruptWaitingIfSyncMessageArrives);
+    sendSync(Messages::NetworkProcess::ProcessWillSuspendImminently(), Messages::NetworkProcess::ProcessWillSuspendImminently::Reply(handled), 0, std::chrono::seconds(1));
 }
     
 void NetworkProcessProxy::sendPrepareToSuspend()

Modified: trunk/Source/WebKit2/UIProcess/WebProcessProxy.cpp (189586 => 189587)


--- trunk/Source/WebKit2/UIProcess/WebProcessProxy.cpp	2015-09-10 19:49:36 UTC (rev 189586)
+++ trunk/Source/WebKit2/UIProcess/WebProcessProxy.cpp	2015-09-10 20:13:52 UTC (rev 189587)
@@ -893,8 +893,7 @@
         return;
 
     bool handled = false;
-    sendSync(Messages::WebProcess::ProcessWillSuspendImminently(), Messages::WebProcess::ProcessWillSuspendImminently::Reply(handled),
-        0, std::chrono::seconds(1), IPC::InterruptWaitingIfSyncMessageArrives);
+    sendSync(Messages::WebProcess::ProcessWillSuspendImminently(), Messages::WebProcess::ProcessWillSuspendImminently::Reply(handled), 0, std::chrono::seconds(1));
 }
 
 void WebProcessProxy::sendPrepareToSuspend()
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to