ServiceWorker: Implement ServiceWorkerRegistration.update() (1)

ServiceWorker: Implement ServiceWorkerRegistration.update() (1)

Spec: https://slightlyoff.github.io/ServiceWorker/spec/service_worker/index.html#service-worker-registration-update
Intent-to-Ship: https://groups.google.com/a/chromium.org/forum/#!topic/Blink-dev/bvi8fXqvNhs

(1) Blink: THIS PATCH
(2) Chromium: https://codereview.chromium.org/1175823002
(3) Blink: https://codereview.chromium.org/1177563005

BUG=450507
TEST=will be added by (3)

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

[nhiroki, 2015-06-09 23:57:51]

Patchset #1 (id:1) has been deleted

[nhiroki, 2015-06-10 03:37:06]

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

[nhiroki, 2015-06-10 03:37:07]

PTAL, thanks!

[falken, 2015-06-10 03:55:03]

lgtm

Please link to the intent to ship in CL description (and also the bug)

[nhiroki, 2015-06-10 04:09:09]

On 2015/06/10 03:55:03, falken wrote:
> lgtm
> 
> Please link to the intent to ship in CL description (and also the bug)

Thank you for reviewing! Updated the CL description and the bug.

[nhiroki, 2015-06-10 04:09:57]

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

[nhiroki, 2015-06-10 04:09:58]

+tkent@, can you review public/platform/? Thanks!

[tkent, 2015-06-10 04:30:42]

lgtm

[nhiroki, 2015-06-10 06:40:00]

The CQ bit was checked by nhiroki@chromium.org

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

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

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

Committed patchset #1 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=196838

[nhiroki, 2015-06-11 04:19:03]

A revert of this CL (patchset #1 id:20001) has been created in
https://codereview.chromium.org/1178753003/ by nhiroki@chromium.org.

The reason for reverting is: I'll remake this CL as per the discussion in the
CL(2)..

[kinuko, 2015-06-11 04:22:48]

On 2015/06/11 04:19:03, nhiroki wrote:
> A revert of this CL (patchset #1 id:20001) has been created in
> https://codereview.chromium.org/1178753003/ by mailto:nhiroki@chromium.org.
> 
> The reason for reverting is: I'll remake this CL as per the discussion in the
> CL(2)..

It's ok but not sure if you needed to revert? Looks like we could just change on
top of it.

[nhiroki, 2015-06-11 04:33:39]

On 2015/06/11 04:22:48, kinuko wrote:
> On 2015/06/11 04:19:03, nhiroki wrote:
> > A revert of this CL (patchset #1 id:20001) has been created in
> > https://codereview.chromium.org/1178753003/ by mailto:nhiroki@chromium.org.
> > 
> > The reason for reverting is: I'll remake this CL as per the discussion in
the
> > CL(2)..
> 
> It's ok but not sure if you needed to revert? Looks like we could just change
on
> top of it.

No special reason, just to rollback the change. Yes, we were able to make a
change on top of that.