Change two layout tests to work with Chrome's 'file' url policy.

• Component: Tools / Tests
• Status: RESOLVED FIXED

[Feng Qian]

The test relies on the fact that file URL scan send XHR requests to any URL.
Chrome is going to have a tighter policy that file url can only send XHR requests to file URLs.

The test needs some changes to accommodate both Safari and Chrome's policies.

Instead of sending an XHR request to http://localhost (which is not existing), the test can send an XHR to the URL itself.

[Feng Qian]

Created attachment 24749 [details]
changes in the test

[Alexey Proskuryakov]

Comment on attachment 24749 [details]
changes in the test

r=me

I presume the test now works in Firefox, too?

[Feng Qian]

The test is failing in FF for different reasons.

Before the change, the test failed because it throws an exception: NS_ERROR_ILLEGAL_VALUE, ex.code is undefined.

After the change, the test failed because no exception is thrown.

(In reply to comment #2)
> (From update of attachment 24749 [details] [edit])
> r=me
> 
> I presume the test now works in Firefox, too?
> 

[Feng Qian]

Created attachment 24755 [details]
patch for another test

Another test has a similar issue. It sends XHR request to 'about:blank'. It is allowed in Safari if the source URL is a file URL. Chrome does not allow it.

The test is testing something else, so make it send URL to itself.

[Alexey Proskuryakov]

Comment on attachment 24755 [details]
patch for another test

r=me

You may need to make these patches easier to land (correct bug title, add ChangeLogs, possibly merge patches into one) - or to find a committer who is willing to do this for you.

[Feng Qian]

Created attachment 24762 [details]
a merge patch with changelog

This is a merged patch with changelog, it is easier for commit.

[Alexey Proskuryakov]

Comment on attachment 24762 [details]
a merge patch with changelog

r=me

[Adam Barth]

Will land.

[Adam Barth]

LayoutTests/ChangeLog: locally modified
LayoutTests/fast/dom/null-document-xmlhttprequest-open.html: locally modified
LayoutTests/fast/dom/xmlhttprequest-invalid-values.html: locally modified
	M	LayoutTests/ChangeLog
	M	LayoutTests/fast/dom/null-document-xmlhttprequest-open.html
	M	LayoutTests/fast/dom/xmlhttprequest-invalid-values.html
Committed r39365
	M	LayoutTests/ChangeLog
	M	LayoutTests/fast/dom/xmlhttprequest-invalid-values.html
	M	LayoutTests/fast/dom/null-document-xmlhttprequest-open.html
r39365 = d0786620de79c0565e62411bd46561ebdb5df15a (trunk)