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

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

Issue 2446863002: Revert of Add threadsafe version of PermissionManager::GetPermissionStatus (Closed)
Patch Set: Created 4 years, 1 month 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 <stddef.h> 7 #include <stddef.h>
8 #include <utility> 8 #include <utility>
9 9
10 #include "base/callback.h" 10 #include "base/callback.h"
(...skipping 104 matching lines...) Expand 10 before | Expand all | Expand 10 after
115 } 115 }
116 116
117 PermissionUmaUtil::PermissionRequested(permission_type_, requesting_origin, 117 PermissionUmaUtil::PermissionRequested(permission_type_, requesting_origin,
118 embedding_origin, profile_); 118 embedding_origin, profile_);
119 119
120 DecidePermission(web_contents, id, requesting_origin, embedding_origin, 120 DecidePermission(web_contents, id, requesting_origin, embedding_origin,
121 user_gesture, callback); 121 user_gesture, callback);
122 } 122 }
123 123
124 ContentSetting PermissionContextBase::GetPermissionStatus( 124 ContentSetting PermissionContextBase::GetPermissionStatus(
125 HostContentSettingsMap* host,
126 const GURL& requesting_origin, 125 const GURL& requesting_origin,
127 const GURL& embedding_origin) const { 126 const GURL& embedding_origin) const {
127
128 // If the permission has been disabled through Finch, block all requests. 128 // If the permission has been disabled through Finch, block all requests.
129 if (IsPermissionKillSwitchOn()) 129 if (IsPermissionKillSwitchOn())
130 return CONTENT_SETTING_BLOCK; 130 return CONTENT_SETTING_BLOCK;
131 131
132 if (IsRestrictedToSecureOrigins() && 132 if (IsRestrictedToSecureOrigins() &&
133 !content::IsOriginSecure(requesting_origin)) { 133 !content::IsOriginSecure(requesting_origin)) {
134 return CONTENT_SETTING_BLOCK; 134 return CONTENT_SETTING_BLOCK;
135 } 135 }
136 136
137 return host->GetContentSetting(requesting_origin, embedding_origin, 137 return HostContentSettingsMapFactory::GetForProfile(profile_)
138 content_settings_type_, std::string()); 138 ->GetContentSetting(requesting_origin, embedding_origin,
139 } 139 content_settings_type_, std::string());
140
141 ContentSetting PermissionContextBase::GetPermissionStatus(
142 const GURL& requesting_origin,
143 const GURL& embedding_origin) const {
144 HostContentSettingsMap* host =
145 HostContentSettingsMapFactory::GetForProfile(profile_);
146 return GetPermissionStatus(host, requesting_origin, embedding_origin);
147 } 140 }
148 141
149 void PermissionContextBase::ResetPermission( 142 void PermissionContextBase::ResetPermission(
150 const GURL& requesting_origin, 143 const GURL& requesting_origin,
151 const GURL& embedding_origin) { 144 const GURL& embedding_origin) {
152 HostContentSettingsMapFactory::GetForProfile(profile_) 145 HostContentSettingsMapFactory::GetForProfile(profile_)
153 ->SetContentSettingDefaultScope(requesting_origin, embedding_origin, 146 ->SetContentSettingDefaultScope(requesting_origin, embedding_origin,
154 content_settings_type_, std::string(), 147 content_settings_type_, std::string(),
155 CONTENT_SETTING_DEFAULT); 148 CONTENT_SETTING_DEFAULT);
156 } 149 }
(...skipping 151 matching lines...) Expand 10 before | Expand all | Expand 10 after
308 content_setting); 301 content_setting);
309 } 302 }
310 303
311 bool PermissionContextBase::IsPermissionKillSwitchOn() const { 304 bool PermissionContextBase::IsPermissionKillSwitchOn() const {
312 const std::string param = variations::GetVariationParamValue( 305 const std::string param = variations::GetVariationParamValue(
313 kPermissionsKillSwitchFieldStudy, 306 kPermissionsKillSwitchFieldStudy,
314 PermissionUtil::GetPermissionString(permission_type_)); 307 PermissionUtil::GetPermissionString(permission_type_));
315 308
316 return param == kPermissionsKillSwitchBlockedValue; 309 return param == kPermissionsKillSwitchBlockedValue;
317 } 310 }
OLDNEW
« no previous file with comments | « chrome/browser/permissions/permission_context_base.h ('k') | chrome/browser/permissions/permission_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698