test(XHRConnection): normalize responseText and response

normalize xhr.responseText and xhr.response

- [x] Tests

Closes #2882
This commit is contained in:
gdi2290
2015-07-27 17:01:05 -07:00
committed by Misko Hevery
parent 96eefdfebc
commit cfedc77ce1
2 changed files with 35 additions and 3 deletions

View File

@ -1,5 +1,6 @@
import {Injectable} from 'angular2/di';
import {Promise, PromiseWrapper, PromiseCompleter} from 'angular2/src/facade/async';
import {isPresent} from 'angular2/src/facade/lang';
import {XHR} from './xhr';
@Injectable()
@ -13,7 +14,7 @@ export class XHRImpl extends XHR {
xhr.onload = function() {
// responseText is the old-school way of retrieving response (supported by IE8 & 9)
// response/responseType properties were introduced in XHR Level2 spec (supported by IE10)
var response = ('response' in xhr) ? xhr.response : xhr.responseText;
var response = isPresent(xhr.response) ? xhr.response : xhr.responseText;
// normalize IE9 bug (http://bugs.jquery.com/ticket/1450)
var status = xhr.status === 1223 ? 204 : xhr.status;