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

Side by Side Diff: chrome/browser/content_settings/tab_specific_content_settings.cc

Issue 2438513002: Remove FOR_EACH_OBSERVER macro usage in chrome/browser/supervised_user (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 unified diff | Download patch
« no previous file with comments | « no previous file | chrome/browser/plugins/plugin_installer.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/content_settings/tab_specific_content_settings.h" 5 #include "chrome/browser/content_settings/tab_specific_content_settings.h"
6 6
7 #include <list> 7 #include <list>
8 8
9 #include "base/command_line.h" 9 #include "base/command_line.h"
10 #include "base/lazy_instance.h" 10 #include "base/lazy_instance.h"
(...skipping 96 matching lines...) Expand 10 before | Expand all | Expand 10 after
107 subresource_filter_blockage_indicated_(false), 107 subresource_filter_blockage_indicated_(false),
108 observer_(this) { 108 observer_(this) {
109 ClearContentSettingsExceptForNavigationRelatedSettings(); 109 ClearContentSettingsExceptForNavigationRelatedSettings();
110 ClearNavigationRelatedContentSettings(); 110 ClearNavigationRelatedContentSettings();
111 111
112 observer_.Add(HostContentSettingsMapFactory::GetForProfile( 112 observer_.Add(HostContentSettingsMapFactory::GetForProfile(
113 Profile::FromBrowserContext(tab->GetBrowserContext()))); 113 Profile::FromBrowserContext(tab->GetBrowserContext())));
114 } 114 }
115 115
116 TabSpecificContentSettings::~TabSpecificContentSettings() { 116 TabSpecificContentSettings::~TabSpecificContentSettings() {
117 FOR_EACH_OBSERVER( 117 for (SiteDataObserver& observer : observer_list_)
118 SiteDataObserver, observer_list_, ContentSettingsDestroyed()); 118 observer.ContentSettingsDestroyed();
119 } 119 }
120 120
121 TabSpecificContentSettings* TabSpecificContentSettings::GetForFrame( 121 TabSpecificContentSettings* TabSpecificContentSettings::GetForFrame(
122 int render_process_id, 122 int render_process_id,
123 int render_frame_id) { 123 int render_frame_id) {
124 DCHECK_CURRENTLY_ON(BrowserThread::UI); 124 DCHECK_CURRENTLY_ON(BrowserThread::UI);
125 125
126 content::RenderFrameHost* frame = content::RenderFrameHost::FromID( 126 content::RenderFrameHost* frame = content::RenderFrameHost::FromID(
127 render_process_id, render_frame_id); 127 render_process_id, render_frame_id);
128 WebContents* web_contents = WebContents::FromRenderFrameHost(frame); 128 WebContents* web_contents = WebContents::FromRenderFrameHost(frame);
(...skipping 726 matching lines...) Expand 10 before | Expand all | Expand 10 after
855 SiteDataObserver* observer) { 855 SiteDataObserver* observer) {
856 observer_list_.AddObserver(observer); 856 observer_list_.AddObserver(observer);
857 } 857 }
858 858
859 void TabSpecificContentSettings::RemoveSiteDataObserver( 859 void TabSpecificContentSettings::RemoveSiteDataObserver(
860 SiteDataObserver* observer) { 860 SiteDataObserver* observer) {
861 observer_list_.RemoveObserver(observer); 861 observer_list_.RemoveObserver(observer);
862 } 862 }
863 863
864 void TabSpecificContentSettings::NotifySiteDataObservers() { 864 void TabSpecificContentSettings::NotifySiteDataObservers() {
865 FOR_EACH_OBSERVER(SiteDataObserver, observer_list_, OnSiteDataAccessed()); 865 for (SiteDataObserver& observer : observer_list_)
866 observer.OnSiteDataAccessed();
866 } 867 }
867 868
868 void TabSpecificContentSettings::ClearGeolocationContentSettings() { 869 void TabSpecificContentSettings::ClearGeolocationContentSettings() {
869 geolocation_usages_state_.ClearStateMap(); 870 geolocation_usages_state_.ClearStateMap();
870 } 871 }
871 872
872 void TabSpecificContentSettings::ClearMidiContentSettings() { 873 void TabSpecificContentSettings::ClearMidiContentSettings() {
873 midi_usages_state_.ClearStateMap(); 874 midi_usages_state_.ClearStateMap();
874 } 875 }
875 876
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
909 static_cast<MicrophoneCameraStateFlags>( 910 static_cast<MicrophoneCameraStateFlags>(
910 TabSpecificContentSettings::MICROPHONE_ACCESSED | 911 TabSpecificContentSettings::MICROPHONE_ACCESSED |
911 TabSpecificContentSettings::MICROPHONE_BLOCKED | 912 TabSpecificContentSettings::MICROPHONE_BLOCKED |
912 TabSpecificContentSettings::CAMERA_ACCESSED | 913 TabSpecificContentSettings::CAMERA_ACCESSED |
913 TabSpecificContentSettings::CAMERA_BLOCKED); 914 TabSpecificContentSettings::CAMERA_BLOCKED);
914 OnMediaStreamPermissionSet( 915 OnMediaStreamPermissionSet(
915 web_contents()->GetLastCommittedURL(), 916 web_contents()->GetLastCommittedURL(),
916 media_blocked, 917 media_blocked,
917 std::string(), std::string(), std::string(), std::string()); 918 std::string(), std::string(), std::string(), std::string());
918 } 919 }
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/plugins/plugin_installer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698