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

Unified Diff: components/content_settings/core/browser/host_content_settings_map.cc

Issue 2440303002: Remove usage of FOR_EACH_OBSERVER macro in components/ (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: components/content_settings/core/browser/host_content_settings_map.cc
diff --git a/components/content_settings/core/browser/host_content_settings_map.cc b/components/content_settings/core/browser/host_content_settings_map.cc
index 0e3f49fea1bfc19b6cd54567d3fd05b1ff4d790f..7f82156ec5980c25f64544aa9d364aa762964704 100644
--- a/components/content_settings/core/browser/host_content_settings_map.cc
+++ b/components/content_settings/core/browser/host_content_settings_map.cc
@@ -803,12 +803,10 @@ void HostContentSettingsMap::OnContentSettingChanged(
const ContentSettingsPattern& secondary_pattern,
ContentSettingsType content_type,
std::string resource_identifier) {
- FOR_EACH_OBSERVER(content_settings::Observer,
- observers_,
- OnContentSettingChanged(primary_pattern,
- secondary_pattern,
- content_type,
- resource_identifier));
+ for (content_settings::Observer& observer : observers_) {
+ observer.OnContentSettingChanged(primary_pattern, secondary_pattern,
+ content_type, resource_identifier);
+ }
}
HostContentSettingsMap::~HostContentSettingsMap() {

Powered by Google App Engine
This is Rietveld 408576698