ServiceWorker: Make "ready" fetches registration from browser process(3/3).

ServiceWorker: Make "ready" fetches registration from browser process(3/3).
  
Cleanup obsolete code, and add layout tests.
Remove "ready after unregister" test as an uninstalling registration should
be excluded when finding a matched registration.
 
Blink CL: https://codereview.chromium.org/894983002/
Chrome CL: https://codereview.chromium.org/894973003/

BUG=454726, 462529
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=191672

[xiang, 2015-02-09 07:41:54]

xiang.long@intel.com changed reviewers:
+ dominicc@chromium.org, falken@chromium.org

[xiang, 2015-02-09 08:31:46]

PTAL, thanks!

[dominicc (has gone to gerrit), 2015-02-16 07:19:23]

https://codereview.chromium.org/912443002/diff/1/LayoutTests/http/tests/servi...
File LayoutTests/http/tests/serviceworker/ready.html (left):

https://codereview.chromium.org/912443002/diff/1/LayoutTests/http/tests/servi...
LayoutTests/http/tests/serviceworker/ready.html:99:
service_worker_unregister_and_register(t, url, scope)
Shouldn't all tests start with this, in case a test fails or is interrupted, we
don't want it to cause the next run to also fail?

[falken, 2015-02-16 09:20:42]

Can you add a test that after ready was resolved, a later ready won't return a
longer matched registration that was installed in the meantime? (if i understand
correctly)

https://codereview.chromium.org/912443002/diff/1/LayoutTests/http/tests/servi...
File LayoutTests/http/tests/serviceworker/ready.html (left):

https://codereview.chromium.org/912443002/diff/1/LayoutTests/http/tests/servi...
LayoutTests/http/tests/serviceworker/ready.html:99:
service_worker_unregister_and_register(t, url, scope)
On 2015/02/16 07:19:22, dominicc wrote:
> Shouldn't all tests start with this, in case a test fails or is interrupted,
we
> don't want it to cause the next run to also fail?

Yep

[xiang, 2015-02-26 08:50:34]

Thanks for the review, PTAL. A test for access "ready" twice has been added.

https://codereview.chromium.org/912443002/diff/1/LayoutTests/http/tests/servi...
File LayoutTests/http/tests/serviceworker/ready.html (left):

https://codereview.chromium.org/912443002/diff/1/LayoutTests/http/tests/servi...
LayoutTests/http/tests/serviceworker/ready.html:99:
service_worker_unregister_and_register(t, url, scope)
On 2015/02/16 09:20:42, falken wrote:
> On 2015/02/16 07:19:22, dominicc wrote:
> > Shouldn't all tests start with this, in case a test fails or is interrupted,
> we
> > don't want it to cause the next run to also fail?
> 
> Yep

Done.

[falken, 2015-03-02 02:16:18]

lgtm

[xiang, 2015-03-10 08:36:19]

Patchset #3 (id:40001) has been deleted

[xiang, 2015-03-10 09:00:23]

xiang.long@intel.com changed reviewers:
+ tkent@chromium.org

[xiang, 2015-03-10 09:01:17]

tkent@ would you please review the public/platform changes? Thanks.

[tkent, 2015-03-10 09:28:20]

public/platform lgtm

[xiang, 2015-03-11 01:32:24]

The CQ bit was checked by xiang.long@intel.com

[xiang, 2015-03-11 01:32:25]

The patchset sent to the CQ was uploaded after l-g-t-m from falken@chromium.org
Link to the patchset: https://codereview.chromium.org/912443002/#ps60001 (title:
"add cleanup")

[commit-bot: I haz the power, 2015-03-11 01:32:35]

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

[xiang, 2015-03-11 01:34:17]

The CQ bit was unchecked by xiang.long@intel.com

[xiang, 2015-03-11 01:37:00]

The CQ bit was checked by xiang.long@intel.com

[commit-bot: I haz the power, 2015-03-11 01:37:20]

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

[commit-bot: I haz the power, 2015-03-11 03:01:04]

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