Attachment 369559 Details for Bug 196307 – Patch

[patch] Patch

bug-196307-20190510100735.patch (text/plain), 8.76 KB, created by Sihui Liu on 2019-05-10 10:07:35 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Sihui Liu

Created: 2019-05-10 10:07:35 PDT

Size: 8.76 KB

patch

obsolete

>Subversion Revision: 245144
>diff --git a/Source/WebKit/ChangeLog b/Source/WebKit/ChangeLog
>index 8e056e6bb57702068e8d6bdc4d594f18d7cbbffd..a9baec5779af846db366e90f6021e83554eecb52 100644
>--- a/Source/WebKit/ChangeLog
>+++ b/Source/WebKit/ChangeLog
>@@ -1,3 +1,22 @@
>+2019-05-10  Sihui Liu  <sihui_liu@apple.com>
>+
>+        [ Mac WK2 iOS Sim] Layout Test http/tests/resourceLoadStatistics/website-data-removal-for-site-navigated-to-with-link-decoration.html is a flaky failure
>+        https://bugs.webkit.org/show_bug.cgi?id=196307
>+        <rdar://problem/49345360>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Delay dumping statistics if there is data being removed.
>+
>+        * NetworkProcess/Classifier/ResourceLoadStatisticsStore.cpp:
>+        (WebKit::ResourceLoadStatisticsStore::removeDataRecords):
>+        * NetworkProcess/Classifier/ResourceLoadStatisticsStore.h:
>+        (WebKit::ResourceLoadStatisticsStore::dataRecordsBeingRemoved const):
>+        * NetworkProcess/Classifier/WebResourceLoadStatisticsStore.cpp:
>+        (WebKit::WebResourceLoadStatisticsStore::dumpResourceLoadStatistics):
>+        (WebKit::WebResourceLoadStatisticsStore::tryDumpResourceLoadStatistics):
>+        * NetworkProcess/Classifier/WebResourceLoadStatisticsStore.h:
>+
> 2019-05-09  Daniel Bates  <dabates@apple.com>
> 
>         [iOS] Unable to commit search on MSN.com, qq.com, or sina.com.cn using enter key (hardware or software keyboard)
>diff --git a/Source/WebKit/NetworkProcess/Classifier/ResourceLoadStatisticsStore.cpp b/Source/WebKit/NetworkProcess/Classifier/ResourceLoadStatisticsStore.cpp
>index 2291fd3db30ddd8af53998e0597705221cdaadd5..b7d25dec302905be36c511834e549b3db29ba156 100644
>--- a/Source/WebKit/NetworkProcess/Classifier/ResourceLoadStatisticsStore.cpp
>+++ b/Source/WebKit/NetworkProcess/Classifier/ResourceLoadStatisticsStore.cpp
>@@ -221,6 +221,7 @@ void ResourceLoadStatisticsStore::removeDataRecords(CompletionHandler<void()>&&
>                 }
>                 weakThis->incrementRecordsDeletedCountForDomains(WTFMove(domainsWithDeletedWebsiteData));
>                 weakThis->setDataRecordsBeingRemoved(false);
>+                weakThis->m_store.tryDumpResourceLoadStatistics();
>                 completionHandler();
> #if !RELEASE_LOG_DISABLED
>                 RELEASE_LOG_INFO_IF(weakThis->m_debugLoggingEnabled, ResourceLoadStatisticsDebug, "Done removing data records.");
>diff --git a/Source/WebKit/NetworkProcess/Classifier/ResourceLoadStatisticsStore.h b/Source/WebKit/NetworkProcess/Classifier/ResourceLoadStatisticsStore.h
>index 220b3c6bf655d2f06b976c141c37d7cce9260a56..0b5775bbb779cc1f1b945621b9a0552484c096c5 100644
>--- a/Source/WebKit/NetworkProcess/Classifier/ResourceLoadStatisticsStore.h
>+++ b/Source/WebKit/NetworkProcess/Classifier/ResourceLoadStatisticsStore.h
>@@ -185,6 +185,8 @@ public:
>     virtual bool isMemoryStore() const { return false; }
>     virtual bool isDatabaseStore()const { return false; }
> 
>+    bool dataRecordsBeingRemoved() const { return m_dataRecordsBeingRemoved; }
>+
> protected:
>     static unsigned computeImportance(const WebCore::ResourceLoadStatistics&);
>     static Vector<OperatingDate> mergeOperatingDates(const Vector<OperatingDate>& existingDates, Vector<OperatingDate>&& newDates);
>diff --git a/Source/WebKit/NetworkProcess/Classifier/WebResourceLoadStatisticsStore.cpp b/Source/WebKit/NetworkProcess/Classifier/WebResourceLoadStatisticsStore.cpp
>index 252fe1ca0316310f3a4df1fe8b92d6841c5c9697..2ace79e77c37f10a20b3c61bdaf33fa916554fd2 100644
>--- a/Source/WebKit/NetworkProcess/Classifier/WebResourceLoadStatisticsStore.cpp
>+++ b/Source/WebKit/NetworkProcess/Classifier/WebResourceLoadStatisticsStore.cpp
>@@ -615,10 +615,23 @@ void WebResourceLoadStatisticsStore::dumpResourceLoadStatistics(CompletionHandle
>     ASSERT(RunLoop::isMain());
> 
>     postTask([this, completionHandler = WTFMove(completionHandler)]() mutable {
>-        String result = m_statisticsStore ? m_statisticsStore->dumpResourceLoadStatistics() : emptyString();
>-        postTaskReply([result = result.isolatedCopy(), completionHandler = WTFMove(completionHandler)]() mutable {
>-            completionHandler(result);
>-        });
>+        m_dumpResourceLoadStatisticsFunction = WTFMove(completionHandler);
>+        if (m_statisticsStore && m_statisticsStore->dataRecordsBeingRemoved())
>+            return;
>+        tryDumpResourceLoadStatistics();
>+    });
>+}
>+
>+void WebResourceLoadStatisticsStore::tryDumpResourceLoadStatistics()
>+{
>+    ASSERT(!RunLoop::isMain());
>+
>+    if (!m_dumpResourceLoadStatisticsFunction)
>+        return;
>+
>+    String result = m_statisticsStore ? m_statisticsStore->dumpResourceLoadStatistics() : emptyString();
>+    postTaskReply([result = result.isolatedCopy(), completionHandler = WTFMove(m_dumpResourceLoadStatisticsFunction)]() mutable {
>+        completionHandler(result);
>     });
> }
> 
>diff --git a/Source/WebKit/NetworkProcess/Classifier/WebResourceLoadStatisticsStore.h b/Source/WebKit/NetworkProcess/Classifier/WebResourceLoadStatisticsStore.h
>index f569dc36431a4dddc4b7ffb441b340e059eddd1d..4468f9903fbef089757052d11523ddcd56055fab 100644
>--- a/Source/WebKit/NetworkProcess/Classifier/WebResourceLoadStatisticsStore.h
>+++ b/Source/WebKit/NetworkProcess/Classifier/WebResourceLoadStatisticsStore.h
>@@ -127,6 +127,7 @@ public:
>     void setPrevalentResource(const RegistrableDomain&, CompletionHandler<void()>&&);
>     void setVeryPrevalentResource(const RegistrableDomain&, CompletionHandler<void()>&&);
>     void dumpResourceLoadStatistics(CompletionHandler<void(String)>&&);
>+    void tryDumpResourceLoadStatistics();
>     void isPrevalentResource(const RegistrableDomain&, CompletionHandler<void(bool)>&&);
>     void isVeryPrevalentResource(const RegistrableDomain&, CompletionHandler<void(bool)>&&);
>     void isRegisteredAsSubresourceUnder(const SubResourceDomain&, const TopFrameDomain&, CompletionHandler<void(bool)>&&);
>@@ -205,6 +206,8 @@ private:
>     bool m_hasScheduledProcessStats { false };
> 
>     bool m_firstNetworkProcessCreated { false };
>+    
>+    CompletionHandler<void(String)> m_dumpResourceLoadStatisticsFunction;
> };
> 
> } // namespace WebKit
>diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
>index b303f73a11f4e5e0b4c7ac4d5298ffd22fbcd855..7d1db959016c779a03c29fcf8d37d7a179b52c8a 100644
>--- a/LayoutTests/ChangeLog
>+++ b/LayoutTests/ChangeLog
>@@ -1,3 +1,14 @@
>+2019-05-10  Sihui Liu  <sihui_liu@apple.com>
>+
>+        [ Mac WK2 iOS Sim] Layout Test http/tests/resourceLoadStatistics/website-data-removal-for-site-navigated-to-with-link-decoration.html is a flaky failure
>+        https://bugs.webkit.org/show_bug.cgi?id=196307
>+        <rdar://problem/49345360>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * platform/ios-simulator-wk2/TestExpectations:
>+        * platform/mac-wk2/TestExpectations:
>+
> 2019-05-09  Daniel Bates  <dabates@apple.com>
> 
>         [iOS] Unable to commit search on MSN.com, qq.com, or sina.com.cn using enter key (hardware or software keyboard)
>diff --git a/LayoutTests/platform/ios-simulator-wk2/TestExpectations b/LayoutTests/platform/ios-simulator-wk2/TestExpectations
>index 594ce74d8991c92af31b2fa56704f87c6fe87ac3..e1e84aca490160b501bd39f6cea7b5672e11fd40 100644
>--- a/LayoutTests/platform/ios-simulator-wk2/TestExpectations
>+++ b/LayoutTests/platform/ios-simulator-wk2/TestExpectations
>@@ -69,8 +69,6 @@ webkit.org/b/196298 fast/viewport/ios/use-minimum-device-width-for-page-without-
> 
> webkit.org/b/162975 http/tests/cache/disk-cache/memory-cache-revalidation-updates-disk-cache.html [ Pass Failure ]
> 
>-webkit.org/b/196307 http/tests/resourceLoadStatistics/website-data-removal-for-site-navigated-to-with-link-decoration.html [ Pass Failure ]
>-
> webkit.org/b/196112 imported/w3c/web-platform-tests/mathml/relations/css-styling/mathvariant-bold.html [ Pass ImageOnlyFailure ]
> webkit.org/b/196112 imported/w3c/web-platform-tests/mathml/relations/css-styling/mathvariant-double-struck.html [ Pass ImageOnlyFailure ]
> webkit.org/b/196112 imported/w3c/web-platform-tests/mathml/relations/css-styling/mathvariant-italic.html [ Pass ImageOnlyFailure ]
>diff --git a/LayoutTests/platform/mac-wk2/TestExpectations b/LayoutTests/platform/mac-wk2/TestExpectations
>index 4241ad914561da5aaab31afc2bab0d4836a6040e..bf537564eacb08f541bc3ad6df81e1fd666baeae 100644
>--- a/LayoutTests/platform/mac-wk2/TestExpectations
>+++ b/LayoutTests/platform/mac-wk2/TestExpectations
>@@ -914,8 +914,6 @@ webkit.org/b/194253 scrollingcoordinator/scrolling-tree/fixed-inside-frame.html
> 
> webkit.org/b/194916 fast/mediastream/MediaStream-video-element.html [ Pass Failure ]
> 
>-webkit.org/b/196307 http/tests/resourceLoadStatistics/website-data-removal-for-site-navigated-to-with-link-decoration.html [ Pass Failure ]
>-
> webkit.org/b/196376 storage/domstorage/localstorage/private-browsing-affects-storage.html [ Pass Failure ]
> 
> webkit.org/b/194780 http/wpt/webauthn/public-key-credential-create-success-hid.https.html [ Pass Failure ]

Actions: View | Formatted Diff | Diff

Attachments on bug 196307: 369498 | 369499 | 369559 | 370256