Backed out changeset ec067b17f7d3 (bug 1453612) for failures on /fetch/api/response/response-static-redirect.html. CLOSED TREE
authorCsoregi Natalia <ncsoregi@mozilla.com>
Mon, 15 Oct 2018 00:21:11 +0300
changeset 499640 00351b000d8835beb11192c939f3953699799940
parent 499639 6213dd2a20f2c544722ff18d62863cbf6e031b8e
child 499641 408c0e44c2d93fae7df447e07b615ab833fdd28d
push id1864
push userffxbld-merge
push dateMon, 03 Dec 2018 15:51:40 +0000
treeherdermozilla-release@f040763d99ad [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs1453612
milestone64.0a1
backs outec067b17f7d32f9080ed97d8a51414d0a37acf8a
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
Backed out changeset ec067b17f7d3 (bug 1453612) for failures on /fetch/api/response/response-static-redirect.html. CLOSED TREE
dom/fetch/Response.cpp
dom/tests/mochitest/fetch/test_response.js
--- a/dom/fetch/Response.cpp
+++ b/dom/fetch/Response.cpp
@@ -135,17 +135,16 @@ Response::Redirect(const GlobalObject& a
   if (aStatus != 301 && aStatus != 302 && aStatus != 303 && aStatus != 307 && aStatus != 308) {
     aRv.ThrowRangeError<MSG_INVALID_REDIRECT_STATUSCODE_ERROR>();
     return nullptr;
   }
 
   Optional<Nullable<fetch::ResponseBodyInit>> body;
   ResponseInit init;
   init.mStatus = aStatus;
-  init.mStatusText.AssignASCII("");
   RefPtr<Response> r = Response::Constructor(aGlobal, body, init, aRv);
   if (NS_WARN_IF(aRv.Failed())) {
     return nullptr;
   }
 
   r->GetInternalHeaders()->Set(NS_LITERAL_CSTRING("Location"),
                                NS_ConvertUTF16toUTF8(parsedURL), aRv);
   if (NS_WARN_IF(aRv.Failed())) {
--- a/dom/tests/mochitest/fetch/test_response.js
+++ b/dom/tests/mochitest/fetch/test_response.js
@@ -100,17 +100,16 @@ function testError() {
   } catch(e) {
     ok(true, "Error response should have immutable headers");
   }
 }
 
 function testRedirect() {
   var res = Response.redirect("./redirect.response");
   is(res.status, 302, "Default redirect has status code 302");
-  is(res.statusText, "", "Default redirect has status text empty");
   var h = res.headers.get("location");
   ok(h === (new URL("./redirect.response", self.location.href)).href, "Location header should be correct absolute URL");
   try {
     res.headers.set("someheader", "not allowed");
     ok(false, "Redirects should have immutable headers");
   } catch(e) {
     ok(true, "Redirects should have immutable headers");
   }