Title: [223220] trunk/Source/WebKit
Revision
223220
Author
[email protected]
Date
2017-10-11 17:46:59 -0700 (Wed, 11 Oct 2017)

Log Message

Add API to clean CacheStorage data
https://bugs.webkit.org/show_bug.cgi?id=178034

Unreviewed.

Patch by Youenn Fablet <[email protected]> on 2017-10-11

* NetworkProcess/NetworkProcess.cpp:
(WebKit::NetworkProcess::fetchWebsiteData): Fixing lifetime of aggregator.

Modified Paths

Diff

Modified: trunk/Source/WebKit/ChangeLog (223219 => 223220)


--- trunk/Source/WebKit/ChangeLog	2017-10-12 00:33:36 UTC (rev 223219)
+++ trunk/Source/WebKit/ChangeLog	2017-10-12 00:46:59 UTC (rev 223220)
@@ -3,6 +3,16 @@
         Add API to clean CacheStorage data
         https://bugs.webkit.org/show_bug.cgi?id=178034
 
+        Unreviewed.
+
+        * NetworkProcess/NetworkProcess.cpp:
+        (WebKit::NetworkProcess::fetchWebsiteData): Fixing lifetime of aggregator.
+
+2017-10-11  Youenn Fablet  <[email protected]>
+
+        Add API to clean CacheStorage data
+        https://bugs.webkit.org/show_bug.cgi?id=178034
+
         Reviewed by Chris Dumez.
 
         Introducing a new website data type for DOMCache.

Modified: trunk/Source/WebKit/NetworkProcess/NetworkProcess.cpp (223219 => 223220)


--- trunk/Source/WebKit/NetworkProcess/NetworkProcess.cpp	2017-10-12 00:33:36 UTC (rev 223219)
+++ trunk/Source/WebKit/NetworkProcess/NetworkProcess.cpp	2017-10-12 00:46:59 UTC (rev 223220)
@@ -403,14 +403,14 @@
             callbackAggregator->m_websiteData.originsWithCredentials = NetworkStorageSession::storageSession(sessionID)->credentialStorage().originsWithCredentials();
     }
 
-    if (websiteDataTypes.contains(WebsiteDataType::DiskCache)) {
-        fetchDiskCacheEntries(sessionID, fetchOptions, [callbackAggregator = WTFMove(callbackAggregator)](auto entries) mutable {
+    if (websiteDataTypes.contains(WebsiteDataType::DOMCache)) {
+        CacheStorage::Engine::fetchEntries(sessionID, fetchOptions.contains(WebsiteDataFetchOption::ComputeSizes), [callbackAggregator = callbackAggregator.copyRef()](auto entries) mutable {
             callbackAggregator->m_websiteData.entries.appendVector(entries);
         });
     }
 
-    if (websiteDataTypes.contains(WebsiteDataType::DOMCache)) {
-        CacheStorage::Engine::fetchEntries(sessionID, fetchOptions.contains(WebsiteDataFetchOption::ComputeSizes), [callbackAggregator = WTFMove(callbackAggregator)](auto entries) mutable {
+    if (websiteDataTypes.contains(WebsiteDataType::DiskCache)) {
+        fetchDiskCacheEntries(sessionID, fetchOptions, [callbackAggregator = WTFMove(callbackAggregator)](auto entries) mutable {
             callbackAggregator->m_websiteData.entries.appendVector(entries);
         });
     }
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to