BrowserTest for delivering push message to service worker.

chrome/test/data/push_messaging/service_worker.js

Index: chrome/test/data/push_messaging/service_worker.js
diff --git a/chrome/test/data/push_messaging/service_worker.js b/chrome/test/data/push_messaging/service_worker.js
index 116b355671ff082a20266841e8ebf2d2a6726a1a..e146220e3dfcc233a2327d22fd165e22f769a327 100644
--- a/chrome/test/data/push_messaging/service_worker.js
+++ b/chrome/test/data/push_messaging/service_worker.js
@@ -2,6 +2,28 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-// Empty service worker script.
+this.onmessage = function(event) {

[Peter Beverloo, 2014/10/24 14:54:58]

This doesn't seem to be used for anything other than your manual testing?

[Michael van Ouwerkerk, 2014/10/27 15:14:30]

Deleted.

+  console.log(event);
+  console.log(event.data);
+  sendMessageToClients('message', 'Message from service worker');
+};
 
-// TODO(mvanouwerkerk): Add test coverage for push event delivery.
+this.onpush = function(event) {
+  console.log(event);
+  console.log(event.data);

[Peter Beverloo, 2014/10/24 14:54:58]

nit: are the console.log() calls here necessary for the test to pass? elsewhere
too.

[Michael van Ouwerkerk, 2014/10/27 15:14:30]

Deleted.

+  sendMessageToClients('push', event.data);
+};
+
+function sendMessageToClients(type, data) {
+  var message = JSON.stringify({
+    'type': type,
+    'data': data
+  });
+  this.clients.getAll().then(function(clients) {

[Peter Beverloo, 2014/10/24 14:54:58]

|this| is a bit odd here -- normally |this| would refer to the function scope,
but clients is defined on the global scope. Just remove |this.|?

[Michael van Ouwerkerk, 2014/10/27 15:14:30]

Done. I think it's perfectly normal for the execution context to be the global
scope, but, yeah, this was redundant here.

+    clients.forEach(function(client) {
+      client.postMessage(message);
+    });
+  }, function(error) {
+    console.log(error);
+  });
+}