[CURL] ResourceError created with error information should have default type Type...
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 15 Feb 2017 02:50:58 +0000 (02:50 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 15 Feb 2017 02:50:58 +0000 (02:50 +0000)
https://bugs.webkit.org/show_bug.cgi?id=168345

Patch by Basuke Suzuki <Basuke.Suzuki@am.sony.com> on 2017-02-14
Reviewed by Alex Christensen.

ResourceError has separate implementaion for each platform
so that the interface should be same.
On CURL port, the constructor with error information has
different default value on `type` parameter. It is Type::Null but other implementaitons have Type::General.
This causes some ResourceError is created inconsistent.

* platform/network/curl/ResourceError.h:
(WebCore::ResourceError::ResourceError):

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

Source/WebCore/ChangeLog
Source/WebCore/platform/network/curl/ResourceError.h

index b49652e..424fc77 100644 (file)
@@ -1,3 +1,19 @@
+2017-02-14  Basuke Suzuki  <Basuke.Suzuki@am.sony.com>
+
+        [CURL] ResourceError created with error information should have default type Type::General
+        https://bugs.webkit.org/show_bug.cgi?id=168345
+
+        Reviewed by Alex Christensen.
+
+        ResourceError has separate implementaion for each platform
+        so that the interface should be same.
+        On CURL port, the constructor with error information has
+        different default value on `type` parameter. It is Type::Null but other implementaitons have Type::General.
+        This causes some ResourceError is created inconsistent. 
+
+        * platform/network/curl/ResourceError.h:
+        (WebCore::ResourceError::ResourceError):
+
 2017-02-14  Youenn Fablet  <youennf@gmail.com>
 
         [WebRTC] Add support for libwebrtc negotiation needed event
index e34f6eb..003f9b6 100644 (file)
@@ -46,7 +46,7 @@ public:
     {
     }
 
-    ResourceError(const String& domain, int errorCode, const URL& failingURL, const String& localizedDescription, Type type = Type::Null)
+    ResourceError(const String& domain, int errorCode, const URL& failingURL, const String& localizedDescription, Type type = Type::General)
         : ResourceErrorBase(domain, errorCode, failingURL, localizedDescription, type)
         , m_sslErrors(0)
     {