ServiceWorker: Make '.ready' return a promise to be resolved with SWRegistration (3/3)

ServiceWorker: Make '.ready' return a promise to be resolved with ServiceWorkerRegistration (3/3)

This enables and tweaks ready.html to follow up the implementation changes.

1) Blink: https://codereview.chromium.org/476043002/
2) Chromium: https://codereview.chromium.org/477593007/
3) Blink: THIS PATCH


BUG=399533
TEST=run_webkit_tests.py --debug http/tests/serviceworker/

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

[nhiroki, 2014-09-10 08:12:07]

Patchset #2 (id:60001) has been deleted

[nhiroki, 2014-09-10 08:13:10]

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

[nhiroki, 2014-09-10 08:13:11]

Ptal, thanks!

[horo, 2014-09-10 08:57:54]

https://codereview.chromium.org/532653002/diff/80001/LayoutTests/http/tests/s...
File LayoutTests/http/tests/serviceworker/ready.html (right):

https://codereview.chromium.org/532653002/diff/80001/LayoutTests/http/tests/s...
LayoutTests/http/tests/serviceworker/ready.html:73: return wait_for_activated(t,
registration);
is this needed?

https://codereview.chromium.org/532653002/diff/80001/LayoutTests/http/tests/s...
LayoutTests/http/tests/serviceworker/ready.html:130: 
could you please add test for 
 register(script1.js)
 unregister()
 register(script2.js)
 ready
 check registration.active.scriptURL

[nhiroki, 2014-09-11 08:16:37]

https://codereview.chromium.org/532653002/diff/80001/LayoutTests/http/tests/s...
File LayoutTests/http/tests/serviceworker/ready.html (right):

https://codereview.chromium.org/532653002/diff/80001/LayoutTests/http/tests/s...
LayoutTests/http/tests/serviceworker/ready.html:73: return wait_for_activated(t,
registration);
On 2014/09/10 08:57:54, horo wrote:
> is this needed?

Good catch. This is needless. Removed.

https://codereview.chromium.org/532653002/diff/80001/LayoutTests/http/tests/s...
LayoutTests/http/tests/serviceworker/ready.html:130: 
On 2014/09/10 08:57:53, horo wrote:
> could you please add test for 
>  register(script1.js)
>  unregister()
>  register(script2.js)
>  ready
>  check registration.active.scriptURL

Hmmm... I wrote such tests and they showed unexpected results. I guess we need
to tweak the Chromium side to fix them, so can I land this CL as is for now?
Wdyt?

[horo, 2014-09-11 08:49:11]

lgtm

[nhiroki, 2014-09-11 09:00:32]

The CQ bit was checked by nhiroki@chromium.org

[commit-bot: I haz the power, 2014-09-11 09:01:20]

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

[commit-bot: I haz the power, 2014-09-11 09:25:07]

Committed patchset #3 (id:100001) as 181806