Title: [235374] trunk/Source/WebKit
Revision
235374
Author
[email protected]
Date
2018-08-27 09:43:23 -0700 (Mon, 27 Aug 2018)

Log Message

Don't launch network process in WebCookieManagerProxy::setHTTPCookieAcceptPolicy
https://bugs.webkit.org/show_bug.cgi?id=188906
<rdar://problem/42875795>

Reviewed by Ryosuke Niwa.

Add callback in early return.

* UIProcess/WebCookieManagerProxy.cpp:
(WebKit::WebCookieManagerProxy::setHTTPCookieAcceptPolicy):

Modified Paths

Diff

Modified: trunk/Source/WebKit/ChangeLog (235373 => 235374)


--- trunk/Source/WebKit/ChangeLog	2018-08-27 16:39:11 UTC (rev 235373)
+++ trunk/Source/WebKit/ChangeLog	2018-08-27 16:43:23 UTC (rev 235374)
@@ -1,3 +1,16 @@
+2018-08-27  Sihui Liu  <[email protected]>
+
+        Don't launch network process in WebCookieManagerProxy::setHTTPCookieAcceptPolicy
+        https://bugs.webkit.org/show_bug.cgi?id=188906
+        <rdar://problem/42875795>
+
+        Reviewed by Ryosuke Niwa.
+
+        Add callback in early return.
+
+        * UIProcess/WebCookieManagerProxy.cpp:
+        (WebKit::WebCookieManagerProxy::setHTTPCookieAcceptPolicy):
+
 2018-08-27  Alex Christensen  <[email protected]>
 
         Transition WKBrowsingContextController from WKPageLoaderClient to WKPageNavigationClient

Modified: trunk/Source/WebKit/UIProcess/WebCookieManagerProxy.cpp (235373 => 235374)


--- trunk/Source/WebKit/UIProcess/WebCookieManagerProxy.cpp	2018-08-27 16:39:11 UTC (rev 235373)
+++ trunk/Source/WebKit/UIProcess/WebCookieManagerProxy.cpp	2018-08-27 16:43:23 UTC (rev 235374)
@@ -251,8 +251,10 @@
     processPool()->setInitialHTTPCookieAcceptPolicy(policy);
 #endif
 
-    if (!processPool()->networkProcess())
+    if (!processPool()->networkProcess()) {
+        callbackFunction(CallbackBase::Error::None);
         return;
+    }
 
     auto callbackID = m_callbacks.put(WTFMove(callbackFunction), processPool()->ensureNetworkProcess().throttler().backgroundActivityToken());
     processPool()->sendToNetworkingProcess(Messages::WebCookieManager::SetHTTPCookieAcceptPolicy(policy, OptionalCallbackID(callbackID)));
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to