Service Worker: Add ServiceWorkerContainer.getRegistrations() method. (Blink)

Service Worker: Add ServiceWorkerContainer.getRegistrations() method. (Blink)

Implement ServiceWorkerContainer.getReistrations() method as per spec:
https://slightlyoff.github.io/ServiceWorker/spec/service_worker/index.html#navigator-service-worker-getRegistrations

Companion CL (Chromium): https://codereview.chromium.org/1146913004/
Companion CL (Blink layout tests): https://codereview.chromium.org/1168393002/
Companion CL (Previous review): https://codereview.chromium.org/1143293003/

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

[jungkees, 2015-06-09 09:02:23]

jungkee.song@samsung.com changed reviewers:
+ falken@chromium.org, kinuko@chromium.org, michaeln@chromium.org,
nhiroki@chromium.org

[jungkees, 2015-06-09 09:05:21]

jungkee.song@samsung.com changed reviewers:
+ tkent@chromium.org

[jungkees, 2015-06-09 09:07:50]

I made this previous CL a 3-sided patch with the blink layout test apart:
https://codereview.chromium.org/1143293003/

PTAL

[falken, 2015-06-10 03:38:54]

lgtm

As suggested in the previous review, I made a chromestatus dashboard entry
https://www.chromestatus.com/features/6734239512920064

[jungkees, 2015-06-10 03:57:51]

The CQ bit was checked by jungkee.song@samsung.com

[commit-bot: I haz the power, 2015-06-10 03:58:02]

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

[commit-bot: I haz the power, 2015-06-10 04:03:49]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-06-10 04:03:50]

Try jobs failed on following builders:
  blink_presubmit on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/blink_presubmit/builds/3...)

[tkent, 2015-06-10 04:28:08]

lgtm

[jungkees, 2015-06-10 04:28:57]

The CQ bit was checked by jungkee.song@samsung.com

[commit-bot: I haz the power, 2015-06-10 04:29:06]

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

[commit-bot: I haz the power, 2015-06-10 05:40:15]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-06-10 05:40:17]

Try jobs failed on following builders:
  linux_blink_rel on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/6...)

[jungkees, 2015-06-10 06:14:26]

It seems like 'method getRegistrations' should be added in
webexposed/global-interface-listing.html and
virtual/stable/webexposed/global-interface-listing.html. Can I add them in this
patch set and re-try the commit?

[jungkees, 2015-06-10 06:16:54]

Ah I mean 'webexposed/global-interface-listing-expected.txt' and
'virtual/stable/webexposed/global-interface-listing-expected.txt'.

[falken, 2015-06-10 06:17:06]

On 2015/06/10 06:14:26, jungkees wrote:
> It seems like 'method getRegistrations' should be added in
> webexposed/global-interface-listing.html and
> virtual/stable/webexposed/global-interface-listing.html. Can I add them in
this
> patch set and re-try the commit?

Yes please go ahead.

[jungkees, 2015-06-10 06:29:52]

Patchset #3 (id:40001) has been deleted

[jungkees, 2015-06-10 06:44:53]

The CQ bit was checked by jungkee.song@samsung.com

[jungkees, 2015-06-10 06:44:53]

The patchset sent to the CQ was uploaded after l-g-t-m from falken@chromium.org,
tkent@chromium.org
Link to the patchset: https://codereview.chromium.org/1165363003/#ps60001
(title: "Add method getRegistrations to global-interface-listing-expected.txt
files.")

[commit-bot: I haz the power, 2015-06-10 06:45:20]

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

[commit-bot: I haz the power, 2015-06-10 07:44:41]

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