Ah yes those warnings were only added for 'install' and 'activate' event handlers <https://src.chromium.org/viewvc/blink/trunk/Source/modules/serviceworkers/ServiceWorkerGlobalScope.cpp?r1=184227&r2=184226&pathrev=184227>. I think at the time it matched the spec.

 Issue 644617  has been merged into this issue.

 Issue 644991  has been merged into this issue.

As I wrote in  Issue 644991 , warning messages might be eliminated by https://crrev.com/2297803002. Could you take a look this issue?

For repro, I created a simple test case: https://amiq11.github.io/sw-one-registration/register.html
Warning will be shown at the devtools' console using stable, but it won't using ToT.

Also I updated the priority to follow  Issue 644991 .

As for repro commented at https://crrev.com/2297803002, I guess you couldn't do it because of no-fetch optimization of the service worker. I added 'register' and 'unregister' button to https://amiq11.github.io/sw-one-registration/register.html for easier testing.

steps to repro:
1. open https://amiq11.github.io/sw-one-registration/register.html
2. open DevTools' console
3. click unregister button just in case
4. click register button
5. Using stable, you can see a warning "Event handler of 'install' event must be added on the initial evaluation of worker script." though you cannot when including the patch.

If you have something wrong, please let me know.
Thanks!

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/bdf22b26c6d9a10f2c8bfc956f59b6c482663d8e

commit bdf22b26c6d9a10f2c8bfc956f59b6c482663d8e
Author: shimazu <shimazu@chromium.org>
Date: Thu Sep 15 06:35:45 2016

Revert "[DevTools] Make log agent optional."

This reverts commit d0125b8fc7acf574d14b7e0dad739d814c8593c5.

Reason:
Warnings from ServiceWorkerGlobalScope doesn't show up in devtools' console

BUG= 594160 
TBR=eostroukhov, pfeldman

Review-Url: https://codereview.chromium.org/2344813002
Cr-Commit-Position: refs/heads/master@{#418793}

[modify] https://crrev.com/bdf22b26c6d9a10f2c8bfc956f59b6c482663d8e/third_party/WebKit/Source/devtools/front_end/sdk/ConsoleModel.js
[modify] https://crrev.com/bdf22b26c6d9a10f2c8bfc956f59b6c482663d8e/third_party/WebKit/Source/devtools/front_end/sdk/TargetManager.js

Updating priority/owner as the revert landed. The remaining work is to add the warning to all events, not just install/activate.

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e0f27b9808ea6a7333669e2404be671fd4231d00

commit e0f27b9808ea6a7333669e2404be671fd4231d00
Author: eostroukhov <eostroukhov@chromium.org>
Date: Fri Sep 16 22:31:03 2016

[Devtools] Add capability for the log domain

BUG= 594160 

Review-Url: https://codereview.chromium.org/2345873002
Cr-Commit-Position: refs/heads/master@{#419304}

[modify] https://crrev.com/e0f27b9808ea6a7333669e2404be671fd4231d00/third_party/WebKit/LayoutTests/http/tests/inspector/workspace-test.js
[modify] https://crrev.com/e0f27b9808ea6a7333669e2404be671fd4231d00/third_party/WebKit/Source/devtools/front_end/main/Main.js
[modify] https://crrev.com/e0f27b9808ea6a7333669e2404be671fd4231d00/third_party/WebKit/Source/devtools/front_end/sdk/ConsoleModel.js
[modify] https://crrev.com/e0f27b9808ea6a7333669e2404be671fd4231d00/third_party/WebKit/Source/devtools/front_end/sdk/ServiceWorkerManager.js
[modify] https://crrev.com/e0f27b9808ea6a7333669e2404be671fd4231d00/third_party/WebKit/Source/devtools/front_end/sdk/Target.js
[modify] https://crrev.com/e0f27b9808ea6a7333669e2404be671fd4231d00/third_party/WebKit/Source/devtools/front_end/sdk/TargetManager.js
[modify] https://crrev.com/e0f27b9808ea6a7333669e2404be671fd4231d00/third_party/WebKit/Source/devtools/front_end/sdk/WorkerManager.js

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e14cea541b991c7907123395ea663f39eea7dcfa

commit e14cea541b991c7907123395ea663f39eea7dcfa
Author: shimazu <shimazu@chromium.org>
Date: Wed Sep 21 03:46:24 2016

ServiceWorker: Shows a warning when adding event handlers asynchronously

As shown at 15 on [1], "The user agents are encouraged to show a warning that
the event listeners must be added on the very first evaluation of the worker
script.". This patch adds warning messages for the events.

[1]: https://w3c.github.io/ServiceWorker/spec/service_worker/index.html#run-service-worker-algorithm

BUG= 594160 

Review-Url: https://codereview.chromium.org/2319563003
Cr-Commit-Position: refs/heads/master@{#419967}

[add] https://crrev.com/e14cea541b991c7907123395ea663f39eea7dcfa/third_party/WebKit/LayoutTests/http/tests/inspector/service-workers/lazy-addeventlisteners-expected.txt
[add] https://crrev.com/e14cea541b991c7907123395ea663f39eea7dcfa/third_party/WebKit/LayoutTests/http/tests/inspector/service-workers/lazy-addeventlisteners.html
[add] https://crrev.com/e14cea541b991c7907123395ea663f39eea7dcfa/third_party/WebKit/LayoutTests/http/tests/inspector/service-workers/resources/service-worker-lazy-addeventlistener.js
[modify] https://crrev.com/e14cea541b991c7907123395ea663f39eea7dcfa/third_party/WebKit/Source/modules/serviceworkers/ServiceWorkerGlobalScope.cpp