Title: [220581] trunk/Source/WebKit
Revision
220581
Author
carlo...@webkit.org
Date
2017-08-10 23:02:33 -0700 (Thu, 10 Aug 2017)

Log Message

[Soup] Do not spawn the network process to setup cookie persistent storage
https://bugs.webkit.org/show_bug.cgi?id=175424

Reviewed by Sergio Villar Senin.

The persistent storage configuration is saved and passed to network process as creation parameters, so we don't
need to spawn the network process to configure it, when it's created eventually it will use the configuration.

* UIProcess/soup/WebCookieManagerProxySoup.cpp:
(WebKit::WebCookieManagerProxy::setCookiePersistentStorage): Use sendToNetworkingProcess() instead of
endToNetworkingProcessRelaunchingIfNecessary().

Modified Paths

Diff

Modified: trunk/Source/WebKit/ChangeLog (220580 => 220581)


--- trunk/Source/WebKit/ChangeLog	2017-08-11 06:01:15 UTC (rev 220580)
+++ trunk/Source/WebKit/ChangeLog	2017-08-11 06:02:33 UTC (rev 220581)
@@ -1,5 +1,19 @@
 2017-08-10  Carlos Garcia Campos  <cgar...@igalia.com>
 
+        [Soup] Do not spawn the network process to setup cookie persistent storage
+        https://bugs.webkit.org/show_bug.cgi?id=175424
+
+        Reviewed by Sergio Villar Senin.
+
+        The persistent storage configuration is saved and passed to network process as creation parameters, so we don't
+        need to spawn the network process to configure it, when it's created eventually it will use the configuration.
+
+        * UIProcess/soup/WebCookieManagerProxySoup.cpp:
+        (WebKit::WebCookieManagerProxy::setCookiePersistentStorage): Use sendToNetworkingProcess() instead of
+        endToNetworkingProcessRelaunchingIfNecessary().
+
+2017-08-10  Carlos Garcia Campos  <cgar...@igalia.com>
+
         [GTK][WPE] Assertion failure in TimerBase inside WebCore::IconRecord::setImageData
         https://bugs.webkit.org/show_bug.cgi?id=173866
         <rdar://problem/33122050>

Modified: trunk/Source/WebKit/UIProcess/soup/WebCookieManagerProxySoup.cpp (220580 => 220581)


--- trunk/Source/WebKit/UIProcess/soup/WebCookieManagerProxySoup.cpp	2017-08-11 06:01:15 UTC (rev 220580)
+++ trunk/Source/WebKit/UIProcess/soup/WebCookieManagerProxySoup.cpp	2017-08-11 06:02:33 UTC (rev 220581)
@@ -33,10 +33,9 @@
 
 void WebCookieManagerProxy::setCookiePersistentStorage(const String& storagePath, uint32_t storageType)
 {
-    processPool()->sendToNetworkingProcessRelaunchingIfNecessary(Messages::WebCookieManager::SetCookiePersistentStorage(storagePath, storageType));
-
     m_cookiePersistentStoragePath = storagePath;
     m_cookiePersistentStorageType = static_cast<SoupCookiePersistentStorageType>(storageType);
+    processPool()->sendToNetworkingProcess(Messages::WebCookieManager::SetCookiePersistentStorage(storagePath, storageType));
 }
 
 void WebCookieManagerProxy::getCookiePersistentStorage(String& storagePath, uint32_t& storageType) const
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to