Remove custom Task implementations in base.

base/threading/worker_pool_unittest.cc

 // Copyright (c) 2011 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 "base/threading/worker_pool.h"
 
 #include "base/bind.h"
 #include "base/location.h"
 #include "base/message_loop.h"
 #include "base/task.h"
 #include "base/test/test_timeouts.h"
 #include "base/time.h"
 #include "base/threading/thread_checker_impl.h"
 #include "base/synchronization/waitable_event.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "testing/platform_test.h"
 
 typedef PlatformTest WorkerPoolTest;
 
 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> {
  public:
   PostTaskAndReplyTester() : finished_(false), test_event_(false, false) {
   }
 
   void RunTest() {
     ASSERT_TRUE(thread_checker_.CalledOnValidThread());
     WorkerPool::PostTaskAndReply(
@@ skipping 28 matching lines @@
   // The Impl version performs its checks even in release builds.
   ThreadCheckerImpl thread_checker_;
 };
 
 }  // namespace
 
 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();
 }
 
 TEST_F(WorkerPoolTest, PostTaskAndReply) {
   MessageLoop message_loop;
   scoped_refptr<PostTaskAndReplyTester> tester(new PostTaskAndReplyTester());
   tester->RunTest();
 
   const TimeDelta kMaxDuration =
       TimeDelta::FromMilliseconds(TestTimeouts::tiny_timeout_ms());
   TimeTicks start = TimeTicks::Now();
   while (!tester->finished() && TimeTicks::Now() - start < kMaxDuration) {
     MessageLoop::current()->RunAllPending();
   }
   EXPECT_TRUE(tester->finished());
 }
 
 }  // namespace base