Remove ActiveDOMObject::stop()

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

Index: third_party/WebKit/Source/core/workers/WorkerThread.cpp
diff --git a/third_party/WebKit/Source/core/workers/WorkerThread.cpp b/third_party/WebKit/Source/core/workers/WorkerThread.cpp
index 0d0b08c7ec2b9f9077adf6e3b742f506d47af537..f03dfb3508ee3a50d82574df321b0017c5fc9281 100644
--- a/third_party/WebKit/Source/core/workers/WorkerThread.cpp
+++ b/third_party/WebKit/Source/core/workers/WorkerThread.cpp
@@ -552,12 +552,13 @@ void WorkerThread::prepareForShutdownOnWorkerThread() {
   m_inspectorTaskRunner->kill();
   workerReportingProxy().willDestroyWorkerGlobalScope();
   InspectorInstrumentation::allAsyncTasksCanceled(globalScope());
+
+  globalScope()->notifyContextDestroyed();

[haraken, 2016/10/04 14:59:58]

This is a just-in-case change. I moved notifyContextDestroyed() here to call it
before we start any disruptive operations.

I want to make the same change to Document::notifyContextDestroyed() but it
needs a couple of more work. So I added a TODO.

   globalScope()->dispose();
   if (m_workerInspectorController) {
     m_workerInspectorController->dispose();
     m_workerInspectorController.clear();
   }
-  globalScope()->notifyContextDestroyed();
   m_consoleMessageStorage.clear();
   workerBackingThread().backingThread().removeTaskObserver(
       m_microtaskRunner.get());