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

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

Issue 2926773002: Check embargo status in PlatformNotificationServiceImpl::CheckPermissionOnIOThread (Closed)
Patch Set: Nits Created 3 years, 6 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 | « chrome/browser/permissions/permission_decision_auto_blocker.h ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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_decision_auto_blocker.h" 5 #include "chrome/browser/permissions/permission_decision_auto_blocker.h"
6 6
7 #include <memory> 7 #include <memory>
8 8
9 #include "base/feature_list.h" 9 #include "base/feature_list.h"
10 #include "base/memory/ptr_util.h" 10 #include "base/memory/ptr_util.h"
11 #include "base/strings/string_number_conversions.h" 11 #include "base/strings/string_number_conversions.h"
12 #include "base/values.h" 12 #include "base/values.h"
13 #include "chrome/browser/browser_process.h" 13 #include "chrome/browser/browser_process.h"
14 #include "chrome/browser/content_settings/host_content_settings_map_factory.h" 14 #include "chrome/browser/content_settings/host_content_settings_map_factory.h"
15 #include "chrome/browser/permissions/permission_blacklist_client.h" 15 #include "chrome/browser/permissions/permission_blacklist_client.h"
16 #include "chrome/browser/permissions/permission_util.h" 16 #include "chrome/browser/permissions/permission_util.h"
17 #include "chrome/browser/profiles/incognito_helpers.h" 17 #include "chrome/browser/profiles/incognito_helpers.h"
18 #include "chrome/browser/profiles/profile.h" 18 #include "chrome/browser/profiles/profile.h"
19 #include "chrome/browser/safe_browsing/safe_browsing_service.h" 19 #include "chrome/browser/safe_browsing/safe_browsing_service.h"
20 #include "chrome/common/chrome_features.h" 20 #include "chrome/common/chrome_features.h"
21 #include "components/content_settings/core/browser/host_content_settings_map.h"
22 #include "components/keyed_service/content/browser_context_dependency_manager.h" 21 #include "components/keyed_service/content/browser_context_dependency_manager.h"
23 #include "components/safe_browsing_db/database_manager.h" 22 #include "components/safe_browsing_db/database_manager.h"
24 #include "components/variations/variations_associated_data.h" 23 #include "components/variations/variations_associated_data.h"
25 #include "content/public/browser/web_contents.h" 24 #include "content/public/browser/web_contents.h"
26 #include "url/gurl.h" 25 #include "url/gurl.h"
27 26
28 namespace { 27 namespace {
29 28
30 // The number of times that users may explicitly dismiss a permission prompt 29 // The number of times that users may explicitly dismiss a permission prompt
31 // from an origin before it is automatically blocked. 30 // from an origin before it is automatically blocked.
(...skipping 159 matching lines...) Expand 10 before | Expand all | Expand 10 after
191 const char PermissionDecisionAutoBlocker::kPermissionBlacklistEmbargoKey[] = 190 const char PermissionDecisionAutoBlocker::kPermissionBlacklistEmbargoKey[] =
192 "blacklisting_embargo_days"; 191 "blacklisting_embargo_days";
193 192
194 // static 193 // static
195 PermissionDecisionAutoBlocker* PermissionDecisionAutoBlocker::GetForProfile( 194 PermissionDecisionAutoBlocker* PermissionDecisionAutoBlocker::GetForProfile(
196 Profile* profile) { 195 Profile* profile) {
197 return PermissionDecisionAutoBlocker::Factory::GetForProfile(profile); 196 return PermissionDecisionAutoBlocker::Factory::GetForProfile(profile);
198 } 197 }
199 198
200 // static 199 // static
200 PermissionResult PermissionDecisionAutoBlocker::GetEmbargoResult(
201 HostContentSettingsMap* settings_map,
202 const GURL& request_origin,
203 ContentSettingsType permission,
204 base::Time current_time) {
205 DCHECK(settings_map);
206 std::unique_ptr<base::DictionaryValue> dict =
207 GetOriginDict(settings_map, request_origin);
208 base::DictionaryValue* permission_dict = GetOrCreatePermissionDict(
209 dict.get(), PermissionUtil::GetPermissionString(permission));
210
211 if (IsUnderEmbargo(permission_dict, features::kPermissionsBlacklist,
212 kPermissionBlacklistEmbargoKey, current_time,
213 base::TimeDelta::FromDays(g_blacklist_embargo_days))) {
214 return PermissionResult(CONTENT_SETTING_BLOCK,
215 PermissionStatusSource::SAFE_BROWSING_BLACKLIST);
216 }
217
218 if (IsUnderEmbargo(permission_dict, features::kBlockPromptsIfDismissedOften,
219 kPermissionDismissalEmbargoKey, current_time,
220 base::TimeDelta::FromDays(g_dismissal_embargo_days))) {
221 return PermissionResult(CONTENT_SETTING_BLOCK,
222 PermissionStatusSource::MULTIPLE_DISMISSALS);
223 }
224
225 if (IsUnderEmbargo(permission_dict, features::kBlockPromptsIfIgnoredOften,
226 kPermissionIgnoreEmbargoKey, current_time,
227 base::TimeDelta::FromDays(g_ignore_embargo_days))) {
228 return PermissionResult(CONTENT_SETTING_BLOCK,
229 PermissionStatusSource::MULTIPLE_IGNORES);
230 }
231
232 return PermissionResult(CONTENT_SETTING_ASK,
233 PermissionStatusSource::UNSPECIFIED);
234 }
235
236 // static
201 void PermissionDecisionAutoBlocker::UpdateFromVariations() { 237 void PermissionDecisionAutoBlocker::UpdateFromVariations() {
202 int dismissals_before_block = -1; 238 int dismissals_before_block = -1;
203 int ignores_before_block = -1; 239 int ignores_before_block = -1;
204 int dismissal_embargo_days = -1; 240 int dismissal_embargo_days = -1;
205 int ignore_embargo_days = -1; 241 int ignore_embargo_days = -1;
206 int blacklist_embargo_days = -1; 242 int blacklist_embargo_days = -1;
207 243
208 std::string dismissals_before_block_value = 244 std::string dismissals_before_block_value =
209 variations::GetVariationParamValueByFeature( 245 variations::GetVariationParamValueByFeature(
210 features::kBlockPromptsIfDismissedOften, kPromptDismissCountKey); 246 features::kBlockPromptsIfDismissedOften, kPromptDismissCountKey);
(...skipping 58 matching lines...) Expand 10 before | Expand all | Expand 10 after
269 callback)); 305 callback));
270 return; 306 return;
271 } 307 }
272 308
273 callback.Run(false /* permission blocked */); 309 callback.Run(false /* permission blocked */);
274 } 310 }
275 311
276 PermissionResult PermissionDecisionAutoBlocker::GetEmbargoResult( 312 PermissionResult PermissionDecisionAutoBlocker::GetEmbargoResult(
277 const GURL& request_origin, 313 const GURL& request_origin,
278 ContentSettingsType permission) { 314 ContentSettingsType permission) {
279 HostContentSettingsMap* map = 315 return GetEmbargoResult(
280 HostContentSettingsMapFactory::GetForProfile(profile_); 316 HostContentSettingsMapFactory::GetForProfile(profile_), request_origin,
281 std::unique_ptr<base::DictionaryValue> dict = 317 permission, clock_->Now());
282 GetOriginDict(map, request_origin);
283 base::DictionaryValue* permission_dict = GetOrCreatePermissionDict(
284 dict.get(), PermissionUtil::GetPermissionString(permission));
285
286 base::Time current_time = clock_->Now();
287 if (IsUnderEmbargo(permission_dict, features::kPermissionsBlacklist,
288 kPermissionBlacklistEmbargoKey, current_time,
289 base::TimeDelta::FromDays(g_blacklist_embargo_days))) {
290 return PermissionResult(CONTENT_SETTING_BLOCK,
291 PermissionStatusSource::SAFE_BROWSING_BLACKLIST);
292 }
293
294 if (IsUnderEmbargo(permission_dict, features::kBlockPromptsIfDismissedOften,
295 kPermissionDismissalEmbargoKey, current_time,
296 base::TimeDelta::FromDays(g_dismissal_embargo_days))) {
297 return PermissionResult(CONTENT_SETTING_BLOCK,
298 PermissionStatusSource::MULTIPLE_DISMISSALS);
299 }
300
301 if (IsUnderEmbargo(permission_dict, features::kBlockPromptsIfIgnoredOften,
302 kPermissionIgnoreEmbargoKey, current_time,
303 base::TimeDelta::FromDays(g_ignore_embargo_days))) {
304 return PermissionResult(CONTENT_SETTING_BLOCK,
305 PermissionStatusSource::MULTIPLE_IGNORES);
306 }
307
308 return PermissionResult(CONTENT_SETTING_ASK,
309 PermissionStatusSource::UNSPECIFIED);
310 } 318 }
311 319
312 int PermissionDecisionAutoBlocker::GetDismissCount( 320 int PermissionDecisionAutoBlocker::GetDismissCount(
313 const GURL& url, 321 const GURL& url,
314 ContentSettingsType permission) { 322 ContentSettingsType permission) {
315 return GetActionCount(url, permission, kPromptDismissCountKey, profile_); 323 return GetActionCount(url, permission, kPromptDismissCountKey, profile_);
316 } 324 }
317 325
318 int PermissionDecisionAutoBlocker::GetIgnoreCount( 326 int PermissionDecisionAutoBlocker::GetIgnoreCount(
319 const GURL& url, 327 const GURL& url,
(...skipping 144 matching lines...) Expand 10 before | Expand all | Expand 10 after
464 scoped_refptr<safe_browsing::SafeBrowsingDatabaseManager> db_manager, 472 scoped_refptr<safe_browsing::SafeBrowsingDatabaseManager> db_manager,
465 int timeout) { 473 int timeout) {
466 db_manager_ = db_manager; 474 db_manager_ = db_manager;
467 safe_browsing_timeout_ = timeout; 475 safe_browsing_timeout_ = timeout;
468 } 476 }
469 477
470 void PermissionDecisionAutoBlocker::SetClockForTesting( 478 void PermissionDecisionAutoBlocker::SetClockForTesting(
471 std::unique_ptr<base::Clock> clock) { 479 std::unique_ptr<base::Clock> clock) {
472 clock_ = std::move(clock); 480 clock_ = std::move(clock);
473 } 481 }
OLDNEW
« no previous file with comments | « chrome/browser/permissions/permission_decision_auto_blocker.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698