Remove the concept of a cleanup task

Source/platform/WebThreadSupportingGC.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "config.h"
 #include "platform/WebThreadSupportingGC.h"
 
 #include "platform/heap/SafePoint.h"
+#include "public/platform/WebScheduler.h"
 #include "wtf/Threading.h"
 
 namespace blink {
 
 PassOwnPtr<WebThreadSupportingGC> WebThreadSupportingGC::create(const char* name)
 {
 #if ENABLE(ASSERT)
     WTF::willCreateThread();
 #endif
     return adoptPtr(new WebThreadSupportingGC(name));
 }
 
 WebThreadSupportingGC::WebThreadSupportingGC(const char* name)
     : m_thread(adoptPtr(Platform::current()->createThread(name)))
 {
 }
 
 WebThreadSupportingGC::~WebThreadSupportingGC()
 {
     if (ThreadState::current()) {
         // WebThread's destructor blocks until all the tasks are processed.
         SafePointScope scope(ThreadState::HeapPointersOnStack);
         m_thread.clear();
     }
 }
 
-void WebThreadSupportingGC::attachGC()
+void WebThreadSupportingGC::initialize()
 {
     m_pendingGCRunner = adoptPtr(new PendingGCRunner);
     m_messageLoopInterruptor = adoptPtr(new MessageLoopInterruptor(&platformThread()));
     platformThread().addTaskObserver(m_pendingGCRunner.get());
     ThreadState::attach();
     ThreadState::current()->addInterruptor(m_messageLoopInterruptor.get());
 }
 
-void WebThreadSupportingGC::detachGC()
+void WebThreadSupportingGC::shutdown()
 {
     ThreadState::current()->removeInterruptor(m_messageLoopInterruptor.get());
     ThreadState::detach();
     platformThread().removeTaskObserver(m_pendingGCRunner.get());
     m_pendingGCRunner = nullptr;
     m_messageLoopInterruptor = nullptr;
+
+    // Ensure no posted tasks will run from this point on.
+    platformThread().scheduler()->shutdown();

[haraken, 2015/05/09 15:54:36]

Just to confirm: Not all threads in Blink are using WebThreadSupportingGC.
Threads that don't need a GC are using WebThread (not WebThreadSupportingGC).
For those threads, are we calling platformThread().scheduler()->shutdown() as
well?

[Sami, 2015/05/11 10:35:32]

Right, that was my understanding as well.
So far worker threads are the only ones that I know of needing special handling
for task execution during shutdown (i.e., calling |close| must prevent any
further tasks from running according to the html spec). The other utility
threads don't have this constraint, so not calling shutdown there is fine.

 }
 
 } // namespace blink