Change Response's statusText's default
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 7 Oct 2019 20:41:59 +0000 (20:41 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 7 Oct 2019 20:41:59 +0000 (20:41 +0000)
https://bugs.webkit.org/show_bug.cgi?id=191879

Patch by Rob Buis <rbuis@igalia.com> on 2019-10-07
Reviewed by Youenn Fablet.

LayoutTests/imported/w3c:

Update improved test results.

* web-platform-tests/fetch/api/response/response-clone-expected.txt:
* web-platform-tests/fetch/api/response/response-init-001-expected.txt:

Source/WebCore:

Change Response's statusText's default to "".

Tests: web-platform-tests/fetch/api/response/response-clone.html
       web-platform-tests/fetch/api/response/response-init-001.html

* Modules/fetch/FetchResponse.h:
* Modules/fetch/FetchResponse.idl:

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

LayoutTests/imported/w3c/ChangeLog
LayoutTests/imported/w3c/web-platform-tests/fetch/api/response/response-clone-expected.txt
LayoutTests/imported/w3c/web-platform-tests/fetch/api/response/response-init-001-expected.txt
Source/WebCore/ChangeLog
Source/WebCore/Modules/fetch/FetchResponse.h
Source/WebCore/Modules/fetch/FetchResponse.idl

index 98d5db2..70fb1b8 100644 (file)
@@ -1,3 +1,15 @@
+2019-10-07  Rob Buis  <rbuis@igalia.com>
+
+        Change Response's statusText's default
+        https://bugs.webkit.org/show_bug.cgi?id=191879
+
+        Reviewed by Youenn Fablet.
+
+        Update improved test results.
+
+        * web-platform-tests/fetch/api/response/response-clone-expected.txt:
+        * web-platform-tests/fetch/api/response/response-init-001-expected.txt:
+
 2019-10-07  Alexey Shvayka  <shvaikalesh@gmail.com>
 
         Sync web-platform-tests/css/css-overflow from upstream
index 979792b..d28f3a6 100644 (file)
@@ -1,5 +1,5 @@
 
-FAIL Check Response's clone with default values, without body assert_equals: Expect default response.statusText is  expected "" but got "OK"
+PASS Check Response's clone with default values, without body 
 PASS Check Response's clone has the expected attribute values 
 PASS Check orginal response's body after cloning 
 PASS Check cloned response's body 
index 52485bd..eea9794 100644 (file)
@@ -3,7 +3,7 @@ PASS Check default value for type attribute
 PASS Check default value for url attribute 
 PASS Check default value for ok attribute 
 PASS Check default value for status attribute 
-FAIL Check default value for statusText attribute assert_equals: Expect default response.statusText is  expected "" but got "OK"
+PASS Check default value for statusText attribute 
 PASS Check default value for body attribute 
 PASS Check status init values and associated getter 
 PASS Check statusText init values and associated getter 
index e857bbc..12cfaed 100644 (file)
@@ -1,3 +1,18 @@
+2019-10-07  Rob Buis  <rbuis@igalia.com>
+
+        Change Response's statusText's default
+        https://bugs.webkit.org/show_bug.cgi?id=191879
+
+        Reviewed by Youenn Fablet.
+
+        Change Response's statusText's default to "".
+
+        Tests: web-platform-tests/fetch/api/response/response-clone.html
+               web-platform-tests/fetch/api/response/response-init-001.html
+
+        * Modules/fetch/FetchResponse.h:
+        * Modules/fetch/FetchResponse.idl:
+
 2019-10-07  Chris Dumez  <cdumez@apple.com>
 
         DOMCache should not prevent pages from entering the back/forward cache
index 5c83a37..25f10a1 100644 (file)
@@ -53,7 +53,7 @@ public:
 
     struct Init {
         unsigned short status { 200 };
-        String statusText { "OK"_s };
+        String statusText;
         Optional<FetchHeaders::Init> headers;
     };
 
index 335b78f..43989c3 100644 (file)
@@ -33,7 +33,7 @@ enum FetchResponseType { "basic", "cors", "default", "error", "opaque", "opaquer
 
 dictionary FetchResponseInit {
     unsigned short status = 200;
-    ByteString statusText = "OK";
+    ByteString statusText = "";
     HeadersInit headers;
 };