Move media library AutoTaskRunner to base and rename ScopedTaskRunner.

base/task_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/memory/ref_counted.h"
 #include "base/task.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace {
 
@@ skipping 31 matching lines @@
 
 TEST(TaskTest, TestCancelInDestructor) {
   // Intentionally not using a scoped_refptr for cancel_in_destructor.
   CancelInDestructor* cancel_in_destructor = new CancelInDestructor();
   cancel_in_destructor->Start();
   CancelableTask* cancelable_task = cancel_in_destructor->cancelable_task();
   ASSERT_TRUE(cancelable_task);
   delete cancelable_task;
 }
 
+class DoneTask : public Task {
+ public:
+  DoneTask(int* run_count, bool* was_deleted)
+      : run_count_(run_count), was_deleted_(was_deleted) {}

[awong, 2011/05/26 19:01:01]

Since you had to break for the initializer list, the {} shoudl be on 2 lines.

DoneTask(int* run_count, bool* was_deleted)
    : run_count_(run_count), was_deleted_(was_deleted) {
}

[Wez, 2011/05/26 20:53:46]

Done.

+  ~DoneTask() {

[awong, 2011/05/26 19:01:01]

virtual

[Wez, 2011/05/26 20:53:46]

Done.

+    *was_deleted_ = true;
+  }
+
+  virtual void Run() {
+    ++(*run_count_);
+  }
+
+  int* run_count_;
+  bool* was_deleted_;
+};
+
+TEST(TaskTest, TestScopedTaskRunnerExitScope) {
+  int run_count = 0;
+  bool was_deleted = false;
+  {
+    base::ScopedTaskRunner runner(new DoneTask(&run_count, &was_deleted));
+  }
+  ASSERT_EQ(1, run_count);

[awong, 2011/05/26 19:01:01]

Use EXPECT_* not ASSERT_* here and in the other new tests. These expectations
seem to still be valid even if the previous failed, so EXPECT is more normal.

[Wez, 2011/05/26 20:53:46]

Done.

+  ASSERT_EQ(true, was_deleted);

[awong, 2011/05/26 19:01:01]

_TRUE, not _EQ

[Wez, 2011/05/26 20:53:46]

Done.

+}
+
+TEST(TaskTest, TestScopedTaskRunnerRelease) {
+  int run_count = 0;
+  bool was_deleted = false;
+  {
+    base::ScopedTaskRunner runner(new DoneTask(&run_count, &was_deleted));
+    delete runner.Release();
+    ASSERT_EQ(true, was_deleted);

[awong, 2011/05/26 19:01:01]

ASSERT_TRUE

[Wez, 2011/05/26 20:53:46]

Done.

+  }
+  ASSERT_EQ(0, run_count);
+}
+
+TEST(TaskTest, TestScopedTaskRunnerManualRun) {
+  int run_count = 0;
+  Task* done_task = NULL;
+  bool was_deleted = false;
+  {
+    base::ScopedTaskRunner runner(new DoneTask(&run_count, &was_deleted));
+    done_task = runner.Release();
+    ASSERT_TRUE(NULL != done_task);

[awong, 2011/05/26 19:01:01]

ASSERT_EQ?

[Wez, 2011/05/26 20:53:46]

Would need to be ASSERT_NE(), which gets confused about the type of NULL. :(

+    ASSERT_EQ(false, was_deleted);
+    done_task->Run();
+    ASSERT_EQ(false, was_deleted);
+  }
+  ASSERT_EQ(1, run_count);
+  delete done_task;
+  ASSERT_EQ(true, was_deleted);
+}
+
 }  // namespace