Attachment 349986 Details for Bug 189684 – Patch

[patch] Patch

bug-189684-20180917181759.patch (text/plain), 4.77 KB, created by Woodrow Wang on 2018-09-17 18:18:00 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Woodrow Wang

Created: 2018-09-17 18:18:00 PDT

Size: 4.77 KB

patch

obsolete

>Subversion Revision: 236097
>diff --git a/Source/WebKit/ChangeLog b/Source/WebKit/ChangeLog
>index dbbe641b0c76d3de8a3da414b1aa375ec644427a..5cf62506444f25d4231f87769f878a65d836b7ad 100644
>--- a/Source/WebKit/ChangeLog
>+++ b/Source/WebKit/ChangeLog
>@@ -1,3 +1,21 @@
>+2018-09-17  Woodrow Wang  <woodrow_wang@apple.com>
>+
>+        Clear persistent storage between tests for resourceLoadStatistics
>+        https://bugs.webkit.org/show_bug.cgi?id=189684
>+        <rdar://problem/44540099>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * UIProcess/API/C/WKWebsiteDataStoreRef.cpp:
>+        (WKWebsiteDataStoreStatisticsResetToConsistentState):
>+        * UIProcess/WebResourceLoadStatisticsStore.cpp:
>+        (WebKit::WebResourceLoadStatisticsStore::scheduleClearInMemoryAndPersistent):
>+
>+        Remove function only called in testing for resetting statistics to consistent
>+        state. 
>+        
>+        (WebKit::WebResourceLoadStatisticsStore::scheduleClearInMemory): Deleted.
>+
> 2018-09-14  Woodrow Wang  <woodrow_wang@apple.com>
> 
>         Clear pending resource load statistics' writes after tests
>diff --git a/Source/WebKit/UIProcess/API/C/WKWebsiteDataStoreRef.cpp b/Source/WebKit/UIProcess/API/C/WKWebsiteDataStoreRef.cpp
>index 7f23e90ff7c6e2ce4d54792c64a0b9623f2efe83..6ef3bf6560358dcd77a2f542f436b463659bb441 100644
>--- a/Source/WebKit/UIProcess/API/C/WKWebsiteDataStoreRef.cpp
>+++ b/Source/WebKit/UIProcess/API/C/WKWebsiteDataStoreRef.cpp
>@@ -476,7 +476,7 @@ void WKWebsiteDataStoreStatisticsResetToConsistentState(WKWebsiteDataStoreRef da
>         return;
> 
>     statisticsStore->resetParametersToDefaultValues([callbackAggregator = callbackAggregator.copyRef()] { });
>-    statisticsStore->scheduleClearInMemory([callbackAggregator = callbackAggregator.copyRef()] { });
>+    statisticsStore->scheduleClearInMemoryAndPersistent(WebKit::WebResourceLoadStatisticsStore::ShouldGrandfather::No, [callbackAggregator = callbackAggregator.copyRef()] { });
> }
> 
> void WKWebsiteDataStoreRemoveAllFetchCaches(WKWebsiteDataStoreRef dataStoreRef, void* context, WKWebsiteDataStoreRemoveFetchCacheRemovalFunction callback)
>diff --git a/Source/WebKit/UIProcess/WebResourceLoadStatisticsStore.cpp b/Source/WebKit/UIProcess/WebResourceLoadStatisticsStore.cpp
>index 6c5089afeb5de3f4859ce14d0631ebca07240a4d..ab2111e60480fbec61ab28294ea8fa9a77607704 100644
>--- a/Source/WebKit/UIProcess/WebResourceLoadStatisticsStore.cpp
>+++ b/Source/WebKit/UIProcess/WebResourceLoadStatisticsStore.cpp
>@@ -38,6 +38,7 @@
> #include "WebsiteDataFetchOption.h"
> #include "WebsiteDataStore.h"
> #include <WebCore/ResourceLoadStatistics.h>
>+#include <wtf/CallbackAggregator.h>
> #include <wtf/CrossThreadCopier.h>
> #include <wtf/NeverDestroyed.h>
> #include <wtf/threads/BinarySemaphore.h>
>@@ -759,40 +760,25 @@ void WebResourceLoadStatisticsStore::scheduleCookieBlockingStateReset()
> }
> #endif
> 
>-void WebResourceLoadStatisticsStore::scheduleClearInMemory(CompletionHandler<void()>&& completionHandler)
>-{
>-    ASSERT(RunLoop::isMain());
>-    postTask([this, completionHandler = WTFMove(completionHandler)]() mutable {
>-
>-        CompletionHandler<void()> callCompletionHandlerOnMainThread = [completionHandler = WTFMove(completionHandler)]() mutable {
>-            postTaskReply(WTFMove(completionHandler));
>-        };
>-
>-        if (!m_memoryStore) {
>-            callCompletionHandlerOnMainThread();
>-            return;
>-        }
>-
>-        m_memoryStore->clear(WTFMove(callCompletionHandlerOnMainThread));
>-    });
>-}
>-
> void WebResourceLoadStatisticsStore::scheduleClearInMemoryAndPersistent(ShouldGrandfather shouldGrandfather, CompletionHandler<void()>&& completionHandler)
> {
>     ASSERT(RunLoop::isMain());
>     postTask([this, shouldGrandfather, completionHandler = WTFMove(completionHandler)] () mutable {
>-        if (m_memoryStore)
>-            m_memoryStore->clear([] { });
>         if (m_persistentStorage)
>             m_persistentStorage->clear();
>-        
>+
>         CompletionHandler<void()> callCompletionHandlerOnMainThread = [completionHandler = WTFMove(completionHandler)]() mutable {
>             postTaskReply(WTFMove(completionHandler));
>         };
> 
>-        if (shouldGrandfather == ShouldGrandfather::Yes && m_memoryStore)
>-            m_memoryStore->grandfatherExistingWebsiteData(WTFMove(callCompletionHandlerOnMainThread));
>-        else
>+        if (m_memoryStore) {
>+            m_memoryStore->clear([this, shouldGrandfather, callCompletionHandlerOnMainThread = WTFMove(callCompletionHandlerOnMainThread)] () mutable {
>+                if (shouldGrandfather == ShouldGrandfather::Yes)
>+                    m_memoryStore->grandfatherExistingWebsiteData(WTFMove(callCompletionHandlerOnMainThread));
>+                else
>+                    callCompletionHandlerOnMainThread();
>+            });
>+        } else
>             callCompletionHandlerOnMainThread();
>     });
> }

Flags:

cdumez: review+
cdumez: commit-queue-

Actions: View | Formatted Diff | Diff

Attachments on bug 189684: 349977 | 349986 | 350032 | 350059 | 350082 | 350130 | 350147 | 350151 | 350279