Remove assertion from ResourceLoadStatisticsClassifier::calculateResourcePrevalence...
authorwilander@apple.com <wilander@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 1 Mar 2018 00:54:45 +0000 (00:54 +0000)
committerwilander@apple.com <wilander@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 1 Mar 2018 00:54:45 +0000 (00:54 +0000)
https://bugs.webkit.org/show_bug.cgi?id=183233

Unreviewed removal of an assertion that's causing layout test crashes.

* Platform/classifier/ResourceLoadStatisticsClassifier.cpp:
(WebKit::ResourceLoadStatisticsClassifier::calculateResourcePrevalence):
    Removed ASSERT(currentPrevalence == Low).

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

Source/WebKit/ChangeLog
Source/WebKit/Platform/classifier/ResourceLoadStatisticsClassifier.cpp

index 92f3413..61fc8b7 100644 (file)
@@ -1,5 +1,16 @@
 2018-02-28  John Wilander  <wilander@apple.com>
 
+        Remove assertion from ResourceLoadStatisticsClassifier::calculateResourcePrevalence() which causes crashes when domains are explicitly set as prevalent without the associated statistics
+        https://bugs.webkit.org/show_bug.cgi?id=183233
+
+        Unreviewed removal of an assertion that's causing layout test crashes.
+
+        * Platform/classifier/ResourceLoadStatisticsClassifier.cpp:
+        (WebKit::ResourceLoadStatisticsClassifier::calculateResourcePrevalence):
+            Removed ASSERT(currentPrevalence == Low).
+
+2018-02-28  John Wilander  <wilander@apple.com>
+
         Add a second tier of prevalence to facilitate telemetry on very prevalent domains
         https://bugs.webkit.org/show_bug.cgi?id=183218
         <rdar://problem/37992388>
index affa733..bdbfdfc 100644 (file)
@@ -52,7 +52,6 @@ ResourceLoadPrevalence ResourceLoadStatisticsClassifier::calculateResourcePreval
         && !subresourceUniqueRedirectsToCount
         && !subframeUnderTopFrameOriginsCount
         && !topFrameUniqueRedirectsToCount) {
-        ASSERT(currentPrevalence == Low);
         return Low;
     }