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

Side by Side Diff: chrome/browser/permissions/permission_context_base.cc

Issue 1312453005: Removed Profile::GetHostContentSettingsMap (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed patch conflict Created 5 years, 3 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
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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/permissions/permission_context_base.h" 5 #include "chrome/browser/permissions/permission_context_base.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "base/prefs/pref_service.h" 8 #include "base/prefs/pref_service.h"
9 #include "chrome/browser/content_settings/host_content_settings_map_factory.h"
9 #include "chrome/browser/permissions/permission_bubble_request_impl.h" 10 #include "chrome/browser/permissions/permission_bubble_request_impl.h"
10 #include "chrome/browser/permissions/permission_context_uma_util.h" 11 #include "chrome/browser/permissions/permission_context_uma_util.h"
11 #include "chrome/browser/permissions/permission_queue_controller.h" 12 #include "chrome/browser/permissions/permission_queue_controller.h"
12 #include "chrome/browser/permissions/permission_request_id.h" 13 #include "chrome/browser/permissions/permission_request_id.h"
13 #include "chrome/browser/profiles/profile.h" 14 #include "chrome/browser/profiles/profile.h"
14 #include "chrome/browser/ui/website_settings/permission_bubble_manager.h" 15 #include "chrome/browser/ui/website_settings/permission_bubble_manager.h"
15 #include "chrome/common/pref_names.h" 16 #include "chrome/common/pref_names.h"
16 #include "components/content_settings/core/browser/host_content_settings_map.h" 17 #include "components/content_settings/core/browser/host_content_settings_map.h"
17 #include "components/content_settings/core/browser/website_settings_registry.h" 18 #include "components/content_settings/core/browser/website_settings_registry.h"
18 #include "content/public/browser/browser_thread.h" 19 #include "content/public/browser/browser_thread.h"
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
50 } 51 }
51 52
52 ContentSetting PermissionContextBase::GetPermissionStatus( 53 ContentSetting PermissionContextBase::GetPermissionStatus(
53 const GURL& requesting_origin, 54 const GURL& requesting_origin,
54 const GURL& embedding_origin) const { 55 const GURL& embedding_origin) const {
55 if (IsRestrictedToSecureOrigins() && 56 if (IsRestrictedToSecureOrigins() &&
56 !content::IsOriginSecure(requesting_origin)) { 57 !content::IsOriginSecure(requesting_origin)) {
57 return CONTENT_SETTING_BLOCK; 58 return CONTENT_SETTING_BLOCK;
58 } 59 }
59 60
60 return profile_->GetHostContentSettingsMap()->GetContentSetting( 61 return HostContentSettingsMapFactory::GetForProfile(profile_)
61 requesting_origin, embedding_origin, permission_type_, std::string()); 62 ->GetContentSetting(requesting_origin,
63 embedding_origin,
64 permission_type_,
65 std::string());
62 } 66 }
63 67
64 void PermissionContextBase::ResetPermission( 68 void PermissionContextBase::ResetPermission(
65 const GURL& requesting_origin, 69 const GURL& requesting_origin,
66 const GURL& embedding_origin) { 70 const GURL& embedding_origin) {
67 profile_->GetHostContentSettingsMap()->SetContentSetting( 71 HostContentSettingsMapFactory::GetForProfile(profile_)->SetContentSetting(
68 ContentSettingsPattern::FromURLNoWildcard(requesting_origin), 72 ContentSettingsPattern::FromURLNoWildcard(requesting_origin),
69 ContentSettingsPattern::FromURLNoWildcard(embedding_origin), 73 ContentSettingsPattern::FromURLNoWildcard(embedding_origin),
70 permission_type_, std::string(), CONTENT_SETTING_DEFAULT); 74 permission_type_, std::string(), CONTENT_SETTING_DEFAULT);
71 } 75 }
72 76
73 void PermissionContextBase::CancelPermissionRequest( 77 void PermissionContextBase::CancelPermissionRequest(
74 content::WebContents* web_contents, 78 content::WebContents* web_contents,
75 const PermissionRequestID& id) { 79 const PermissionRequestID& id) {
76 DCHECK_CURRENTLY_ON(content::BrowserThread::UI); 80 DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
77 81
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
114 } 118 }
115 119
116 if (IsRestrictedToSecureOrigins() && 120 if (IsRestrictedToSecureOrigins() &&
117 !content::IsOriginSecure(requesting_origin)) { 121 !content::IsOriginSecure(requesting_origin)) {
118 NotifyPermissionSet(id, requesting_origin, embedding_origin, callback, 122 NotifyPermissionSet(id, requesting_origin, embedding_origin, callback,
119 false /* persist */, CONTENT_SETTING_BLOCK); 123 false /* persist */, CONTENT_SETTING_BLOCK);
120 return; 124 return;
121 } 125 }
122 126
123 ContentSetting content_setting = 127 ContentSetting content_setting =
124 profile_->GetHostContentSettingsMap() 128 HostContentSettingsMapFactory::GetForProfile(profile_)
125 ->GetContentSettingAndMaybeUpdateLastUsage( 129 ->GetContentSettingAndMaybeUpdateLastUsage(
126 requesting_origin, embedding_origin, permission_type_, 130 requesting_origin, embedding_origin, permission_type_,
127 std::string()); 131 std::string());
128 132
129 if (content_setting == CONTENT_SETTING_ALLOW || 133 if (content_setting == CONTENT_SETTING_ALLOW ||
130 content_setting == CONTENT_SETTING_BLOCK) { 134 content_setting == CONTENT_SETTING_BLOCK) {
131 NotifyPermissionSet(id, requesting_origin, embedding_origin, callback, 135 NotifyPermissionSet(id, requesting_origin, embedding_origin, callback,
132 false /* persist */, content_setting); 136 false /* persist */, content_setting);
133 return; 137 return;
134 } 138 }
(...skipping 83 matching lines...) Expand 10 before | Expand all | Expand 10 after
218 DCHECK_CURRENTLY_ON(content::BrowserThread::UI); 222 DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
219 223
220 if (persist) 224 if (persist)
221 UpdateContentSetting(requesting_origin, embedding_origin, content_setting); 225 UpdateContentSetting(requesting_origin, embedding_origin, content_setting);
222 226
223 UpdateTabContext(id, requesting_origin, 227 UpdateTabContext(id, requesting_origin,
224 content_setting == CONTENT_SETTING_ALLOW); 228 content_setting == CONTENT_SETTING_ALLOW);
225 229
226 if (content_setting == CONTENT_SETTING_DEFAULT) { 230 if (content_setting == CONTENT_SETTING_DEFAULT) {
227 content_setting = 231 content_setting =
228 profile_->GetHostContentSettingsMap()->GetDefaultContentSetting( 232 HostContentSettingsMapFactory::GetForProfile(profile_)
229 permission_type_, nullptr); 233 ->GetDefaultContentSetting(permission_type_, nullptr);
230 } 234 }
231 235
232 DCHECK_NE(content_setting, CONTENT_SETTING_DEFAULT); 236 DCHECK_NE(content_setting, CONTENT_SETTING_DEFAULT);
233 callback.Run(content_setting); 237 callback.Run(content_setting);
234 } 238 }
235 239
236 void PermissionContextBase::CleanUpBubble(const PermissionRequestID& id) { 240 void PermissionContextBase::CleanUpBubble(const PermissionRequestID& id) {
237 size_t success = pending_bubbles_.erase(id.ToString()); 241 size_t success = pending_bubbles_.erase(id.ToString());
238 DCHECK(success == 1) << "Missing request " << id.ToString(); 242 DCHECK(success == 1) << "Missing request " << id.ToString();
239 } 243 }
240 244
241 void PermissionContextBase::UpdateContentSetting( 245 void PermissionContextBase::UpdateContentSetting(
242 const GURL& requesting_origin, 246 const GURL& requesting_origin,
243 const GURL& embedding_origin, 247 const GURL& embedding_origin,
244 ContentSetting content_setting) { 248 ContentSetting content_setting) {
245 DCHECK_EQ(requesting_origin, requesting_origin.GetOrigin()); 249 DCHECK_EQ(requesting_origin, requesting_origin.GetOrigin());
246 DCHECK_EQ(embedding_origin, embedding_origin.GetOrigin()); 250 DCHECK_EQ(embedding_origin, embedding_origin.GetOrigin());
247 DCHECK(content_setting == CONTENT_SETTING_ALLOW || 251 DCHECK(content_setting == CONTENT_SETTING_ALLOW ||
248 content_setting == CONTENT_SETTING_BLOCK); 252 content_setting == CONTENT_SETTING_BLOCK);
249 253
250 profile_->GetHostContentSettingsMap()->SetContentSetting( 254 HostContentSettingsMapFactory::GetForProfile(profile_)->SetContentSetting(
251 ContentSettingsPattern::FromURLNoWildcard(requesting_origin), 255 ContentSettingsPattern::FromURLNoWildcard(requesting_origin),
252 ContentSettingsPattern::FromURLNoWildcard(embedding_origin), 256 ContentSettingsPattern::FromURLNoWildcard(embedding_origin),
253 permission_type_, std::string(), content_setting); 257 permission_type_, std::string(), content_setting);
254 } 258 }
OLDNEW
« no previous file with comments | « chrome/browser/pepper_broker_infobar_delegate.cc ('k') | chrome/browser/permissions/permission_context_base_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698