workers: Change how debugger-tasks are posted to the 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 b7e1c004f8659557021be89208b453a8a8b999b8..e43ba015109b85d44c666cd947274212f2cc4b27 100644
--- a/Source/core/workers/WorkerThread.cpp
+++ b/Source/core/workers/WorkerThread.cpp
@@ -249,24 +249,6 @@ private:
     bool m_isInstrumented;
 };
 
-class RunDebuggerQueueTask final : public ExecutionContextTask {
-public:
-    static PassOwnPtr<RunDebuggerQueueTask> create(WorkerThread* thread)
-    {
-        return adoptPtr(new RunDebuggerQueueTask(thread));
-    }
-    virtual void performTask(ExecutionContext* context) override
-    {
-        ASSERT(context->isWorkerGlobalScope());
-        m_thread->runDebuggerTask(WorkerThread::DontWaitForMessage);
-    }
-
-private:
-    explicit RunDebuggerQueueTask(WorkerThread* thread) : m_thread(thread) { }
-
-    WorkerThread* m_thread;
-};
-
 WorkerThread::WorkerThread(PassRefPtr<WorkerLoaderProxy> workerLoaderProxy, WorkerReportingProxy& workerReportingProxy)
     : m_started(false)
     , m_terminated(false)
@@ -552,10 +534,9 @@ void WorkerThread::terminateV8Execution()
     v8::V8::TerminateExecution(m_isolate);
 }
 
-void WorkerThread::postDebuggerTask(const WebTraceLocation& location, PassOwnPtr<ExecutionContextTask> task)
+void WorkerThread::appendDebuggerTask(PassOwnPtr<WebThread::Task> task)

[kinuko, 2015/05/18 15:19:49]

Assuming that we're also going to remove willEnterNestedLoop etc from this
class, it feels that these two methods (append/run debugger task) and
m_debuggerMessageQueue could be possibly also moved out of this class, so we
could centralize these debugger-specific post tasking logic into one place--
could it be right?

[sadrul, 2015/05/19 03:50:47]

That's an interesting idea. pfeldman@/yurus@: what do you think? Is this
something that could go with https://codereview.chromium.org/1133903005/?

 {
-    m_debuggerMessageQueue.append(WorkerThreadTask::create(*this, task, false));
-    postTask(location, RunDebuggerQueueTask::create(this));
+    m_debuggerMessageQueue.append(task);
 }
 
 MessageQueueWaitResult WorkerThread::runDebuggerTask(WaitMode waitMode)