Make SequencedWorkerPool a TaskRunner

base/threading/sequenced_worker_pool.cc

Index: base/threading/sequenced_worker_pool.cc
diff --git a/base/threading/sequenced_worker_pool.cc b/base/threading/sequenced_worker_pool.cc
index 35028547ae41f2d6dfbff2f13209ce7d3bc8b414..a5a5c4301077cd67473a2bc1c9cd65955045a607 100644
--- a/base/threading/sequenced_worker_pool.cc
+++ b/base/threading/sequenced_worker_pool.cc
@@ -4,7 +4,6 @@
 
 #include "base/threading/sequenced_worker_pool.h"
 
-#include "base/compiler_specific.h"
 #include "base/logging.h"
 #include "base/metrics/histogram.h"
 #include "base/stringprintf.h"
@@ -145,6 +144,20 @@ bool SequencedWorkerPool::PostSequencedWorkerTaskWithShutdownBehavior(
                         from_here, task);
 }
 
+bool SequencedWorkerPool::PostDelayedTask(
+    const tracked_objects::Location& from_here,
+    const Closure& task,
+    int64 delay_ms) {
+  // TODO(akalin): Add support for non-zero delays.
+  DLOG_IF(WARNING, delay_ms > 0) << "Ignoring delay of " << delay_ms << " ms";

[willchan no longer on Chromium, 2012/02/23 19:39:57]

I suggest NOTREACHED() instead. If this starts happening, we should know about
it and change the consumer or actually implement it.

[akalin, 2012/02/23 22:52:21]

Done.  Used DCHECK_EQ instead of NOTREACHED().  Added delegate method to skip
testing delays (since it'll hit the NOTREACHED()).

+  return PostWorkerTask(from_here, task);
+}
+
+bool SequencedWorkerPool::RunsTasksOnCurrentThread() const {
+  // TODO(akalin): Add support for this.

[willchan no longer on Chromium, 2012/02/23 19:39:57]

When is this coming? Do not land without it, or use a NOTREACHED() to make sure
no one is calling it.

[akalin, 2012/02/23 22:52:21]

Use NOTREACHED().  I'll see if I can implement this in an upcoming CL.

+  return true;
+}
+
 void SequencedWorkerPool::FlushForTesting() {
   {
     AutoLock lock(lock_);