Attachment 343291 Details for Bug 186806 – Patch

[patch] Patch

bug-186806-20180621170610.patch (text/plain), 7.15 KB, created by Sihui Liu on 2018-06-21 17:06:11 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Sihui Liu

Created: 2018-06-21 17:06:11 PDT

Size: 7.15 KB

patch

obsolete

>Subversion Revision: 233065
>diff --git a/Source/WebKit/ChangeLog b/Source/WebKit/ChangeLog
>index 05aa2ff1519d440bb2bbc3af5f1e0d6ce90a5161..1542874ff13b8e0e822426ee4ef3b8c2388ad1e7 100644
>--- a/Source/WebKit/ChangeLog
>+++ b/Source/WebKit/ChangeLog
>@@ -1,3 +1,28 @@
>+2018-06-21  Sihui Liu  <sihui_liu@apple.com>
>+
>+        REGRESSION (r231850): Cookie file cannot be read or written by network process
>+        https://bugs.webkit.org/show_bug.cgi?id=186806
>+        <rdar://problem/41113791>
>+
>+        Reviewed by Geoffrey Garen.
>+
>+        Add defaultSessionPendingCookies to NetworkProcessCreationParameters, so pending cookies of default session 
>+        can be added right after default session is set. This improves the fix r231850 as it does not send additional 
>+        message and avoids the regression.
>+
>+        * NetworkProcess/NetworkProcess.cpp:
>+        (WebKit::NetworkProcess::initializeNetworkProcess):
>+        * NetworkProcess/NetworkProcessCreationParameters.cpp:
>+        (WebKit::NetworkProcessCreationParameters::encode const):
>+        (WebKit::NetworkProcessCreationParameters::decode):
>+        * NetworkProcess/NetworkProcessCreationParameters.h:
>+        * NetworkProcess/mac/RemoteNetworkingContext.mm:
>+        (WebKit::RemoteNetworkingContext::ensureWebsiteDataStoreSession):
>+        * UIProcess/Network/NetworkProcessProxy.cpp:
>+        (WebKit::NetworkProcessProxy::NetworkProcessProxy):
>+        * UIProcess/WebProcessPool.cpp:
>+        (WebKit::WebProcessPool::ensureNetworkProcess):
>+
> 2018-06-21  Commit Queue  <commit-queue@webkit.org>
> 
>         Unreviewed, rolling out r232884.
>diff --git a/Source/WebKit/NetworkProcess/NetworkProcess.cpp b/Source/WebKit/NetworkProcess/NetworkProcess.cpp
>index 722a752c2fde72c345e3e0b95fd8700f665766de..184d127c38bf66af5e6c01df80039b2206f9e458 100644
>--- a/Source/WebKit/NetworkProcess/NetworkProcess.cpp
>+++ b/Source/WebKit/NetworkProcess/NetworkProcess.cpp
>@@ -269,6 +269,10 @@ void NetworkProcess::initializeNetworkProcess(NetworkProcessCreationParameters&&
> 
>     SessionTracker::setSession(PAL::SessionID::defaultSessionID(), NetworkSession::create(WTFMove(parameters.defaultSessionParameters)));
> 
>+    auto* defaultSession = SessionTracker::networkSession(PAL::SessionID::defaultSessionID());
>+    for (const auto& cookie : parameters.defaultSessionPendingCookies)
>+        defaultSession->networkStorageSession().setCookie(cookie);
>+
>     for (auto& supplement : m_supplements.values())
>         supplement->initialize(parameters);
> 
>diff --git a/Source/WebKit/NetworkProcess/NetworkProcessCreationParameters.cpp b/Source/WebKit/NetworkProcess/NetworkProcessCreationParameters.cpp
>index 2010b43739898df2fa3b18427d0f89dc96b9971c..8374dd3f94bbfab6965c1e8f92436b2acc18424f 100644
>--- a/Source/WebKit/NetworkProcess/NetworkProcessCreationParameters.cpp
>+++ b/Source/WebKit/NetworkProcess/NetworkProcessCreationParameters.cpp
>@@ -55,6 +55,7 @@ void NetworkProcessCreationParameters::encode(IPC::Encoder& encoder) const
> #if PLATFORM(COCOA)
>     encoder << uiProcessCookieStorageIdentifier;
> #endif
>+    encoder << defaultSessionPendingCookies;
> #if PLATFORM(IOS)
>     encoder << cookieStorageDirectoryExtensionHandle;
>     encoder << containerCachesDirectoryExtensionHandle;
>@@ -147,6 +148,8 @@ bool NetworkProcessCreationParameters::decode(IPC::Decoder& decoder, NetworkProc
>     if (!decoder.decode(result.uiProcessCookieStorageIdentifier))
>         return false;
> #endif
>+    if (!decoder.decode(result.defaultSessionPendingCookies))
>+        return false;
> #if PLATFORM(IOS)
>     std::optional<SandboxExtension::Handle> cookieStorageDirectoryExtensionHandle;
>     decoder >> cookieStorageDirectoryExtensionHandle;
>diff --git a/Source/WebKit/NetworkProcess/NetworkProcessCreationParameters.h b/Source/WebKit/NetworkProcess/NetworkProcessCreationParameters.h
>index 4388945b7cf5a60a10c52b9425aca8ba1d14fdb3..6bfdaec8c04a3a0d594f62148b2efaa6f94b3156 100644
>--- a/Source/WebKit/NetworkProcess/NetworkProcessCreationParameters.h
>+++ b/Source/WebKit/NetworkProcess/NetworkProcessCreationParameters.h
>@@ -28,6 +28,7 @@
> #include "CacheModel.h"
> #include "NetworkSessionCreationParameters.h"
> #include "SandboxExtension.h"
>+#include <WebCore/Cookie.h>
> #include <wtf/ProcessID.h>
> #include <wtf/Vector.h>
> #include <wtf/text/WTFString.h>
>@@ -65,6 +66,7 @@ struct NetworkProcessCreationParameters {
> #if PLATFORM(COCOA)
>     Vector<uint8_t> uiProcessCookieStorageIdentifier;
> #endif
>+    Vector<WebCore::Cookie> defaultSessionPendingCookies;
> #if PLATFORM(IOS)
>     SandboxExtension::Handle cookieStorageDirectoryExtensionHandle;
>     SandboxExtension::Handle containerCachesDirectoryExtensionHandle;
>diff --git a/Source/WebKit/NetworkProcess/mac/RemoteNetworkingContext.mm b/Source/WebKit/NetworkProcess/mac/RemoteNetworkingContext.mm
>index 1e02e91c8ca35e4a97ceaef61809155fdd53d2f5..154678c8e2f2f04c72673b73e3a083f6a46b915d 100644
>--- a/Source/WebKit/NetworkProcess/mac/RemoteNetworkingContext.mm
>+++ b/Source/WebKit/NetworkProcess/mac/RemoteNetworkingContext.mm
>@@ -46,12 +46,8 @@ namespace WebKit {
> void RemoteNetworkingContext::ensureWebsiteDataStoreSession(WebsiteDataStoreParameters&& parameters)
> {
>     auto sessionID = parameters.networkSessionParameters.sessionID;
>-    if (auto* session = NetworkStorageSession::storageSession(sessionID)) {
>-        ASSERT(parameters.pendingCookies.isEmpty() || sessionID == PAL::SessionID::defaultSessionID());
>-        for (const auto& cookie : parameters.pendingCookies)
>-            session->setCookie(cookie);
>+    if (NetworkStorageSession::storageSession(sessionID))
>         return;
>-    }
> 
>     String base;
>     if (SessionTracker::getIdentifierBase().isNull())
>diff --git a/Source/WebKit/UIProcess/Network/NetworkProcessProxy.cpp b/Source/WebKit/UIProcess/Network/NetworkProcessProxy.cpp
>index cbb182b65a6c0ace15300de964333013464624be..e7b2eec41562d29ff0b279a94abea646e84a3b23 100644
>--- a/Source/WebKit/UIProcess/Network/NetworkProcessProxy.cpp
>+++ b/Source/WebKit/UIProcess/Network/NetworkProcessProxy.cpp
>@@ -82,6 +82,9 @@ NetworkProcessProxy::NetworkProcessProxy(WebProcessPool& processPool)
>     , m_throttler(*this, processPool.shouldTakeUIBackgroundAssertion())
> {
>     connect();
>+
>+    if (auto* websiteDataStore = m_processPool.websiteDataStore())
>+        m_websiteDataStores.set(websiteDataStore->websiteDataStore().sessionID(), makeRef(websiteDataStore->websiteDataStore()));
> }
> 
> NetworkProcessProxy::~NetworkProcessProxy()
>diff --git a/Source/WebKit/UIProcess/WebProcessPool.cpp b/Source/WebKit/UIProcess/WebProcessPool.cpp
>index 7c6cec014ffd74e1ac52be4491c3743e85e10811..8284da5ac6dddf94057fcc2df88786e7dbd9a443 100644
>--- a/Source/WebKit/UIProcess/WebProcessPool.cpp
>+++ b/Source/WebKit/UIProcess/WebProcessPool.cpp
>@@ -528,11 +528,6 @@ NetworkProcessProxy& WebProcessPool::ensureNetworkProcess(WebsiteDataStore* with
>             m_websiteDataStore->websiteDataStore().networkProcessDidCrash();
>     }
> 
>-    if (m_websiteDataStore) {
>-        m_networkProcess->addSession(makeRef(m_websiteDataStore->websiteDataStore()));
>-        m_websiteDataStore->websiteDataStore().clearPendingCookies();
>-    }
>-    
>     if (withWebsiteDataStore) {
>         m_networkProcess->addSession(makeRef(*withWebsiteDataStore));
>         withWebsiteDataStore->clearPendingCookies();

Actions: View | Formatted Diff | Diff

Attachments on bug 186806: 343069 | 343101 | 343291 | 343377 | 343379