[WK2] Resource load statistics code is spamming the WebProcess with a test-only message
authorbfulgham@apple.com <bfulgham@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 30 Jun 2017 22:31:54 +0000 (22:31 +0000)
committerbfulgham@apple.com <bfulgham@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 30 Jun 2017 22:31:54 +0000 (22:31 +0000)
https://bugs.webkit.org/show_bug.cgi?id=174048
<rdar://problem/33062376>

Patch by John Wilander <wilander@apple.com> on 2017-06-30
Reviewed by Brent Fulgham.

Fix the telemetry data message send so that it is not generated during normal operations.
This is only meant to be used by WebKitTestRunner.

* UIProcess/WebResourceLoadStatisticsTelemetry.cpp:
(WebKit::WebResourceLoadStatisticsTelemetry::calculateAndSubmit):

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

Source/WebKit2/ChangeLog
Source/WebKit2/UIProcess/WebResourceLoadStatisticsTelemetry.cpp

index e00b40a..0013b34 100644 (file)
@@ -1,3 +1,17 @@
+2017-06-30  John Wilander  <wilander@apple.com>
+
+        [WK2] Resource load statistics code is spamming the WebProcess with a test-only message 
+        https://bugs.webkit.org/show_bug.cgi?id=174048
+        <rdar://problem/33062376>
+
+        Reviewed by Brent Fulgham.
+
+        Fix the telemetry data message send so that it is not generated during normal operations.
+        This is only meant to be used by WebKitTestRunner.
+
+        * UIProcess/WebResourceLoadStatisticsTelemetry.cpp:
+        (WebKit::WebResourceLoadStatisticsTelemetry::calculateAndSubmit): 
+
 2017-06-30  Daniel Bates  <dabates@apple.com>
 
         Attempt to fix the WPE build following <https://trac.webkit.org/changeset/219013>
index 5ec46f0..d13aea2 100644 (file)
@@ -189,7 +189,7 @@ void WebResourceLoadStatisticsTelemetry::calculateAndSubmit(const ResourceLoadSt
     ASSERT(!RunLoop::isMain());
     
     auto sortedPrevalentResources = resourceLoadStatisticsStore.sortedPrevalentResourceTelemetry();
-    if (sortedPrevalentResources.size() < minimumPrevalentResourcesForTelemetry) {
+    if (notifyPagesWhenTelemetryWasCaptured && sortedPrevalentResources.size() < minimumPrevalentResourcesForTelemetry) {
         notifyPages(0, 0, 0);
         return;
     }