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

Unified Diff: third_party/WebKit/Source/platform/scheduler/base/task_queue_impl.cc

Issue 2420303002: Remove usage of FOR_EACH_OBSERVER macro in platform/scheduler (Closed)
Patch Set: Created 4 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | third_party/WebKit/Source/platform/scheduler/base/task_queue_manager.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/platform/scheduler/base/task_queue_impl.cc
diff --git a/third_party/WebKit/Source/platform/scheduler/base/task_queue_impl.cc b/third_party/WebKit/Source/platform/scheduler/base/task_queue_impl.cc
index 0785e8d223049ce92bd458346d1d60784a432023..1ed68f4ad32617bab87ea9599bc6a0e8b18f7eb5 100644
--- a/third_party/WebKit/Source/platform/scheduler/base/task_queue_impl.cc
+++ b/third_party/WebKit/Source/platform/scheduler/base/task_queue_impl.cc
@@ -507,17 +507,15 @@ void TaskQueueImpl::NotifyWillProcessTask(
DCHECK(should_notify_observers_);
if (main_thread_only().blame_context)
main_thread_only().blame_context->Enter();
- FOR_EACH_OBSERVER(base::MessageLoop::TaskObserver,
- main_thread_only().task_observers,
- WillProcessTask(pending_task));
+ for (auto& observer : main_thread_only().task_observers)
+ observer.WillProcessTask(pending_task);
}
void TaskQueueImpl::NotifyDidProcessTask(
const base::PendingTask& pending_task) {
DCHECK(should_notify_observers_);
- FOR_EACH_OBSERVER(base::MessageLoop::TaskObserver,
- main_thread_only().task_observers,
- DidProcessTask(pending_task));
+ for (auto& observer : main_thread_only().task_observers)
+ observer.DidProcessTask(pending_task);
if (main_thread_only().blame_context)
main_thread_only().blame_context->Leave();
}
« no previous file with comments | « no previous file | third_party/WebKit/Source/platform/scheduler/base/task_queue_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698