Add BrowsingDataCounterFactory

chrome/browser/browsing_data/browsing_data_counter_factory.h

+// Copyright 2016 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 CHROME_BROWSER_BROWSING_DATA_BROWSING_DATA_COUNTER_FACTORY_H_
+#define CHROME_BROWSER_BROWSING_DATA_BROWSING_DATA_COUNTER_FACTORY_H_
+
+#include "base/macros.h"
+#include "base/strings/string16.h"
+
+class Profile;
+
+namespace browsing_data {
+class BrowsingDataCounter;
+}
+
+class BrowsingDataCounterFactory {
+ public:
+  // Creates a new instance of BrowsingDataCounter that is counting the data
+  // related to a given deletion preference |pref_name| for |profile|.
+  static browsing_data::BrowsingDataCounter* GetForPreference(

[msramek, 2016/07/07 11:55:33]

Consider calling this GetForProfileAndPreference or maybe shorter
GetForProfileAndPref. In that spirit, I would also list the |profile| first.

[ioanap, 2016/07/07 16:30:48]

GetForProfileAndPref sounds good!

Done.

+      std::string pref_name,

[msramek, 2016/07/07 11:55:33]

nit: use a const ref to avoid copying the string

[ioanap, 2016/07/07 16:30:48]

Done.

+      Profile* profile);
+
+ private:
+  BrowsingDataCounterFactory();
+  ~BrowsingDataCounterFactory();
+
+  DISALLOW_COPY_AND_ASSIGN(BrowsingDataCounterFactory);
+};
+
+#endif  // CHROME_BROWSER_BROWSING_DATA_BROWSING_DATA_COUNTER_FACTORY_H_