Oilpan: We need to collect all garbage before shutting down the worker thread

Source/core/workers/WorkerThread.cpp

Index: Source/core/workers/WorkerThread.cpp
diff --git a/Source/core/workers/WorkerThread.cpp b/Source/core/workers/WorkerThread.cpp
index 7d6efc7e0b26cba91178b60b26e619b6bdb76e67..0624c50c63529021112dfe9d5eebfff6390b19d3 100644
--- a/Source/core/workers/WorkerThread.cpp
+++ b/Source/core/workers/WorkerThread.cpp
@@ -137,23 +137,21 @@ void WorkerThread::workerThread()
 
     ThreadIdentifier threadID = m_threadID;
 
+#ifdef !ENABLE_OILPAN

[zerny-chromium, 2014/03/05 09:43:23]

#if !ENABLE_OILPAN

As seen in the build failure.

[kouhei (in TOK), 2014/03/05 09:47:41]

or
#if !ENABLE(OILPAN)
to be correct.

     ASSERT(m_workerGlobalScope->hasOneRef());
+#endif
 
     // The below assignment will destroy the context, which will in turn notify messaging proxy.
     // We cannot let any objects survive past thread exit, because no other thread will run GC or otherwise destroy them.
     m_workerGlobalScope = nullptr;
 
-    // Cleanup thread heap which causes all objects to be finalized.
-    // After this call thread heap must be empty.
-    ThreadState::current()->cleanup();
+    ThreadState::detach();
 
     // Clean up PlatformThreadData before WTF::WTFThreadData goes away!
     PlatformThreadData::current().destroy();
 
     // The thread object may be already destroyed from notification now, don't try to access "this".
     detachThread(threadID);
-
-    ThreadState::detach();
 }
 
 void WorkerThread::runEventLoop()