Add "Site Settings" option to Clear Browsing Data on Android

components/browsing_data/core/counters/site_settings_counter.h

Index: components/browsing_data/core/counters/site_settings_counter.h
diff --git a/components/browsing_data/core/counters/site_settings_counter.h b/components/browsing_data/core/counters/site_settings_counter.h
new file mode 100644
index 0000000000000000000000000000000000000000..f5c46df1008817f986fe69cecbde32420395feba
--- /dev/null
+++ b/components/browsing_data/core/counters/site_settings_counter.h
@@ -0,0 +1,33 @@
+// Copyright 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef COMPONENTS_BROWSING_DATA_CORE_COUNTERS_SITE_SETTINGS_COUNTER_H_
+#define COMPONENTS_BROWSING_DATA_CORE_COUNTERS_SITE_SETTINGS_COUNTER_H_
+
+#include <memory>
+#include <vector>

[msramek, 2017/05/08 10:53:56]

Not needed.

+
+#include "components/browsing_data/core/counters/browsing_data_counter.h"
+#include "components/content_settings/core/browser/host_content_settings_map.h"
+
+namespace browsing_data {
+
+class SiteSettingsCounter : public browsing_data::BrowsingDataCounter {
+ public:
+  explicit SiteSettingsCounter(HostContentSettingsMap* hcsm);
+  ~SiteSettingsCounter() override;
+
+  const char* GetPrefName() const override;
+
+ private:
+  void OnInitialized() override;
+
+  void Count() override;
+
+  scoped_refptr<HostContentSettingsMap> map_;

[msramek, 2017/05/08 10:53:56]

Do we need to keep this pointer? We don't observe HCSM, just call a method on
it...

[dullweber, 2017/05/08 15:50:24]

I need the pointer because I can't access the HostContentSettingsMapFactory from
components/

+};
+
+}  // namespace browsing_data
+
+#endif  // COMPONENTS_BROWSING_DATA_CORE_COUNTERS_SITE_SETTINGS_COUNTER_H_