Bug 1508996 - Change Response's statusText's default, r=valentin
☠☠ backed out by 7d429f711f4d ☠ ☠
authorAndrea Marchesini <amarchesini@mozilla.com>
Mon, 25 Feb 2019 16:54:11 +0000
changeset 460931 b591e1e39f8004f581311acd28e8e2645011594f
parent 460930 50e17e84f24d4fae03f4b499d7c76fd229d224ea
child 460932 da356c59e50db348e0a6ed391e7ab6fc3ef1606f
push id35613
push usernerli@mozilla.com
push dateTue, 26 Feb 2019 03:52:35 +0000
treeherdermozilla-central@faec87a80ed1 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersvalentin
bugs1508996
milestone67.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1508996 - Change Response's statusText's default, r=valentin Differential Revision: https://phabricator.services.mozilla.com/D21026
dom/tests/mochitest/fetch/test_response.js
dom/webidl/Response.webidl
testing/web-platform/meta/fetch/api/response/response-init-001.html.ini
--- a/dom/tests/mochitest/fetch/test_response.js
+++ b/dom/tests/mochitest/fetch/test_response.js
@@ -1,15 +1,15 @@
 function testDefaultCtor() {
   var res = new Response();
   is(res.type, "default", "Default Response type is default");
   ok(res.headers instanceof Headers, "Response should have non-null Headers object");
   is(res.url, "", "URL should be empty string");
   is(res.status, 200, "Default status is 200");
-  is(res.statusText, "OK", "Default statusText is OK");
+  is(res.statusText, "", "Default statusText is an empty string");
 }
 
 function testClone() {
   var orig = new Response("This is a body", {
               status: 404,
               statusText: "Not Found",
               headers: { "Content-Length": 5 },
             });
--- a/dom/webidl/Response.webidl
+++ b/dom/webidl/Response.webidl
@@ -39,13 +39,13 @@ Response implements Body;
 // See bug 1387483.
 partial interface Response {
   [GetterThrows, Func="mozilla::dom::DOMPrefs::javascript_options_streams"]
   readonly attribute ReadableStream? body;
 };
 
 dictionary ResponseInit {
   unsigned short status = 200;
-  ByteString statusText = "OK";
+  ByteString statusText = "";
   HeadersInit headers;
 };
 
 enum ResponseType { "basic", "cors", "default", "error", "opaque", "opaqueredirect" };
deleted file mode 100644
--- a/testing/web-platform/meta/fetch/api/response/response-init-001.html.ini
+++ /dev/null
@@ -1,4 +0,0 @@
-[response-init-001.html]
-  [Check default value for statusText attribute]
-    expected: FAIL
-