[Worker] Call prepareForShutdown ASAP when terminated forcibly

third_party/WebKit/Source/core/loader/ThreadableLoaderTest.cpp

Index: third_party/WebKit/Source/core/loader/ThreadableLoaderTest.cpp
diff --git a/third_party/WebKit/Source/core/loader/ThreadableLoaderTest.cpp b/third_party/WebKit/Source/core/loader/ThreadableLoaderTest.cpp
index 1bd3110db62fd7c7ba77ed16b894b06683bf8405..b49170d9b2b2245a223f4114f3f9475c8b1c3376 100644
--- a/third_party/WebKit/Source/core/loader/ThreadableLoaderTest.cpp
+++ b/third_party/WebKit/Source/core/loader/ThreadableLoaderTest.cpp
@@ -256,6 +256,9 @@ public:
     void onTearDown() override
     {
         postTaskToWorkerGlobalScope(BLINK_FROM_HERE, createCrossThreadTask(&WorkerThreadableLoaderTestHelper::clearLoader, crossThreadUnretained(this)));
+        WaitableEvent event;
+        postTaskToWorkerGlobalScope(BLINK_FROM_HERE, createCrossThreadTask(&signal, crossThreadUnretained(&event)));
+        event.wait();
         m_workerThread->terminateAndWait();
 
         // Needed to clean up the things on the main thread side and
@@ -328,6 +331,11 @@ private:
         m_workerThread->postTask(location, std::move(task));
     }
 
+    static void signal(WaitableEvent* event)

[nhiroki, 2016/09/06 05:06:40]

We could remove this as follows?

postTaskToWorkerGlobalScope(BLINK_FROM_HERE,
createCrossThreadTask(&WaitableEvent::signal, crossThreadUnretained(&event)));

[yhirano, 2016/09/06 05:38:03]

It's a good idea, thank you!

+    {
+        event->signal();
+    }
+
     RefPtr<SecurityOrigin> m_securityOrigin;
     std::unique_ptr<MockWorkerReportingProxy> m_mockWorkerReportingProxy;
     std::unique_ptr<WorkerThreadForTest> m_workerThread;