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

Unified Diff: cc/raster/task_graph_work_queue.cc

Issue 1666283002: Reland - Refactor signaling in RWP (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: x++ > ++x Created 4 years, 10 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/raster/task_graph_work_queue.cc
diff --git a/cc/raster/task_graph_work_queue.cc b/cc/raster/task_graph_work_queue.cc
index 29474023c6539a17ab44c9cc8b0b0d67645dd635..f4368e1cbf5428016244766e7721da9b292942e7 100644
--- a/cc/raster/task_graph_work_queue.cc
+++ b/cc/raster/task_graph_work_queue.cc
@@ -97,9 +97,11 @@ void TaskGraphWorkQueue::ScheduleTasks(NamespaceToken token, TaskGraph* graph) {
continue;
// Skip if already running.
- if (std::find(task_namespace.running_tasks.begin(),
- task_namespace.running_tasks.end(),
- node.task) != task_namespace.running_tasks.end())
+ if (std::any_of(task_namespace.running_tasks.begin(),
+ task_namespace.running_tasks.end(),
+ [&node](const CategorizedTask& task) {
+ return task.second == node.task;
+ }))
continue;
task_namespace.ready_to_run_tasks[node.category].push_back(PrioritizedTask(
@@ -127,9 +129,11 @@ void TaskGraphWorkQueue::ScheduleTasks(NamespaceToken token, TaskGraph* graph) {
continue;
// Skip if already running.
- if (std::find(task_namespace.running_tasks.begin(),
- task_namespace.running_tasks.end(),
- node.task) != task_namespace.running_tasks.end())
+ if (std::any_of(task_namespace.running_tasks.begin(),
+ task_namespace.running_tasks.end(),
+ [&node](const CategorizedTask& task) {
+ return task.second == node.task;
+ }))
continue;
DCHECK(std::find(task_namespace.completed_tasks.begin(),
@@ -195,7 +199,8 @@ TaskGraphWorkQueue::PrioritizedTask TaskGraphWorkQueue::GetNextTaskToRun(
}
// Add task to |running_tasks|.
- task_namespace->running_tasks.push_back(task.task);
+ task_namespace->running_tasks.push_back(
+ std::make_pair(task.category, task.task));
return task;
}
@@ -205,8 +210,11 @@ void TaskGraphWorkQueue::CompleteTask(const PrioritizedTask& completed_task) {
scoped_refptr<Task> task(completed_task.task);
// Remove task from |running_tasks|.
- auto it = std::find(task_namespace->running_tasks.begin(),
- task_namespace->running_tasks.end(), task);
+ auto it = std::find_if(task_namespace->running_tasks.begin(),
+ task_namespace->running_tasks.end(),
+ [&completed_task](const CategorizedTask& task) {
+ return task.second == completed_task.task;
+ });
DCHECK(it != task_namespace->running_tasks.end());
std::swap(*it, task_namespace->running_tasks.back());
task_namespace->running_tasks.pop_back();

Powered by Google App Engine
This is Rietveld 408576698