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

Unified Diff: ios/web/web_state/global_web_state_event_tracker.mm

Issue 2420013005: Remove usage of FOR_EACH_OBSERVER macro in ios/ (Closed)
Patch Set: typo 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 | « ios/web/active_state_manager_impl.mm ('k') | ios/web/web_state/web_state_impl.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ios/web/web_state/global_web_state_event_tracker.mm
diff --git a/ios/web/web_state/global_web_state_event_tracker.mm b/ios/web/web_state/global_web_state_event_tracker.mm
index 267b38c1046fcc41d68e98873f3ee3b635bc5c70..49ed9f9aa723f510787156c1cad00494152825d9 100644
--- a/ios/web/web_state/global_web_state_event_tracker.mm
+++ b/ios/web/web_state/global_web_state_event_tracker.mm
@@ -112,42 +112,42 @@ void GlobalWebStateEventTracker::RemoveObserver(
void GlobalWebStateEventTracker::NavigationItemsPruned(
WebState* web_state,
size_t pruned_item_count) {
- FOR_EACH_OBSERVER(GlobalWebStateObserver, observer_list_,
- NavigationItemsPruned(web_state, pruned_item_count));
+ for (auto& observer : observer_list_)
+ observer.NavigationItemsPruned(web_state, pruned_item_count);
}
void GlobalWebStateEventTracker::NavigationItemChanged(WebState* web_state) {
- FOR_EACH_OBSERVER(GlobalWebStateObserver, observer_list_,
- NavigationItemChanged(web_state));
+ for (auto& observer : observer_list_)
+ observer.NavigationItemChanged(web_state);
}
void GlobalWebStateEventTracker::NavigationItemCommitted(
WebState* web_state,
const LoadCommittedDetails& load_details) {
- FOR_EACH_OBSERVER(GlobalWebStateObserver, observer_list_,
- NavigationItemCommitted(web_state, load_details));
+ for (auto& observer : observer_list_)
+ observer.NavigationItemCommitted(web_state, load_details);
}
void GlobalWebStateEventTracker::WebStateDidStartLoading(WebState* web_state) {
- FOR_EACH_OBSERVER(GlobalWebStateObserver, observer_list_,
- WebStateDidStartLoading(web_state));
+ for (auto& observer : observer_list_)
+ observer.WebStateDidStartLoading(web_state);
}
void GlobalWebStateEventTracker::WebStateDidStopLoading(WebState* web_state) {
- FOR_EACH_OBSERVER(GlobalWebStateObserver, observer_list_,
- WebStateDidStopLoading(web_state));
+ for (auto& observer : observer_list_)
+ observer.WebStateDidStopLoading(web_state);
}
void GlobalWebStateEventTracker::PageLoaded(
WebState* web_state,
PageLoadCompletionStatus load_completion_status) {
- FOR_EACH_OBSERVER(GlobalWebStateObserver, observer_list_,
- PageLoaded(web_state, load_completion_status));
+ for (auto& observer : observer_list_)
+ observer.PageLoaded(web_state, load_completion_status);
}
void GlobalWebStateEventTracker::WebStateDestroyed(WebState* web_state) {
- FOR_EACH_OBSERVER(GlobalWebStateObserver, observer_list_,
- WebStateDestroyed(web_state));
+ for (auto& observer : observer_list_)
+ observer.WebStateDestroyed(web_state);
}
} // namespace web
« no previous file with comments | « ios/web/active_state_manager_impl.mm ('k') | ios/web/web_state/web_state_impl.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698