fix(xhr): fix #657, sometimes xhr will fire onreadystatechange with d…

…one twice (#658)
angular · Mar 7, 2017 · 36c0899 · 36c0899
1 parent 84459f1
commit 36c0899
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/lib/browser/browser.ts b/lib/browser/browser.ts
@@ -73,7 +73,9 @@ function patchXHR(window: any) {
     }
     const newListener = data.target[XHR_LISTENER] = () => {
       if (data.target.readyState === data.target.DONE) {
-        if (!data.aborted && self[XHR_SCHEDULED]) {
+        // sometimes on some browsers XMLHttpRequest will fire onreadystatechange with
+        // readyState=4 multiple times, so we need to check task state here
+        if (!data.aborted && self[XHR_SCHEDULED] && task.state === 'scheduled') {
           task.invoke();
         }
       }