Bug 12307 – No way to test error condition of synchronous XMLHttpRequest to file:///

• Component: XML
• Status: NEW

[Jesse Costello-Good]

There is no way to tell whether an asynchronous XML HTTP request to the local disk (when running a page from file:///) has failed. 

// #1 works ok from HTTP
var r = new XMLHttpRequest();
r.open("GET", "http://www.apple.com/nowhere.html", true);
r.send();
// wait ...
r.status == 404

// #2 works ok synchronously
var r = new XMLHttpRequest();
r.open("GET", "nowhere.html", false);
r.send(); // throws an error 

// #3 but asynchronously ... no way to detect the error
var r = new XMLHttpRequest();
r.open("GET", "nowhere.html", true);
r.send();
// wait ... no error thrown
r.status == 0


Both Internet Explorer and Firefox seem to ignore async when requesting a resource from the local disk. They both throw an exception like Safari does in the second example. But they also both throw an exception in the third example whereas Safari doesn't.

[Alexey Proskuryakov]

Created attachment 13181 [details]
test case

In the future, we will probably dispatch an error event for XMLHttpRequest.

From my testing, it appears that IE 7 refuses to load local files altogether. Firefox 2 does raise an exception in send(), but if a local file is accessible, then the actual loading happens asynchronously. This behavior is not in accordance with draft XMLHttpRequest spec AFAICT.

[Julien Chaffraix]

(In reply to comment #1)
> Created an attachment (id=13181) [edit]
> test case
> 
> In the future, we will probably dispatch an error event for XMLHttpRequest.

We have implemented 'onerror' from XMLHttpRequest level 2, which means that we now have a way to check for asynchronous request failures.

However during my quick testing, I have tried doing a synchronous one to a non-existent file (just modify the given test case to do a synchronous request) and we do not dispatch an error event nor do we throw an exception.

We do not raise an exception because it breaks some website (see http://trac.webkit.org/changeset/23889) but we could at least dispatch an event so there is a way to determine that an error occurred.

[Alexey Proskuryakov]

Renaming the bug, as it now tracks sync requests, not async ones.

[Marcus Better]

(In reply to comment #1)
> From my testing, it appears that IE 7 refuses to load local files
> altogether.

This is not quite accurate, it can load local files if the native XHR implementation is switched on in the "Advanced" tab of the "Internet Options".

[Alexey Proskuryakov]

Created attachment 233084 [details]
WIP patch

Fails some tests.

[Alexey Proskuryakov]

Created attachment 233094 [details]
slightly better, but still fails tests