Service Worker: Enable Clients.get(id)

Service Worker: Enable Clients.get(id)

This method allows a service worker to get a promise that resolves with the
Client object matching a given id.

Intent to implement and ship: https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/XlBHOIOCjUk

Spec discussion: https://github.com/slightlyoff/ServiceWorker/issues/723#issuecomment-153978848
Spec: https://slightlyoff.github.io/ServiceWorker/spec/service_worker/#clients-get-method

CL: https://codereview.chromium.org/1439333002/ and https://codereview.chromium.org/1731823003/
Test result: http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=clients-get

BUG=552310
Committed: https://crrev.com/0d2d3a059242663ac56cb036c2fb0bba0d7322d6
Cr-Commit-Position: refs/heads/master@{#378678}

[jungkees, 2016-02-29 08:26:42]

Description was changed from

==========
Service Worker: Enable Clients.get(id)

This method allows a service worker to get a promise that resolves with the
Client object matching a given id.

Intent to implement and ship:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/XlBHOIOCjUk

Spec discussion:
https://github.com/slightlyoff/ServiceWorker/issues/723#issuecomment-153978848
Spec:
https://slightlyoff.github.io/ServiceWorker/spec/service_worker/#clients-get-...

CL: https://codereview.chromium.org/1439333002/ and
https://codereview.chromium.org/1731823003/
Test result:
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=cli...

BUG=552310
==========

to

==========
Service Worker: Enable Clients.get(id)

This method allows a service worker to get a promise that resolves with the
Client object matching a given id.

Intent to implement and ship:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/XlBHOIOCjUk

Spec discussion:
https://github.com/slightlyoff/ServiceWorker/issues/723#issuecomment-153978848
Spec:
https://slightlyoff.github.io/ServiceWorker/spec/service_worker/#clients-get-...

CL: https://codereview.chromium.org/1439333002/ and
https://codereview.chromium.org/1731823003/
Test result:
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=cli...

BUG=552310
==========

[jungkees, 2016-02-29 08:26:42]

jungkee.song@samsung.com changed reviewers:
+ falken@chromium.org, jinho.bang@samsung.com, nhiroki@chromium.org,
tkent@chromium.org

[jungkees, 2016-02-29 08:27:53]

As discussed in
https://bugs.chromium.org/p/chromium/issues/detail?id=552310#c12, I uploaded a
patch to enable Clients.get(id). PTAL

[nhiroki, 2016-02-29 08:50:34]

lgtm

[jungkees, 2016-02-29 10:25:50]

tkent@, could you review it?

[tkent, 2016-02-29 23:11:46]

lgtm

[jungkees, 2016-03-02 01:32:04]

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

[commit-bot: I haz the power, 2016-03-02 01:32:52]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1743163002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1743163002/1

[commit-bot: I haz the power, 2016-03-02 01:59:49]

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

[commit-bot: I haz the power, 2016-03-02 01:59:53]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[jungkees, 2016-03-02 02:45:01]

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

[commit-bot: I haz the power, 2016-03-02 02:45:40]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1743163002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1743163002/1

[commit-bot: I haz the power, 2016-03-02 03:48:11]

Description was changed from

==========
Service Worker: Enable Clients.get(id)

This method allows a service worker to get a promise that resolves with the
Client object matching a given id.

Intent to implement and ship:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/XlBHOIOCjUk

Spec discussion:
https://github.com/slightlyoff/ServiceWorker/issues/723#issuecomment-153978848
Spec:
https://slightlyoff.github.io/ServiceWorker/spec/service_worker/#clients-get-...

CL: https://codereview.chromium.org/1439333002/ and
https://codereview.chromium.org/1731823003/
Test result:
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=cli...

BUG=552310
==========

to

==========
Service Worker: Enable Clients.get(id)

This method allows a service worker to get a promise that resolves with the
Client object matching a given id.

Intent to implement and ship:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/XlBHOIOCjUk

Spec discussion:
https://github.com/slightlyoff/ServiceWorker/issues/723#issuecomment-153978848
Spec:
https://slightlyoff.github.io/ServiceWorker/spec/service_worker/#clients-get-...

CL: https://codereview.chromium.org/1439333002/ and
https://codereview.chromium.org/1731823003/
Test result:
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=cli...

BUG=552310
==========

[commit-bot: I haz the power, 2016-03-02 03:48:12]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-02 03:50:32]

Description was changed from

==========
Service Worker: Enable Clients.get(id)

This method allows a service worker to get a promise that resolves with the
Client object matching a given id.

Intent to implement and ship:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/XlBHOIOCjUk

Spec discussion:
https://github.com/slightlyoff/ServiceWorker/issues/723#issuecomment-153978848
Spec:
https://slightlyoff.github.io/ServiceWorker/spec/service_worker/#clients-get-...

CL: https://codereview.chromium.org/1439333002/ and
https://codereview.chromium.org/1731823003/
Test result:
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=cli...

BUG=552310
==========

to

==========
Service Worker: Enable Clients.get(id)

This method allows a service worker to get a promise that resolves with the
Client object matching a given id.

Intent to implement and ship:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/XlBHOIOCjUk

Spec discussion:
https://github.com/slightlyoff/ServiceWorker/issues/723#issuecomment-153978848
Spec:
https://slightlyoff.github.io/ServiceWorker/spec/service_worker/#clients-get-...

CL: https://codereview.chromium.org/1439333002/ and
https://codereview.chromium.org/1731823003/
Test result:
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=cli...

BUG=552310

Committed: https://crrev.com/0d2d3a059242663ac56cb036c2fb0bba0d7322d6
Cr-Commit-Position: refs/heads/master@{#378678}
==========

[commit-bot: I haz the power, 2016-03-02 03:50:33]

Patchset 1 (id:??) landed as
https://crrev.com/0d2d3a059242663ac56cb036c2fb0bba0d7322d6
Cr-Commit-Position: refs/heads/master@{#378678}