Make AbstractWorker a ContextLifecycleObserver rather than an ActiveDOMObject

third_party/WebKit/Source/core/workers/InProcessWorkerBase.cpp

Index: third_party/WebKit/Source/core/workers/InProcessWorkerBase.cpp
diff --git a/third_party/WebKit/Source/core/workers/InProcessWorkerBase.cpp b/third_party/WebKit/Source/core/workers/InProcessWorkerBase.cpp
index 7fc496960bebaa06b2fff17c0e1f9429162ae43d..202ef70ace3dffe1cbffb7e78f0aefe9aa132c5c 100644
--- a/third_party/WebKit/Source/core/workers/InProcessWorkerBase.cpp
+++ b/third_party/WebKit/Source/core/workers/InProcessWorkerBase.cpp
@@ -44,8 +44,6 @@ void InProcessWorkerBase::postMessage(ExecutionContext* context, PassRefPtr<Seri
 
 bool InProcessWorkerBase::initialize(ExecutionContext* context, const String& url, ExceptionState& exceptionState)
 {
-    suspendIfNeeded();

[kinuko, 2016/02/29 09:33:51]

We no longer need this?

[haraken, 2016/02/29 10:52:47]

No. Actually we didn't need this even before this CL since InProcessWorkerBase
doesn't implement suspend/resume. (Accurately speaking, it was needed just to
avoid hitting an assert in ActiveDOMObject.h.)

-
     KURL scriptURL = resolveURL(url, exceptionState);
     if (scriptURL.isEmpty())
         return false;
@@ -69,7 +67,7 @@ void InProcessWorkerBase::terminate()
         m_contextProxy->terminateWorkerGlobalScope();
 }
 
-void InProcessWorkerBase::stop()
+void InProcessWorkerBase::contextDestroyed()
 {
     terminate();
 }