Chromium Code Reviews| Index: cc/raster/task_graph_runner_perftest.cc |
| diff --git a/cc/raster/task_graph_runner_perftest.cc b/cc/raster/task_graph_runner_perftest.cc |
| index a825f1b373ab7fd3d33d318462f0911857041908..6b9b309443e40340bfa1687dff4a69a8c86ef923 100644 |
| --- a/cc/raster/task_graph_runner_perftest.cc |
| +++ b/cc/raster/task_graph_runner_perftest.cc |
| @@ -272,6 +272,12 @@ class TaskGraphRunnerPerfTest : public testing::Test { |
| 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
ericrk
2016/05/19 23:19:37
In that case, would auto& be more appropriate - th
prashant.n
2016/05/20 11:48:52
Now I understood why vmpstr added comment. Thank y
|
| + // Instead of calling DidComplete(), call Reset() as same task would be |
| + // reused as if new. |
| + task->state().Reset(); |
| + } |
| + |
| return completed_tasks->size(); |
| } |