ServiceWorker: Support navigator.serviceWorker.active

ServiceWorker: Support navigator.serviceWorker.active

This change plumbs '.active' and you will be able to access
an active worker via 'navigator.serviceWorker.active'.


BUG=379012
TEST=content_unittest --gtest_filter=ServiceWorker*
TEST=run_webkit_tests.py http/tests/serviceworker/waiting.html
TEST=run_webkit_tests.py  http/tests/serviceworker/active.html (http://crrev.com/373013002)

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

[nhiroki, 2014-07-08 08:33:20]

Ptal, thanks!

[nhiroki, 2014-07-08 14:38:22]

+jschuh@

Hi Justin, could you review service_worker_messages.h? Thanks.

[jschuh, 2014-07-08 17:15:22]

ipc security lgtm (notes: browser -> renderer URLs and routing info).

[michaeln, 2014-07-09 02:07:06]

lgtm

https://codereview.chromium.org/354643003/diff/200001/content/child/service_w...
File content/child/service_worker/service_worker_provider_context.cc (right):

https://codereview.chromium.org/354643003/diff/200001/content/child/service_w...
content/child/service_worker/service_worker_provider_context.cc:66: which =
active_.get();
style nit: these one line blocks don't need {}'s

[nhiroki, 2014-07-09 06:14:27]

https://codereview.chromium.org/354643003/diff/200001/content/child/service_w...
File content/child/service_worker/service_worker_provider_context.cc (right):

https://codereview.chromium.org/354643003/diff/200001/content/child/service_w...
content/child/service_worker/service_worker_provider_context.cc:66: which =
active_.get();
On 2014/07/09 02:07:05, michaeln wrote:
> style nit: these one line blocks don't need {}'s

Done.

[nhiroki, 2014-07-09 06:15:30]

The CQ bit was checked by nhiroki@chromium.org

[commit-bot: I haz the power, 2014-07-09 06:16:37]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/nhiroki@chromium.org/354643003/240001

[commit-bot: I haz the power, 2014-07-09 10:07:08]

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

[commit-bot: I haz the power, 2014-07-09 11:57:28]

Change committed as 282006