cc: Remove ScheduleOnOriginThread() and CompleteOnOriginThread().

cc/raster/task_graph_runner_perftest.cc

 // Copyright 2014 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 <stddef.h>
 #include <stdint.h>
 
 #include <memory>
 #include <vector>
 
@@ skipping 254 matching lines @@
          it != top_level_tasks.end(); ++it) {
       graph->nodes.push_back(TaskGraph::Node(
           it->get(), 0u, 0u, static_cast<uint32_t>(tasks.size())));
     }
   }
 
   size_t CollectCompletedTasks(Task::Vector* completed_tasks) {
     DCHECK(completed_tasks->empty());
     task_graph_runner_->CollectCompletedTasks(namespace_token_,
                                               completed_tasks);
+    for (auto task : *completed_tasks) {

[vmpstr, 2016/05/18 19:08:34]

nit: auto* task

[prashant.n, 2016/05/19 14:32:30]

It is scoped_refptr class object, so it would not work with auto*.

[ericrk, 2016/05/19 23:19:37]

In that case, would auto& be more appropriate - the current version will
addref/release with each iteration.

[prashant.n, 2016/05/20 11:48:52]

Now I understood why vmpstr added comment. Thank you for point out this. I fixed
this and some other places.

+      // Instead of calling DidComplete(), call Reset() as same task would be
+      // reused as if new.
+      task->state().Reset();
+    }
+
     return completed_tasks->size();
   }
 
   // Test uses SynchronousTaskGraphRunner, as this implementation introduces
   // minimal additional complexity over the TaskGraphWorkQueue helpers.
   std::unique_ptr<SynchronousTaskGraphRunner> task_graph_runner_;
   NamespaceToken namespace_token_;
   LapTimer timer_;
 };
 
@@ skipping 28 matching lines @@
   RunScheduleAndExecuteTasksTest("0_1_0", 0, 1, 0);
   RunScheduleAndExecuteTasksTest("0_32_0", 0, 32, 0);
   RunScheduleAndExecuteTasksTest("2_1_0", 2, 1, 0);
   RunScheduleAndExecuteTasksTest("2_32_0", 2, 32, 0);
   RunScheduleAndExecuteTasksTest("2_1_1", 2, 1, 1);
   RunScheduleAndExecuteTasksTest("2_32_1", 2, 32, 1);
 }
 
 }  // namespace
 }  // namespace cc