Attachment 341872 Details for Bug 186247 – Patch

[patch] Patch

bug-186247-20180603144958.patch (text/plain), 1.65 KB, created by Andy Estes on 2018-06-03 14:49:59 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Andy Estes

Created: 2018-06-03 14:49:59 PDT

Size: 1.65 KB

patch

obsolete

>Subversion Revision: 232453
>diff --git a/Source/WebKit/ChangeLog b/Source/WebKit/ChangeLog
>index eaa082cceb5e276f71fa5e9a137a7ee98d7cf84f..71c9a758ffb2ec076348ad68360e4dfd3ce6f435 100644
>--- a/Source/WebKit/ChangeLog
>+++ b/Source/WebKit/ChangeLog
>@@ -1,3 +1,16 @@
>+2018-06-03  Andy Estes  <aestes@apple.com>
>+
>+        [Wi-Fi Assertions] Resume assertions when NetworkProcess::cancelPrepareToSuspend is called
>+        https://bugs.webkit.org/show_bug.cgi?id=186247
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Since we suspended assertions in prepareToSuspend, we need to resume them if
>+        cancelPrepareToSuspend is called.
>+
>+        * NetworkProcess/NetworkProcess.cpp:
>+        (WebKit::NetworkProcess::cancelPrepareToSuspend):
>+
> 2018-06-03  Brent Fulgham  <bfulgham@apple.com>
> 
>         Make sure that the fencePort received over IPC has the expected disposition (SEND)
>diff --git a/Source/WebKit/NetworkProcess/NetworkProcess.cpp b/Source/WebKit/NetworkProcess/NetworkProcess.cpp
>index db13dbc9201c6f41b2c85ea2b58792a2a3e500f0..91d30b66b99acd9311f6bc58cb7f9eae97961ca3 100644
>--- a/Source/WebKit/NetworkProcess/NetworkProcess.cpp
>+++ b/Source/WebKit/NetworkProcess/NetworkProcess.cpp
>@@ -832,6 +832,7 @@ void NetworkProcess::cancelPrepareToSuspend()
>     // message. And NetworkProcessProxy expects to receive either a NetworkProcessProxy::ProcessReadyToSuspend-
>     // or NetworkProcessProxy::DidCancelProcessSuspension- message, but not both.
>     RELEASE_LOG(ProcessSuspension, "%p - NetworkProcess::cancelPrepareToSuspend()", this);
>+    platformProcessDidResume();
>     for (auto& connection : m_webProcessConnections)
>         connection->endSuspension();
> }

Actions: View | Formatted Diff | Diff

Attachments on bug 186247: 341872 | 341884