Initial implementation of stashed message ports, layout tests

Initial implementation of stashed message ports, layout tests

This is part of a series of patches to implement an
initial version of a mechanism for a MessagePort to
outlive a service worker as described in
https://github.com/mkruisselbrink/navigator-connect/blob/gh-pages/docs/api-idea.md#persisted-messageports

[1/3] https://codereview.chromium.org/1008533005/, blink side implementation
[2/3] https://codereview.chromium.org/1110103003/, content side implementation
[3/3] this patch, layout tests

BUG=426458, 483108
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=195643

[Marijn Kruisselbrink, 2015-04-30 18:39:05]

Patchset #1 (id:1) has been deleted

[Marijn Kruisselbrink, 2015-04-30 18:39:12]

Patchset #1 (id:20001) has been deleted

[Marijn Kruisselbrink, 2015-04-30 18:39:16]

Patchset #1 (id:40001) has been deleted

[scheib, 2015-05-06 22:36:17]

scheib@chromium.org changed reviewers:
+ scheib@chromium.org

[scheib, 2015-05-06 22:36:18]

https://codereview.chromium.org/1063533002/diff/80001/LayoutTests/http/tests/...
File
LayoutTests/http/tests/serviceworker/chromium/resources/stashed-port-worker.js
(right):

https://codereview.chromium.org/1063533002/diff/80001/LayoutTests/http/tests/...
LayoutTests/http/tests/serviceworker/chromium/resources/stashed-port-worker.js:1:

It would help to have more unique / descriptive names than 
stashed-port-worker.js
stashed-ports-worker.js

https://codereview.chromium.org/1063533002/diff/80001/LayoutTests/http/tests/...
File LayoutTests/http/tests/serviceworker/chromium/stashed-ports.html (right):

https://codereview.chromium.org/1063533002/diff/80001/LayoutTests/http/tests/...
LayoutTests/http/tests/serviceworker/chromium/stashed-ports.html:25: function
delay(ms) {
unused

https://codereview.chromium.org/1063533002/diff/80001/LayoutTests/http/tests/...
File LayoutTests/http/tests/serviceworker/resources/stashed-ports-worker.js
(right):

https://codereview.chromium.org/1063533002/diff/80001/LayoutTests/http/tests/...
LayoutTests/http/tests/serviceworker/resources/stashed-ports-worker.js:5:
function wait_for_reply(test, port) {
wait_for_reply, wait_for_message, very similar names, work differently, how
about:

wait_for_message_on_port
wait_for_message_from_source

https://codereview.chromium.org/1063533002/diff/80001/LayoutTests/http/tests/...
LayoutTests/http/tests/serviceworker/resources/stashed-ports-worker.js:55: },
'Messages send to stashed port arrive as global events.');
send -> sent

[Marijn Kruisselbrink, 2015-05-13 02:46:03]

https://codereview.chromium.org/1063533002/diff/80001/LayoutTests/http/tests/...
File
LayoutTests/http/tests/serviceworker/chromium/resources/stashed-port-worker.js
(right):

https://codereview.chromium.org/1063533002/diff/80001/LayoutTests/http/tests/...
LayoutTests/http/tests/serviceworker/chromium/resources/stashed-port-worker.js:1:

On 2015/05/06 22:36:18, scheib wrote:
> It would help to have more unique / descriptive names than 
> stashed-port-worker.js
> stashed-ports-worker.js

Done.

https://codereview.chromium.org/1063533002/diff/80001/LayoutTests/http/tests/...
File LayoutTests/http/tests/serviceworker/chromium/stashed-ports.html (right):

https://codereview.chromium.org/1063533002/diff/80001/LayoutTests/http/tests/...
LayoutTests/http/tests/serviceworker/chromium/stashed-ports.html:25: function
delay(ms) {
On 2015/05/06 22:36:18, scheib wrote:
> unused

Done.

https://codereview.chromium.org/1063533002/diff/80001/LayoutTests/http/tests/...
File LayoutTests/http/tests/serviceworker/resources/stashed-ports-worker.js
(right):

https://codereview.chromium.org/1063533002/diff/80001/LayoutTests/http/tests/...
LayoutTests/http/tests/serviceworker/resources/stashed-ports-worker.js:5:
function wait_for_reply(test, port) {
On 2015/05/06 22:36:18, scheib wrote:
> wait_for_reply, wait_for_message, very similar names, work differently, how
> about:
> 
> wait_for_message_on_port
> wait_for_message_from_source

Done.

https://codereview.chromium.org/1063533002/diff/80001/LayoutTests/http/tests/...
LayoutTests/http/tests/serviceworker/resources/stashed-ports-worker.js:55: },
'Messages send to stashed port arrive as global events.');
On 2015/05/06 22:36:18, scheib wrote:
> send -> sent

Done.

[scheib, 2015-05-13 04:03:07]

LGTM

Though, why have one test script inline in HTML and other external in .js?

[Marijn Kruisselbrink, 2015-05-13 04:13:12]

On 2015/05/13 04:03:07, scheib wrote:
> LGTM
> 
> Though, why have one test script inline in HTML and other external in .js?

One test just tests functionality from within a serviceworker, so the HTML file
doesn't need to do anything besides installing/launching a service worker (which
is all done by service_worker_test), the other test tests behavior that includes
killing the service worker, so there the logic that is running outside the
service worker (in the html file) is more complicated.

[Marijn Kruisselbrink, 2015-05-20 21:49:06]

The CQ bit was checked by mek@chromium.org

[commit-bot: I haz the power, 2015-05-20 21:49:29]

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

[commit-bot: I haz the power, 2015-05-20 23:32:15]

Committed patchset #3 (id:100001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=195643