TaskGraphRunner refactor

cc/test/test_task_graph_runner.cc

Index: cc/test/test_task_graph_runner.cc
diff --git a/cc/test/test_task_graph_runner.cc b/cc/test/test_task_graph_runner.cc
index abc0d28902b115a2801e56e6e5fa483b5919f264..8a43622ad7a6a8ef71018648c3bcd9e1e8b09093 100644
--- a/cc/test/test_task_graph_runner.cc
+++ b/cc/test/test_task_graph_runner.cc
@@ -6,18 +6,12 @@
 
 namespace cc {
 
-TestTaskGraphRunner::TestTaskGraphRunner()
-    : worker_thread_(this, "CompositorTileWorker1") {
-  worker_thread_.Start();
+TestTaskGraphRunner::TestTaskGraphRunner() {
+  Start("TestTaskGraphRunner1", base::SimpleThread::Options());

[reveman, 2015/11/23 19:47:06]

nit: TestTaskGraphRunner, no need for the "1" suffix

Fyi, some telemetry tests depend on the name of these threads having the
"CompositorTileWorker" prefix. Pretty sure TestTaskGraphRunner is not used by
any telemetry tests so this should be fine.

[ericrk, 2015/11/23 21:49:50]

Done.

 }
 
 TestTaskGraphRunner::~TestTaskGraphRunner() {
-  TaskGraphRunner::Shutdown();
-  worker_thread_.Join();
-}
-
-void TestTaskGraphRunner::Run() {
-  TaskGraphRunner::Run();
+  Shutdown();
 }
 
 }  // namespace cc