Remove custom Task implementations in base.

base/threading/worker_pool_unittest.cc

Index: base/threading/worker_pool_unittest.cc
diff --git a/base/threading/worker_pool_unittest.cc b/base/threading/worker_pool_unittest.cc
index cd20f478044fdc42c2e89020a34cc0722c54d6e8..1ecbaf7b9ed240b2c9c323d49fd944ef1bbc4802 100644
--- a/base/threading/worker_pool_unittest.cc
+++ b/base/threading/worker_pool_unittest.cc
@@ -21,18 +21,9 @@ namespace base {
 
 namespace {
 
-class PostTaskTestTask : public Task {
- public:
-  explicit PostTaskTestTask(WaitableEvent* event) : event_(event) {
-  }
-
-  void Run() {
-    event_->Signal();
-  }
-
- private:
-  WaitableEvent* event_;
-};
+void PostTaskTestHelper(WaitableEvent* event) {

[willchan no longer on Chromium, 2011/11/23 18:15:46]

Why is this wrapper function necessary?

[dcheng, 2011/11/23 18:47:49]

Done.

+  event->Signal();
+}
 
 class PostTaskAndReplyTester
     : public base::RefCountedThreadSafe<PostTaskAndReplyTester> {
@@ -81,8 +72,12 @@ TEST_F(WorkerPoolTest, PostTask) {
   WaitableEvent test_event(false, false);
   WaitableEvent long_test_event(false, false);
 
-  WorkerPool::PostTask(FROM_HERE, new PostTaskTestTask(&test_event), false);
-  WorkerPool::PostTask(FROM_HERE, new PostTaskTestTask(&long_test_event), true);
+  WorkerPool::PostTask(FROM_HERE,
+                       base::Bind(&PostTaskTestHelper, &test_event),
+                       false);
+  WorkerPool::PostTask(FROM_HERE,
+                       base::Bind(&PostTaskTestHelper, &long_test_event),
+                       true);
 
   test_event.Wait();
   long_test_event.Wait();