NotificationsYou must be signed in to change notification settings
Fork20.6k
Star59.5k

Commitabdc89a

authored

Ajax: Simplify jQuery.ajaxSettings.xhr

Previously, jQuery.ajaxSettings.xhr, contents were wrapped in a try-catchas we defined jQuery.support.ajax & jQuery.support.cors executed during thejQuery load and we didn't want to crash if IE had native XHR disabled (whichis possible). While jQuery hasn't supported the ActiveX-based XHR since 2.0,jQuery with XHR disabled could still be used for its other features in sucha crippled browser.Sincegh-4347, jQuery.support.ajax & jQuery.support.cors no longer exist, sowe don't need the try-catch anymore.Fixesgh-1967Closesgh-4467Refgh-4347

1 parentd7e6419 commitabdc89aCopy full SHA for abdc89a

File tree

1 file changed

-3

lines changed

src/ajax
- xhr.js

1 file changed

-3

lines changed

`‎src/ajax/xhr.js`

Lines changed: 1 addition & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -6,9 +6,7 @@ define( [`
`6`	`6`	`"use strict";`
`7`	`7`
`8`	`8`	`jQuery.ajaxSettings.xhr=function(){`
`9`		`-try{`
`10`		`-returnnewwindow.XMLHttpRequest();`
`11`		`-}catch(e){}`
	`9`	`+returnnewwindow.XMLHttpRequest();`
`12`	`10`	`};`
`13`	`11`
`14`	`12`	`varxhrSuccessStatus={`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commitabdc89a

File tree

1 file changed

1 file changed

`‎src/ajax/xhr.js`

0 commit comments