From 6f1050df4fa885bd59ce85adbef7350ea93911a3 Mon Sep 17 00:00:00 2001 From: Gonzalo Ruiz de Villa Date: Wed, 13 Nov 2013 21:24:15 +0100 Subject: fix(httpBackend): should not read response data when request is aborted When a request is aborted, it makes no sense to read the response headers or text. Also in IE9, trying to read data (either response headers or text) from an aborted request throws an Error c00c023f. Fixes #4913 Closes #4940 --- test/ng/httpBackendSpec.js | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+) (limited to 'test/ng') diff --git a/test/ng/httpBackendSpec.js b/test/ng/httpBackendSpec.js index b262d8a2..cc73ea8e 100644 --- a/test/ng/httpBackendSpec.js +++ b/test/ng/httpBackendSpec.js @@ -118,6 +118,25 @@ describe('$httpBackend', function() { }); }); + it('should not try to read response data when request is aborted', function() { + callback.andCallFake(function(status, response, headers) { + expect(status).toBe(-1); + expect(response).toBe(null); + expect(headers).toBe(null); + }); + $backend('GET', '/url', null, callback, {}, 2000); + xhr = MockXhr.$$lastInstance; + spyOn(xhr, 'abort'); + + fakeTimeout.flush(); + expect(xhr.abort).toHaveBeenCalledOnce(); + + xhr.status = 0; + xhr.readyState = 4; + xhr.onreadystatechange(); + expect(callback).toHaveBeenCalledOnce(); + }); + it('should abort request on timeout', function() { callback.andCallFake(function(status, response) { expect(status).toBe(-1); -- cgit v1.2.3