Changeset 229115 in webkit

Timestamp:

Feb 28, 2018 4:54:45 PM (6 years ago)

Author:

wilander@apple.com

Message:

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).

Location:

trunk/Source/WebKit

Files:

• ChangeLog (modified) (1 diff)
• Platform/classifier/ResourceLoadStatisticsClassifier.cpp (modified) (1 diff)

trunk/Source/WebKit/ChangeLog

@@ +1 @@
+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>
 

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

@@ -53 +53 @@
         && !subframeUnderTopFrameOriginsCount
         && !topFrameUniqueRedirectsToCount) {
-        ASSERT(currentPrevalence == Low);
         return Low;
     }