Title: [210728] trunk/Source/WebKit2
Revision
210728
Author
carlo...@webkit.org
Date
2017-01-13 00:17:13 -0800 (Fri, 13 Jan 2017)

Log Message

[SOUP] Do not use the legacy session ID in WebFrameNetworkingContext::storageSession()
https://bugs.webkit.org/show_bug.cgi?id=166970

Reviewed by Sergio Villar Senin.

Get the session for the current page instead.

* WebProcess/WebCoreSupport/soup/WebFrameNetworkingContext.cpp:
(WebKit::WebFrameNetworkingContext::storageSession):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (210727 => 210728)


--- trunk/Source/WebKit2/ChangeLog	2017-01-13 07:36:54 UTC (rev 210727)
+++ trunk/Source/WebKit2/ChangeLog	2017-01-13 08:17:13 UTC (rev 210728)
@@ -1,3 +1,15 @@
+2017-01-12  Carlos Garcia Campos  <cgar...@igalia.com>
+
+        [SOUP] Do not use the legacy session ID in WebFrameNetworkingContext::storageSession()
+        https://bugs.webkit.org/show_bug.cgi?id=166970
+
+        Reviewed by Sergio Villar Senin.
+
+        Get the session for the current page instead.
+
+        * WebProcess/WebCoreSupport/soup/WebFrameNetworkingContext.cpp:
+        (WebKit::WebFrameNetworkingContext::storageSession):
+
 2017-01-12  Joseph Pecoraro  <pecor...@apple.com>
 
         Remove unnecessary includes of Page.h

Modified: trunk/Source/WebKit2/WebProcess/WebCoreSupport/soup/WebFrameNetworkingContext.cpp (210727 => 210728)


--- trunk/Source/WebKit2/WebProcess/WebCoreSupport/soup/WebFrameNetworkingContext.cpp	2017-01-13 07:36:54 UTC (rev 210727)
+++ trunk/Source/WebKit2/WebProcess/WebCoreSupport/soup/WebFrameNetworkingContext.cpp	2017-01-13 08:17:13 UTC (rev 210728)
@@ -83,9 +83,10 @@
 
 NetworkStorageSession& WebFrameNetworkingContext::storageSession() const
 {
-    if (frame() && frame()->page()->usesEphemeralSession())
-        return *NetworkStorageSession::storageSession(SessionID::legacyPrivateSessionID());
-
+    if (frame()) {
+        if (auto* storageSession = NetworkStorageSession::storageSession(frame()->page()->sessionID()))
+            return *storageSession;
+    }
     return NetworkStorageSession::defaultStorageSession();
 }
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to