Title: [179593] trunk/Source/WebCore
Revision
179593
Author
[email protected]
Date
2015-02-03 22:25:03 -0800 (Tue, 03 Feb 2015)

Log Message

Regression(r179584): Assertion hit in toResourceLoadPriority() on Yosemite
https://bugs.webkit.org/show_bug.cgi?id=141230

Reviewed by Alexey Proskuryakov.

Handle -1 priority value again in toResourceLoadPriority() as it seems to
be returned by CFNetwork on some configurations.

No new tests, already covered by existing tests.

* platform/network/cf/ResourceRequestCFNet.h:
(WebCore::toResourceLoadPriority):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (179592 => 179593)


--- trunk/Source/WebCore/ChangeLog	2015-02-04 06:16:37 UTC (rev 179592)
+++ trunk/Source/WebCore/ChangeLog	2015-02-04 06:25:03 UTC (rev 179593)
@@ -1,5 +1,20 @@
 2015-02-03  Chris Dumez  <[email protected]>
 
+        Regression(r179584): Assertion hit in toResourceLoadPriority() on Yosemite
+        https://bugs.webkit.org/show_bug.cgi?id=141230
+
+        Reviewed by Alexey Proskuryakov.
+
+        Handle -1 priority value again in toResourceLoadPriority() as it seems to
+        be returned by CFNetwork on some configurations.
+
+        No new tests, already covered by existing tests.
+
+        * platform/network/cf/ResourceRequestCFNet.h:
+        (WebCore::toResourceLoadPriority):
+
+2015-02-03  Chris Dumez  <[email protected]>
+
         Drop ResourceLoadPriorityUnresolved resource load priority and use Optional<> instead
         https://bugs.webkit.org/show_bug.cgi?id=141186
 

Modified: trunk/Source/WebCore/platform/network/cf/ResourceRequestCFNet.h (179592 => 179593)


--- trunk/Source/WebCore/platform/network/cf/ResourceRequestCFNet.h	2015-02-04 06:16:37 UTC (rev 179592)
+++ trunk/Source/WebCore/platform/network/cf/ResourceRequestCFNet.h	2015-02-04 06:25:03 UTC (rev 179593)
@@ -44,6 +44,7 @@
 inline ResourceLoadPriority toResourceLoadPriority(int priority)
 {
     switch (priority) {
+    case -1:
     case 0:
         return ResourceLoadPriorityVeryLow;
     case 1:
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to