[iOS] TestWebKitAPI.WebKit.WKHTTPCookieStoreWithoutProcessPool fails because cookies...
[WebKit-https.git] / Source / WebKit / ChangeLog
index 7ec3353..9e33562 100644 (file)
@@ -1,3 +1,21 @@
+2018-06-05  Sihui Liu  <sihui_liu@apple.com>
+
+        [iOS] TestWebKitAPI.WebKit.WKHTTPCookieStoreWithoutProcessPool fails because cookies aren't flushed to file
+        https://bugs.webkit.org/show_bug.cgi?id=186303
+        <rdar://problem/40468716>
+
+        Reviewed by Geoffrey Garen.
+
+        No matter there is an observer or not, cookies in UI process should always be flushed to 
+        file when new process pool is created, such that cookies created via API can be synced to
+        network process.
+
+        * UIProcess/API/APIHTTPCookieStore.cpp:
+        (API::HTTPCookieStore::HTTPCookieStore):
+        (API::HTTPCookieStore::registerObserver):
+        (API::HTTPCookieStore::cookieManagerDestroyed):
+        (API::HTTPCookieStore::registerForNewProcessPoolNotifications):
+
 2018-06-05  Jeremy Jones  <jeremyj@apple.com>
 
         secheuristic should only use touch began and ended, not changed.