Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(815)

Unified Diff: cc/resources/worker_pool_unittest.cc

Issue 17351017: Re-land: cc: Add raster finished signals to RasterWorkerPool. (Closed) Base URL: http://git.chromium.org/chromium/src.git@new-graph-build
Patch Set: rebase Created 7 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: cc/resources/worker_pool_unittest.cc
diff --git a/cc/resources/worker_pool_unittest.cc b/cc/resources/worker_pool_unittest.cc
index f137155789a08290a3740bc7398e65b493fa700c..e4be59f4d10b294c0a388792c430fc0fcd0cea29 100644
--- a/cc/resources/worker_pool_unittest.cc
+++ b/cc/resources/worker_pool_unittest.cc
@@ -53,7 +53,6 @@ class FakeWorkerPool : public WorkerPool {
const base::Closure& reply,
const base::Closure& dependency,
int count) {
- unsigned priority = 0u;
TaskGraph graph;
scoped_refptr<FakeWorkerPoolTaskImpl> completion_task(
@@ -61,32 +60,29 @@ class FakeWorkerPool : public WorkerPool {
base::Bind(&FakeWorkerPool::OnTasksCompleted,
base::Unretained(this)),
base::Closure()));
- scoped_ptr<GraphNode> completion_node(new GraphNode);
- completion_node->set_task(completion_task.get());
+ scoped_ptr<internal::GraphNode> completion_node(
+ new internal::GraphNode(completion_task.get(), 0u));
scoped_refptr<FakeWorkerPoolTaskImpl> dependency_task(
new FakeWorkerPoolTaskImpl(dependency, base::Closure()));
- scoped_ptr<GraphNode> dependency_node(new GraphNode);
- dependency_node->set_task(dependency_task.get());
+ scoped_ptr<internal::GraphNode> dependency_node(
+ new internal::GraphNode(dependency_task.get(), 0u));
TaskVector tasks;
for (int i = 0; i < count; ++i) {
scoped_refptr<FakeWorkerPoolTaskImpl> task(
new FakeWorkerPoolTaskImpl(callback, reply));
- scoped_ptr<GraphNode> node(new GraphNode);
- node->set_task(task.get());
+ scoped_ptr<internal::GraphNode> node(
+ new internal::GraphNode(task.get(), 0u));
node->add_dependent(completion_node.get());
completion_node->add_dependency();
dependency_node->add_dependent(node.get());
node->add_dependency();
- node->set_priority(priority++);
graph.set(task.get(), node.Pass());
tasks.push_back(task.get());
}
- completion_node->set_priority(priority++);
graph.set(completion_task.get(), completion_node.Pass());
- dependency_node->set_priority(priority++);
graph.set(dependency_task.get(), dependency_node.Pass());
scheduled_tasks_completion_.reset(new CompletionEvent);

Powered by Google App Engine
This is Rietveld 408576698