Rework EventSource CORS tests to be usable from Workers.

Rework EventSource CORS tests to be usable from Workers.

Improve EventSource-in-Workers test coverage by recasting existing
CORS tests as js-test.js tests, enabling them to also be testable from
within Workers. As was, we had no such Worker test coverage.

Also add two new tests covering failure-on-redirect handling: one due
to the CORS access check, the other due to CSP connect-src's.

R=abarth@chromium.org,mkwst@chromium.org
BUG=386520
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=176950

[sof, 2014-06-22 13:30:33]

Please take a look.

Improve CORS+CSP testing for EventSource-in-Workers a bit; depends on
https://codereview.chromium.org/340363003/

[abarth-chromium, 2014-06-22 14:48:55]

rslgtm

[Mike West, 2014-06-23 06:13:37]

The tests are much clean, thanks for putting this patch together.

LGTM.

https://codereview.chromium.org/345813005/diff/1/LayoutTests/http/tests/resou...
File LayoutTests/http/tests/resources/redirect.php (right):

https://codereview.chromium.org/345813005/diff/1/LayoutTests/http/tests/resou...
LayoutTests/http/tests/resources/redirect.php:13:
header("Access-Control-Allow-Origin: *");
It doesn't matter for any of these tests, but it might be reasonable to change
the hard-coded '*' into a GET param to check redirect behavior in a future
patch.

[sof, 2014-06-23 06:42:18]

https://codereview.chromium.org/345813005/diff/1/LayoutTests/http/tests/resou...
File LayoutTests/http/tests/resources/redirect.php (right):

https://codereview.chromium.org/345813005/diff/1/LayoutTests/http/tests/resou...
LayoutTests/http/tests/resources/redirect.php:13:
header("Access-Control-Allow-Origin: *");
On 2014/06/23 06:13:37, Mike West wrote:
> It doesn't matter for any of these tests, but it might be reasonable to change
> the hard-coded '*' into a GET param to check redirect behavior in a future
> patch.

Good idea, a redirect script this "high up" is better off parameterizing that
out. Added 'cors_allow_origin' (instead of 'cors_enabled'.)

[sof, 2014-06-25 20:30:03]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2014-06-25 20:31:22]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sigbjornf@opera.com/345813005/20001

[commit-bot: I haz the power, 2014-06-25 21:36:42]

Change committed as 176950