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

Side by Side Diff: components/browsing_data/core/counters/site_settings_counter.cc

Issue 2816723002: Add "Site Settings" option to Clear Browsing Data on Android (Closed)
Patch Set: Fix failing test on Android due to HCSM being created before FeatureList Created 3 years, 7 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
(Empty)
1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "components/browsing_data/core/counters/site_settings_counter.h"
6
7 #include "components/browsing_data/core/pref_names.h"
8 #include "components/content_settings/core/browser/content_settings_registry.h"
9
10 namespace browsing_data {
11
12 SiteSettingsCounter::SiteSettingsCounter(HostContentSettingsMap* map)
13 : map_(map) {
14 DCHECK(map_);
15 }
16
17 SiteSettingsCounter::~SiteSettingsCounter() {}
18
19 void SiteSettingsCounter::OnInitialized() {}
20
21 const char* SiteSettingsCounter::GetPrefName() const {
22 return browsing_data::prefs::kDeleteSiteSettings;
23 }
24
25 void SiteSettingsCounter::Count() {
26 std::set<ContentSettingsPattern> patterns;
msramek 2017/05/08 10:53:56 style: Include what you use.
dullweber 2017/05/08 15:50:24 I added includes for <set> and content_settings_pa
27 base::Time period_start = GetPeriodStart();
28 auto* registry = content_settings::ContentSettingsRegistry::GetInstance();
29 for (const content_settings::ContentSettingsInfo* info : *registry) {
30 ContentSettingsType type = info->website_settings_info()->type();
31 ContentSettingsForOneType content_settings_list;
32 map_->GetSettingsForOneType(type, content_settings::ResourceIdentifier(),
33 &content_settings_list);
34 for (const auto& content_setting : content_settings_list) {
35 if (content_setting.source == "preference") {
36 base::Time last_modified = map_->GetSettingLastModifiedDate(
37 content_setting.primary_pattern, content_setting.secondary_pattern,
38 type);
39 if (last_modified >= period_start) {
40 patterns.insert(content_setting.primary_pattern);
41 }
42 }
43 }
44 }
45 ReportResult(patterns.size());
46 }
47
48 } // namespace browsing_data
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698