WebResourceLoadStatisticsStore::callHasStorageAccessForFrameHandler() should take...
authorcdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 6 Jul 2018 19:33:06 +0000 (19:33 +0000)
committercdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 6 Jul 2018 19:33:06 +0000 (19:33 +0000)
https://bugs.webkit.org/show_bug.cgi?id=187392

Reviewed by Youenn Fablet.

* UIProcess/WebResourceLoadStatisticsStore.cpp:
(WebKit::WebResourceLoadStatisticsStore::callHasStorageAccessForFrameHandler):
* UIProcess/WebResourceLoadStatisticsStore.h:

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

Source/WebKit/ChangeLog
Source/WebKit/UIProcess/WebResourceLoadStatisticsStore.cpp
Source/WebKit/UIProcess/WebResourceLoadStatisticsStore.h

index 5dd9b1c..a30ac7b 100644 (file)
@@ -1,5 +1,16 @@
 2018-07-06  Chris Dumez  <cdumez@apple.com>
 
+        WebResourceLoadStatisticsStore::callHasStorageAccessForFrameHandler() should take in a CompletionHandler
+        https://bugs.webkit.org/show_bug.cgi?id=187392
+
+        Reviewed by Youenn Fablet.
+
+        * UIProcess/WebResourceLoadStatisticsStore.cpp:
+        (WebKit::WebResourceLoadStatisticsStore::callHasStorageAccessForFrameHandler):
+        * UIProcess/WebResourceLoadStatisticsStore.h:
+
+2018-07-06  Chris Dumez  <cdumez@apple.com>
+
         [iOS] Assertion failure in WebProcessProxy::allProcesses() (isMainThread())
         https://bugs.webkit.org/show_bug.cgi?id=187394
 
index fea65e3..7bd1e26 100644 (file)
@@ -231,7 +231,7 @@ void WebResourceLoadStatisticsStore::hasStorageAccess(String&& subFrameHost, Str
     });
 }
 
-void WebResourceLoadStatisticsStore::callHasStorageAccessForFrameHandler(const String& resourceDomain, const String& firstPartyDomain, uint64_t frameID, uint64_t pageID, Function<void(bool hasAccess)>&& callback)
+void WebResourceLoadStatisticsStore::callHasStorageAccessForFrameHandler(const String& resourceDomain, const String& firstPartyDomain, uint64_t frameID, uint64_t pageID, CompletionHandler<void(bool hasAccess)>&& callback)
 {
     ASSERT(RunLoop::isMain());
 
index 86d01e6..b929631 100644 (file)
@@ -140,7 +140,7 @@ public:
     void removeAllStorageAccess();
     void callUpdatePrevalentDomainsToPartitionOrBlockCookiesHandler(const Vector<String>& domainsToPartition, const Vector<String>& domainsToBlock, const Vector<String>& domainsToNeitherPartitionNorBlock, ShouldClearFirst, CompletionHandler<void()>&&);
     void callRemoveDomainsHandler(const Vector<String>& domains);
-    void callHasStorageAccessForFrameHandler(const String& resourceDomain, const String& firstPartyDomain, uint64_t frameID, uint64_t pageID, WTF::Function<void(bool)>&&);
+    void callHasStorageAccessForFrameHandler(const String& resourceDomain, const String& firstPartyDomain, uint64_t frameID, uint64_t pageID, CompletionHandler<void(bool)>&&);
 
 private:
     explicit WebResourceLoadStatisticsStore(WebsiteDataStore&);