Title: [224638] trunk/Source/WebCore
Revision
224638
Author
commit-qu...@webkit.org
Date
2017-11-09 11:00:59 -0800 (Thu, 09 Nov 2017)

Log Message

ResourceErrorBase::setType should not ASSERT if it is a no op
https://bugs.webkit.org/show_bug.cgi?id=179490

Patch by Youenn Fablet <you...@apple.com> on 2017-11-09
Reviewed by Chris Dumez.

No change of behavior.

* platform/network/ResourceErrorBase.cpp:
(WebCore::ResourceErrorBase::setType):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (224637 => 224638)


--- trunk/Source/WebCore/ChangeLog	2017-11-09 18:50:03 UTC (rev 224637)
+++ trunk/Source/WebCore/ChangeLog	2017-11-09 19:00:59 UTC (rev 224638)
@@ -1,3 +1,15 @@
+2017-11-09  Youenn Fablet  <you...@apple.com>
+
+        ResourceErrorBase::setType should not ASSERT if it is a no op
+        https://bugs.webkit.org/show_bug.cgi?id=179490
+
+        Reviewed by Chris Dumez.
+
+        No change of behavior.
+
+        * platform/network/ResourceErrorBase.cpp:
+        (WebCore::ResourceErrorBase::setType):
+
 2017-11-09  Zalan Bujtas  <za...@apple.com>
 
         [LayoutState cleanup] Remove renderer data members from state maintainers.

Modified: trunk/Source/WebCore/platform/network/ResourceErrorBase.cpp (224637 => 224638)


--- trunk/Source/WebCore/platform/network/ResourceErrorBase.cpp	2017-11-09 18:50:03 UTC (rev 224637)
+++ trunk/Source/WebCore/platform/network/ResourceErrorBase.cpp	2017-11-09 19:00:59 UTC (rev 224638)
@@ -60,7 +60,7 @@
 void ResourceErrorBase::setType(Type type)
 {
     // setType should only be used to specialize the error type.
-    ASSERT(m_type == Type::General || m_type == Type::Null || (m_type == Type::Cancellation && type == Type::AccessControl));
+    ASSERT(m_type == type || m_type == Type::General || m_type == Type::Null || (m_type == Type::Cancellation && type == Type::AccessControl));
     m_type = type;
 }
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to