Add FIXME comment after r223803
authorachristensen@apple.com <achristensen@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 21 Oct 2017 06:16:54 +0000 (06:16 +0000)
committerachristensen@apple.com <achristensen@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 21 Oct 2017 06:16:54 +0000 (06:16 +0000)
https://bugs.webkit.org/show_bug.cgi?id=178418

* UIProcess/WebProcessPool.cpp:
(WebKit::WebProcessPool::ensureNetworkProcess):
We need some work to better support customized persistent data stores.  Add a comment indicating such.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@223808 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit/ChangeLog
Source/WebKit/UIProcess/WebProcessPool.cpp

index b87ddc9..f69dd21 100644 (file)
@@ -1,5 +1,14 @@
 2017-10-20  Alex Christensen  <achristensen@webkit.org>
 
+        Add FIXME comment after r223803
+        https://bugs.webkit.org/show_bug.cgi?id=178418
+
+        * UIProcess/WebProcessPool.cpp:
+        (WebKit::WebProcessPool::ensureNetworkProcess):
+        We need some work to better support customized persistent data stores.  Add a comment indicating such.
+
+2017-10-20  Alex Christensen  <achristensen@webkit.org>
+
         Fix API tests after r223791.
         https://bugs.webkit.org/show_bug.cgi?id=178418
 
index 24dace2..10f8699 100644 (file)
@@ -427,6 +427,9 @@ NetworkProcessProxy& WebProcessPool::ensureNetworkProcess(WebsiteDataStore* with
     if (withWebsiteDataStore) {
         auto websiteDataStoreParameters = withWebsiteDataStore->parameters();
         parameters.defaultSessionParameters = websiteDataStoreParameters.networkSessionParameters;
+
+        // FIXME: This isn't conceptually correct, but it's needed to preserve behavior introduced in r213241.
+        // We should separate the concept of the default session from the currently used persistent session.
         parameters.defaultSessionParameters.sessionID = PAL::SessionID::defaultSessionID();
     }