Attachment 359161 Details for Bug 193400 – Patch

[patch] Patch

bug-193400-20190115082952.patch (text/plain), 1.70 KB, created by youenn fablet on 2019-01-15 08:29:52 PST

(hide)

Description:

Filename:

MIME Type:

Creator: youenn fablet

Created: 2019-01-15 08:29:52 PST

Size: 1.70 KB

patch

obsolete

>Subversion Revision: 239787
>diff --git a/Source/WebKit/ChangeLog b/Source/WebKit/ChangeLog
>index e3cec7c5ed64bb1d528944e374485abdd73ccf8b..8900fdd46bc8a92984016fdca4238af219b9d424 100644
>--- a/Source/WebKit/ChangeLog
>+++ b/Source/WebKit/ChangeLog
>@@ -1,3 +1,15 @@
>+2019-01-15  Youenn Fablet  <youenn@apple.com>
>+
>+        Remove the specific handling of ephemeral sessions from CacheStorage::Engine::from
>+        https://bugs.webkit.org/show_bug.cgi?id=193400
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        It is no longer needed as we correctly handle ephemeral sessions removal.
>+
>+        * NetworkProcess/cache/CacheStorageEngine.cpp:
>+        (WebKit::CacheStorage::Engine::from):
>+
> 2019-01-10  Youenn Fablet  <youenn@apple.com>
> 
>         Add a new SPI to request for cache storage quota increase
>diff --git a/Source/WebKit/NetworkProcess/cache/CacheStorageEngine.cpp b/Source/WebKit/NetworkProcess/cache/CacheStorageEngine.cpp
>index 34122bb61420a5e447b35712fbdf7c40fc7dfb4c..1b841cfe6748f02aa1768f487ae7acb126863c99 100644
>--- a/Source/WebKit/NetworkProcess/cache/CacheStorageEngine.cpp
>+++ b/Source/WebKit/NetworkProcess/cache/CacheStorageEngine.cpp
>@@ -80,9 +80,6 @@ void Engine::from(NetworkProcess& networkProcess, PAL::SessionID sessionID, Func
>         return;
>     }
> 
>-    if (sessionID.isEphemeral())
>-        sessionID = PAL::SessionID::legacyPrivateSessionID();
>-
>     networkProcess.cacheStorageParameters(sessionID, [networkProcess = makeRef(networkProcess), sessionID, callback = WTFMove(callback)] (auto&& rootPath, auto quota) mutable {
>         callback(networkProcess->ensureCacheEngine(sessionID, [&] {
>             return adoptRef(*new Engine { sessionID, networkProcess.get(), String { rootPath }, quota });

Actions: View | Formatted Diff | Diff

Attachments on bug 193400: 359043 | 359161