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

Unified Diff: chrome/browser/ui/app_list/start_page_service.cc

Issue 2424563002: Replace FOR_EACH_OBSERVER in c/b/ui/app_list with range-based for (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
Index: chrome/browser/ui/app_list/start_page_service.cc
diff --git a/chrome/browser/ui/app_list/start_page_service.cc b/chrome/browser/ui/app_list/start_page_service.cc
index 052e2bdb16877b7415cd26be98a3c7976c549d87..5610c5622e37c6d2b509a9ba1116b1f7c966c945 100644
--- a/chrome/browser/ui/app_list/start_page_service.cc
+++ b/chrome/browser/ui/app_list/start_page_service.cc
@@ -485,15 +485,13 @@ void StartPageService::OnSpeechResult(
speech_result_obtained_ = true;
RecordAction(UserMetricsAction("AppList_SearchedBySpeech"));
}
- FOR_EACH_OBSERVER(StartPageObserver,
- observers_,
- OnSpeechResult(query, is_final));
+ for (auto& observer : observers_)
+ observer.OnSpeechResult(query, is_final);
}
void StartPageService::OnSpeechSoundLevelChanged(int16_t level) {
- FOR_EACH_OBSERVER(StartPageObserver,
- observers_,
- OnSpeechSoundLevelChanged(level));
+ for (auto& observer : observers_)
+ observer.OnSpeechSoundLevelChanged(level);
}
void StartPageService::OnSpeechRecognitionStateChanged(
@@ -532,9 +530,8 @@ void StartPageService::OnSpeechRecognitionStateChanged(
speech_button_toggled_manually_ = false;
speech_result_obtained_ = false;
state_ = new_state;
- FOR_EACH_OBSERVER(StartPageObserver,
- observers_,
- OnSpeechRecognitionStateChanged(new_state));
+ for (auto& observer : observers_)
+ observer.OnSpeechRecognitionStateChanged(new_state);
}
void StartPageService::GetSpeechAuthParameters(std::string* auth_scope,
« no previous file with comments | « chrome/browser/ui/app_list/app_list_syncable_service.cc ('k') | chrome/browser/ui/app_list/test/fake_profile_store.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698