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

Unified Diff: chrome/browser/browsing_data/cookies_tree_model.cc

Issue 2423883005: Remove FOR_EACH_OBSERVER macro usage in chrome/browser/android (Closed)
Patch Set: explicit types 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
Index: chrome/browser/browsing_data/cookies_tree_model.cc
diff --git a/chrome/browser/browsing_data/cookies_tree_model.cc b/chrome/browser/browsing_data/cookies_tree_model.cc
index bfcf2fb8134b18786bd93347f6bb85882bcc2093..bae258ecf6f48f80bb9c82ef529d8fdaa6c60c62 100644
--- a/chrome/browser/browsing_data/cookies_tree_model.cc
+++ b/chrome/browser/browsing_data/cookies_tree_model.cc
@@ -1601,9 +1601,8 @@ void CookiesTreeModel::RecordBatchSeen() {
void CookiesTreeModel::NotifyObserverBeginBatch() {
// Only notify the model once if we're batching in a nested manner.
if (batches_started_++ == 0) {
- FOR_EACH_OBSERVER(Observer,
- cookies_observer_list_,
- TreeModelBeginBatch(this));
+ for (Observer& observer : cookies_observer_list_)
+ observer.TreeModelBeginBatch(this);
}
}
@@ -1617,9 +1616,8 @@ void CookiesTreeModel::MaybeNotifyBatchesEnded() {
// called in a nested manner.
if (batches_ended_ == batches_started_ &&
batches_seen_ == batches_expected_) {
- FOR_EACH_OBSERVER(Observer,
- cookies_observer_list_,
- TreeModelEndBatch(this));
+ for (Observer& observer : cookies_observer_list_)
+ observer.TreeModelEndBatch(this);
SetBatchExpectation(0, true);
}
}

Powered by Google App Engine
This is Rietveld 408576698