Move didStopWorkerRunLoop out of WorkerScriptController

Now that we no longer have real worker run loop and all we're doing
in didStopWorkerRunLoop is deleting per-worker-thread IPC dispatchers
the timing should be no longer that sensitive; we should be able to call
it somewhere after we actually stopped event loop / before we stop the thread.

BUG=571614

Review URL: https://codereview.chromium.org/1547123002

Cr-Commit-Position: refs/heads/master@{#366930}

third_party/WebKit/Source/bindings/core/v8/WorkerScriptController.cpp

@@ -129,10 +129,6 @@ void WorkerScriptController::dispose()
 
     m_world->dispose();
 
-    // The corresponding call to didStartRunLoop() is in WorkerThread::initialize().
-    // See http://webkit.org/b/83104#c14 for why this is here.
-    m_workerGlobalScope->thread()->didStopRunLoop();
-
     if (isContextInitialized())
         m_scriptState->disposePerContextData();
 }

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

@@ -295,7 +295,6 @@ void WorkerThread::initialize(PassOwnPtr<WorkerThreadStartupData> startupData)
         m_workerGlobalScope = createWorkerGlobalScope(startupData);
         m_workerGlobalScope->scriptLoaded(sourceCode.length(), cachedMetaData.get() ? cachedMetaData->size() : 0);
 
-        // The corresponding call to didStopRunLoop() is in ~WorkerScriptController().
         didStartRunLoop();
 
         // Notify proxy that a new WorkerGlobalScope has been created and started.
@@ -334,6 +333,10 @@ void WorkerThread::shutdown()
 
     workerGlobalScope()->dispose();
 
+    // This should be called after the WorkerGlobalScope's disposed (which may
+    // trigger some last-minutes cleanups) and before the thread actually stops.
+    didStopRunLoop();
+
     backingThread().removeTaskObserver(m_microtaskRunner.get());
     postTask(BLINK_FROM_HERE, createSameThreadTask(&WorkerThread::performShutdownTask, this));
 }