Backed out changeset 6ed95396bdb6 (bug 1362354) for failing web-platform-test /html/browsers/browsing-the-web/navigating-across-documents/010.html. r=backout
authorSebastian Hengst <archaeopteryx@coole-files.de>
Thu, 27 Jul 2017 12:48:45 +0200
changeset 422497 0739667c02050d60fec1f965307a8a85f8c40b8b
parent 422496 5f396e8e22faa2852a29f3e1bfda22018f5aa3ca
child 422498 eb898389180abf04d7bb0c603a9e097671c3759e
push id1517
push userjlorenzo@mozilla.com
push dateThu, 14 Sep 2017 16:50:54 +0000
treeherdermozilla-release@3b41fd564418 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbackout
bugs1362354
milestone56.0a1
backs out6ed95396bdb623610e88b4d59ed267d513ae2886
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 6ed95396bdb6 (bug 1362354) for failing web-platform-test /html/browsers/browsing-the-web/navigating-across-documents/010.html. r=backout
dom/xhr/XMLHttpRequestMainThread.cpp
testing/web-platform/meta/XMLHttpRequest/abort-after-stop.htm.ini
--- a/dom/xhr/XMLHttpRequestMainThread.cpp
+++ b/dom/xhr/XMLHttpRequestMainThread.cpp
@@ -2232,24 +2232,16 @@ XMLHttpRequestMainThread::OnStopRequest(
 {
   AUTO_PROFILER_LABEL("XMLHttpRequestMainThread::OnStopRequest", NETWORK);
 
   if (request != mChannel) {
     // Can this still happen?
     return NS_OK;
   }
 
-  if (status == NS_BINDING_ABORTED &&
-      ((mState == State::opened && mFlagSend) ||
-        mState == State::headers_received ||
-        mState == State::loading)) {
-    // Make sure we send event abort if termination reason is end-user abort.
-    Abort();
-  }
-
   mWaitingForOnStopRequest = false;
 
   if (mRequestObserver) {
     NS_ASSERTION(mFirstStartRequestSeen, "Inconsistent state!");
     mFirstStartRequestSeen = false;
     mRequestObserver->OnStopRequest(request, ctxt, status);
   }
 
--- a/testing/web-platform/meta/XMLHttpRequest/abort-after-stop.htm.ini
+++ b/testing/web-platform/meta/XMLHttpRequest/abort-after-stop.htm.ini
@@ -1,4 +1,6 @@
 [abort-after-stop.htm]
   type: testharness
   expected: TIMEOUT
+  [XMLHttpRequest: abort event should fire when stop() method is used]
+    expected: FAIL