Prevent a service worker from keeping itself alive by self postMessage.

content/browser/service_worker/service_worker_dispatcher_host.h

Index: content/browser/service_worker/service_worker_dispatcher_host.h
diff --git a/content/browser/service_worker/service_worker_dispatcher_host.h b/content/browser/service_worker/service_worker_dispatcher_host.h
index 5037c370c96db8a3e629eaf69a8b412d80825c9f..87e16dece7f401294e9dd47c5588096d6f345717 100644
--- a/content/browser/service_worker/service_worker_dispatcher_host.h
+++ b/content/browser/service_worker/service_worker_dispatcher_host.h
@@ -13,6 +13,7 @@
 #include "base/id_map.h"
 #include "base/macros.h"
 #include "base/memory/weak_ptr.h"
+#include "base/optional.h"
 #include "base/strings/string16.h"
 #include "content/browser/service_worker/service_worker_registration_status.h"
 #include "content/common/service_worker/service_worker.mojom.h"
@@ -189,6 +190,7 @@ class CONTENT_EXPORT ServiceWorkerDispatcherHost
       const url::Origin& source_origin,
       const std::vector<int>& sent_message_ports,
       const StatusCallback& callback,
+      const base::Optional<base::TimeDelta>& timeout,

[falken, 2016/11/21 08:01:48]

super nit: would it be better for timeout to come before |callback| for
consistency with DispatchExtendableMessageEventAfterStartWorker?

[Marijn Kruisselbrink, 2016/12/01 19:54:30]

Done

       const SourceInfo& source_info);
   void DispatchExtendableMessageEventAfterStartWorker(
       scoped_refptr<ServiceWorkerVersion> worker,
@@ -196,6 +198,7 @@ class CONTENT_EXPORT ServiceWorkerDispatcherHost
       const url::Origin& source_origin,
       const std::vector<int>& sent_message_ports,
       const ExtendableMessageEventSource& source,
+      const base::Optional<base::TimeDelta>& timeout,
       const StatusCallback& callback);
   template <typename SourceInfo>
   void DidFailToDispatchExtendableMessageEvent(