syncxhr doesn't deadlock layout test

Layout test to ensure we bypass ServiceWorkers for synchronous resource loads to avoid deadlocks when the worker and page are running in the same process.

[1] https://codereview.chromium.org/553333005/
[2] this cl

BUG=413103
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=182888

[michaeln, 2014-09-17 23:03:48]

michaeln@chromium.org changed reviewers:
+ horo@chromium.org

[michaeln, 2014-09-17 23:09:54]

ptal, i can't land this until the other change is in

[horo, 2014-09-18 01:04:29]

lgtm with nits.

https://codereview.chromium.org/573293005/diff/1/LayoutTests/http/tests/servi...
File
LayoutTests/http/tests/serviceworker/resources/sync-xhr-doesnt-deadlock-iframe.html
(right):

https://codereview.chromium.org/573293005/diff/1/LayoutTests/http/tests/servi...
LayoutTests/http/tests/serviceworker/resources/sync-xhr-doesnt-deadlock-iframe.html:5:
var url = 'sync-xhr-doesnt-deadlock.data?bustcache=' + Date.now();
ditto

https://codereview.chromium.org/573293005/diff/1/LayoutTests/http/tests/servi...
File LayoutTests/http/tests/serviceworker/resources/sync-xhr-doesnt-deadlock.js
(right):

https://codereview.chromium.org/573293005/diff/1/LayoutTests/http/tests/servi...
LayoutTests/http/tests/serviceworker/resources/sync-xhr-doesnt-deadlock.js:2: if
(event.request.url.indexOf('sync-xhr-doesnt-deadlock.data') == -1)
nit: Indent two spaces.
http://www.chromium.org/blink/serviceworker/testing#TOC-Wrapping-and-Indentation

https://codereview.chromium.org/573293005/diff/1/LayoutTests/http/tests/servi...
File LayoutTests/http/tests/serviceworker/sync-xhr-doesnt-deadlock-expected.txt
(right):

https://codereview.chromium.org/573293005/diff/1/LayoutTests/http/tests/servi...
LayoutTests/http/tests/serviceworker/sync-xhr-doesnt-deadlock-expected.txt:1:
This is a testharness.js-based test.
We don't need this file.
The testharness expected files are not required.

[michaeln, 2014-09-30 00:42:36]

On 2014/09/18 01:04:29, horo wrote:
> lgtm with nits.
> 
>
https://codereview.chromium.org/573293005/diff/1/LayoutTests/http/tests/servi...
> File
>
LayoutTests/http/tests/serviceworker/resources/sync-xhr-doesnt-deadlock-iframe.html
> (right):
> 
>
https://codereview.chromium.org/573293005/diff/1/LayoutTests/http/tests/servi...
>
LayoutTests/http/tests/serviceworker/resources/sync-xhr-doesnt-deadlock-iframe.html:5:
> var url = 'sync-xhr-doesnt-deadlock.data?bustcache=' + Date.now();
> ditto
> 
>
https://codereview.chromium.org/573293005/diff/1/LayoutTests/http/tests/servi...
> File
LayoutTests/http/tests/serviceworker/resources/sync-xhr-doesnt-deadlock.js
> (right):
> 
>
https://codereview.chromium.org/573293005/diff/1/LayoutTests/http/tests/servi...
> LayoutTests/http/tests/serviceworker/resources/sync-xhr-doesnt-deadlock.js:2:
if
> (event.request.url.indexOf('sync-xhr-doesnt-deadlock.data') == -1)
> nit: Indent two spaces.
>
http://www.chromium.org/blink/serviceworker/testing#TOC-Wrapping-and-Indentation
> 
>
https://codereview.chromium.org/573293005/diff/1/LayoutTests/http/tests/servi...
> File
LayoutTests/http/tests/serviceworker/sync-xhr-doesnt-deadlock-expected.txt
> (right):
> 
>
https://codereview.chromium.org/573293005/diff/1/LayoutTests/http/tests/servi...
> LayoutTests/http/tests/serviceworker/sync-xhr-doesnt-deadlock-expected.txt:1:
> This is a testharness.js-based test.
> We don't need this file.
> The testharness expected files are not required.

Done, done, done

[dominicc (has gone to gerrit), 2014-09-30 05:42:02]

The CQ bit was checked by dominicc@chromium.org

[commit-bot: I haz the power, 2014-09-30 05:42:16]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/573293005/40001

[commit-bot: I haz the power, 2014-09-30 06:46:54]

Committed patchset #3 (id:40001) as 182888