ServiceWorker: Issue a warning after initial execution of worker script

Source/modules/serviceworkers/ServiceWorkerGlobalScope.h

Index: Source/modules/serviceworkers/ServiceWorkerGlobalScope.h
diff --git a/Source/modules/serviceworkers/ServiceWorkerGlobalScope.h b/Source/modules/serviceworkers/ServiceWorkerGlobalScope.h
index df83c0dec1c8d21a2709e3b92692cda7743252c7..966f40237e69cad5ec665903dea64d970ce3ded8 100644
--- a/Source/modules/serviceworkers/ServiceWorkerGlobalScope.h
+++ b/Source/modules/serviceworkers/ServiceWorkerGlobalScope.h
@@ -57,6 +57,9 @@ public:
     virtual bool isServiceWorkerGlobalScope() const override { return true; }
     virtual void stopFetch() override;
 
+    // WorkerGlobalScope
+    virtual void postInitialize() override;
+
     // ServiceWorkerGlobalScope.idl
     ServiceWorkerClients* clients();
     String scope(ExecutionContext*);
@@ -71,6 +74,7 @@ public:
     void close(ExceptionState&);
 
     // EventTarget
+    virtual bool addEventListener(const AtomicString& eventType, PassRefPtr<EventListener>, bool useCapture = false) override;

[kouhei (in TOK), 2014/10/22 07:28:28]

Don't we need to the same for removeEventListener too?

[shimazu, 2014/10/22 08:33:56]

No, I think.
This is because the removed function won't be executed even if the
removeEventListener is not executed.

     virtual const AtomicString& interfaceName() const override;
 
     DEFINE_ATTRIBUTE_EVENT_LISTENER(install);
@@ -89,6 +93,7 @@ private:
     PersistentWillBeMember<ServiceWorkerClients> m_clients;
     OwnPtr<FetchManager> m_fetchManager;
     PersistentWillBeMember<CacheStorage> m_caches;
+    bool m_scriptEvaluated;
 };
 
 } // namespace blink