Add task location information to WorkerThread

Add task location information to WorkerThread

BUG=450977
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=190594

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=190668

[Sami, 2015-02-19 20:03:52]

skyostil@chromium.org changed reviewers:
+ haraken@chromium.org

[Sami, 2015-02-19 20:04:06]

Last one in the series, I promise :)

[haraken, 2015-02-19 23:07:28]

LGTM

[Sami, 2015-02-20 16:27:50]

The CQ bit was checked by skyostil@chromium.org

[commit-bot: I haz the power, 2015-02-20 16:28:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/899923004/20001

[commit-bot: I haz the power, 2015-02-20 22:27:40]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=190594

[Ken Russell (switch to Gerrit), 2015-02-20 23:02:49]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/922023003/ by kbr@chromium.org.

The reason for reverting is: Collided with a revert of dependent patch
https://codereview.chromium.org/941403002 .
.

[Sami, 2015-02-23 15:25:49]

Thanks Ken, relanding.

[Sami, 2015-02-23 15:26:01]

The CQ bit was checked by skyostil@chromium.org

[commit-bot: I haz the power, 2015-02-23 15:26:38]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/899923004/20001

[commit-bot: I haz the power, 2015-02-23 15:27:17]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-02-23 15:27:19]

Failed to apply patch for Source/core/workers/WorkerThread.cpp:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file Source/core/workers/WorkerThread.cpp
  Hunk #3 FAILED at 423.
  1 out of 5 hunks FAILED -- saving rejects to file
Source/core/workers/WorkerThread.cpp.rej

Patch:       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
566ce066c9b35978c3e8b5e30190d1bd9c522989..69d26a5b8871f1dc5d0e94ae47a0381e0592f315
100644
--- a/Source/core/workers/WorkerThread.cpp
+++ b/Source/core/workers/WorkerThread.cpp
@@ -170,7 +170,7 @@ public:
         // Now queue the task as a cancellable one.
         OwnPtr<WorkerThreadCancelableTask> task =
WorkerThreadCancelableTask::create(bind(&WorkerSharedTimer::OnTimeout, this));
         m_lastQueuedTask = task->createWeakPtr();
-        m_workerThread->postDelayedTask(task.release(), delay);
+        m_workerThread->postDelayedTask(FROM_HERE, task.release(), delay);
     }
 
     virtual void stop()
@@ -352,7 +352,7 @@ void WorkerThread::initialize()
 
     postInitialize();
 
-    postDelayedTask(createSameThreadTask(&WorkerThread::idleHandler, this),
kShortIdleHandlerDelayMs);
+    postDelayedTask(FROM_HERE, createSameThreadTask(&WorkerThread::idleHandler,
this), kShortIdleHandlerDelayMs);
 }
 
 void WorkerThread::cleanup()
@@ -423,7 +423,7 @@ public:
 
         // Stick a shutdown command at the end of the queue, so that we deal
         // with all the cleanup tasks the databases post first.
-        workerGlobalScope->postTask(WorkerThreadShutdownFinishTask::create());
+        workerGlobalScope->postTask(FROM_HERE,
WorkerThreadShutdownFinishTask::create());
     }
 
     virtual bool isCleanupTask() const { return true; }
@@ -474,7 +474,7 @@ void WorkerThread::stopInternal()
     m_workerGlobalScope->script()->scheduleExecutionTermination();
    
InspectorInstrumentation::didKillAllExecutionContextTasks(m_workerGlobalScope.get());
     m_debuggerMessageQueue.kill();
-    postTask(WorkerThreadShutdownStartTask::create());
+    postTask(FROM_HERE, WorkerThreadShutdownStartTask::create());
 }
 
 void WorkerThread::didStartRunLoop()
@@ -519,23 +519,23 @@ void WorkerThread::idleHandler()
             delay = kShortIdleHandlerDelayMs;
     }
 
-    postDelayedTask(createSameThreadTask(&WorkerThread::idleHandler, this),
delay);
+    postDelayedTask(FROM_HERE, createSameThreadTask(&WorkerThread::idleHandler,
this), delay);
 }
 
-void WorkerThread::postTask(PassOwnPtr<ExecutionContextTask> task)
+void WorkerThread::postTask(const WebTraceLocation& location,
PassOwnPtr<ExecutionContextTask> task)
 {
-    m_thread->postTask(FROM_HERE, WorkerThreadTask::create(*this, task,
true).leakPtr());
+    m_thread->postTask(location, WorkerThreadTask::create(*this, task,
true).leakPtr());
 }
 
-void WorkerThread::postDelayedTask(PassOwnPtr<ExecutionContextTask> task, long
long delayMs)
+void WorkerThread::postDelayedTask(const WebTraceLocation& location,
PassOwnPtr<ExecutionContextTask> task, long long delayMs)
 {
-    m_thread->postDelayedTask(FROM_HERE, WorkerThreadTask::create(*this, task,
true).leakPtr(), delayMs);
+    m_thread->postDelayedTask(location, WorkerThreadTask::create(*this, task,
true).leakPtr(), delayMs);
 }
 
-void WorkerThread::postDebuggerTask(PassOwnPtr<ExecutionContextTask> task)
+void WorkerThread::postDebuggerTask(const WebTraceLocation& location,
PassOwnPtr<ExecutionContextTask> task)
 {
     m_debuggerMessageQueue.append(WorkerThreadTask::create(*this, task,
false));
-    postTask(RunDebuggerQueueTask::create(this));
+    postTask(location, RunDebuggerQueueTask::create(this));
 }
 
 MessageQueueWaitResult WorkerThread::runDebuggerTask(WaitMode waitMode)

[commit-bot: I haz the power, 2015-02-23 15:27:27]

The CQ bit was unchecked by commit-bot@chromium.org

[Sami, 2015-02-23 15:35:30]

New patchsets have been uploaded after l-g-t-m from haraken@chromium.org

[Sami, 2015-02-23 15:35:39]

The CQ bit was checked by skyostil@chromium.org

[commit-bot: I haz the power, 2015-02-23 15:36:34]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/899923004/40001

[commit-bot: I haz the power, 2015-02-23 17:07:41]

Committed patchset #3 (id:40001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=190668