Backed out changeset b85f5ec21392 (bug 1014466) for Android 2.3 Opt M-8 test failures on a CLOSED TREE
authorCarsten "Tomcat" Book <cbook@mozilla.com>
Fri, 06 Jun 2014 10:21:36 +0200
changeset 207373 0c006b5d36c852c91e5b7c562405f17c5f61252e
parent 207372 701627f5212db2c2fd8b4dda6fae088f2f02931c
child 207374 701a232171832309d15bb311affa5a4c54579f65
push id494
push userraliiev@mozilla.com
push dateMon, 25 Aug 2014 18:42:16 +0000
treeherdermozilla-release@a3cc3e46b571 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs1014466
milestone32.0a1
backs outb85f5ec213926ad06398d2c59cf199f8373e16f1
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 b85f5ec21392 (bug 1014466) for Android 2.3 Opt M-8 test failures on a CLOSED TREE
dom/workers/XMLHttpRequest.cpp
--- a/dom/workers/XMLHttpRequest.cpp
+++ b/dom/workers/XMLHttpRequest.cpp
@@ -1695,22 +1695,17 @@ XMLHttpRequest::MaybePin(ErrorResult& aR
 }
 
 void
 XMLHttpRequest::MaybeDispatchPrematureAbortEvents(ErrorResult& aRv)
 {
   mWorkerPrivate->AssertIsOnWorkerThread();
   MOZ_ASSERT(mProxy);
 
-  // Only send readystatechange event when state changed.
-  bool isStateChanged = false;
-  if (mStateData.mReadyState != 4) {
-    isStateChanged = true;
-    mStateData.mReadyState = 4;
-  }
+  mStateData.mReadyState = 4;
 
   if (mProxy->mSeenUploadLoadStart) {
     MOZ_ASSERT(mUpload);
 
     DispatchPrematureAbortEvent(mUpload, NS_LITERAL_STRING("abort"), true,
                                 aRv);
     if (aRv.Failed()) {
       return;
@@ -1721,22 +1716,20 @@ XMLHttpRequest::MaybeDispatchPrematureAb
     if (aRv.Failed()) {
       return;
     }
 
     mProxy->mSeenUploadLoadStart = false;
   }
 
   if (mProxy->mSeenLoadStart) {
-    if (isStateChanged) {
-      DispatchPrematureAbortEvent(this, NS_LITERAL_STRING("readystatechange"),
-                                  false, aRv);
-      if (aRv.Failed()) {
-        return;
-      }
+    DispatchPrematureAbortEvent(this, NS_LITERAL_STRING("readystatechange"),
+                                false, aRv);
+    if (aRv.Failed()) {
+      return;
     }
 
     DispatchPrematureAbortEvent(this, NS_LITERAL_STRING("abort"), false, aRv);
     if (aRv.Failed()) {
       return;
     }
 
     DispatchPrematureAbortEvent(this, NS_LITERAL_STRING("loadend"), false,