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

Side by Side Diff: cc/resources/task_graph_runner.cc

Issue 638353002: [C++11 Allowed Features] Declares a type-safe null pointer converting from NULL to nullptr in src/… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Formating. Created 6 years, 2 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "cc/resources/task_graph_runner.h" 5 #include "cc/resources/task_graph_runner.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 8
9 #include "base/debug/trace_event.h" 9 #include "base/debug/trace_event.h"
10 #include "base/strings/stringprintf.h" 10 #include "base/strings/stringprintf.h"
11 #include "base/threading/thread_restrictions.h" 11 #include "base/threading/thread_restrictions.h"
12 12
13 namespace cc { 13 namespace cc {
14 namespace { 14 namespace {
15 15
16 // Helper class for iterating over all dependents of a task. 16 // Helper class for iterating over all dependents of a task.
17 class DependentIterator { 17 class DependentIterator {
18 public: 18 public:
19 DependentIterator(TaskGraph* graph, const Task* task) 19 DependentIterator(TaskGraph* graph, const Task* task)
20 : graph_(graph), 20 : graph_(graph),
21 task_(task), 21 task_(task),
22 current_index_(static_cast<size_t>(-1)), 22 current_index_(static_cast<size_t>(-1)),
23 current_node_(NULL) { 23 current_node_(nullptr) {
24 ++(*this); 24 ++(*this);
25 } 25 }
26 26
27 TaskGraph::Node& operator->() const { 27 TaskGraph::Node& operator->() const {
28 DCHECK_LT(current_index_, graph_->edges.size()); 28 DCHECK_LT(current_index_, graph_->edges.size());
29 DCHECK_EQ(graph_->edges[current_index_].task, task_); 29 DCHECK_EQ(graph_->edges[current_index_].task, task_);
30 DCHECK(current_node_); 30 DCHECK(current_node_);
31 return *current_node_; 31 return *current_node_;
32 } 32 }
33 33
(...skipping 437 matching lines...) Expand 10 before | Expand all | Expand 10 after
471 471
472 // Finally add task to |completed_tasks_|. 472 // Finally add task to |completed_tasks_|.
473 task_namespace->completed_tasks.push_back(task); 473 task_namespace->completed_tasks.push_back(task);
474 474
475 // If namespace has finished running all tasks, wake up origin thread. 475 // If namespace has finished running all tasks, wake up origin thread.
476 if (HasFinishedRunningTasksInNamespace(task_namespace)) 476 if (HasFinishedRunningTasksInNamespace(task_namespace))
477 has_namespaces_with_finished_running_tasks_cv_.Signal(); 477 has_namespaces_with_finished_running_tasks_cv_.Signal();
478 } 478 }
479 479
480 } // namespace cc 480 } // namespace cc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698