Set the "waiting" Service Worker of Service Worker providers

Set the "waiting" Service Worker of Service Worker providers

This provides the Service Worker for the implementation of navigator.serviceWorker.waiting in Blink. This will be covered by subsequent layout tests in Blink.

BUG=379012
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=276751

[dominicc (has gone to gerrit), 2014-06-10 05:06:07]

PTAL

Blink-side layout tests are in https://codereview.chromium.org/324893002/

[dominicc (has gone to gerrit), 2014-06-10 05:07:33]

+tsepez for content/common/service_worker/service_worker_messages.h

[Tom Sepez, 2014-06-10 18:10:53]

Messages LGTM.

[falken, 2014-06-10 20:34:28]

lgtm

https://codereview.chromium.org/317953004/diff/40001/content/child/service_wo...
File content/child/service_worker/service_worker_dispatcher.cc (right):

https://codereview.chromium.org/317953004/diff/40001/content/child/service_wo...
content/child/service_worker/service_worker_dispatcher.cc:276: // Populate the
.current field with the new worker object.
can you update the comment to say .controller also?

https://codereview.chromium.org/317953004/diff/40001/content/child/service_wo...
File content/child/service_worker/web_service_worker_provider_impl.cc (right):

https://codereview.chromium.org/317953004/diff/40001/content/child/service_wo...
content/child/service_worker/web_service_worker_provider_impl.cc:52:
GetDispatcher()->GetServiceWorker(context_->waiting()->info(), false));
as the body is more than a single line, the if statement should probably have
braces

https://codereview.chromium.org/317953004/diff/40001/content/child/service_wo...
content/child/service_worker/web_service_worker_provider_impl.cc:56:
GetDispatcher()->GetServiceWorker(context_->current()->info(), false));
here too

[dominicc (has gone to gerrit), 2014-06-11 18:06:20]

The CQ bit was checked by dominicc@chromium.org

[commit-bot: I haz the power, 2014-06-11 18:10:08]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/dominicc@chromium.org/317953004/80001

[commit-bot: I haz the power, 2014-06-11 22:28:57]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  mac_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_rel/buil...)
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)
  win_chromium_x64_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_x64_rel/...)

[commit-bot: I haz the power, 2014-06-11 22:32:32]

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

[commit-bot: I haz the power, 2014-06-11 22:32:33]

Try jobs failed on following builders:
  mac_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_rel/buil...)
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)
  win_chromium_x64_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_x64_rel/...)

[dominicc (has gone to gerrit), 2014-06-12 14:54:25]

The CQ bit was checked by dominicc@chromium.org

[commit-bot: I haz the power, 2014-06-12 14:56:33]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/dominicc@chromium.org/317953004/100001

[commit-bot: I haz the power, 2014-06-12 18:27:40]

Change committed as 276751