Count number of origins with data affected by clearing "cookies and site data".

chrome/browser/browsing_data/cookie_counter.h

+// Copyright (c) 2016 The Chromium Authors. All rights reserved.

[msramek, 2017/01/09 12:54:43]

nit: s/(c) //g  (we don't use "(c)" anymore in new files)

nit: It's 2017. Happy new year!

[dullweber, 2017/01/09 16:05:46]

Updated all new files

+// 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_COOKIE_COUNTER_H_
+#define CHROME_BROWSER_BROWSING_DATA_COOKIE_COUNTER_H_
+
+#include "base/memory/weak_ptr.h"
+#include "chrome/browser/profiles/profile.h"

[msramek, 2017/01/09 12:54:43]

nit: Forward declaration should be enough.

[dullweber, 2017/01/09 16:05:46]

Done.

+#include "components/browsing_data/core/counters/browsing_data_counter.h"
+
+class CookieCounter : public browsing_data::BrowsingDataCounter {

[msramek, 2017/01/09 12:54:43]

Even though we colloquially refers to the item in the CBD dialog as "cookies", I
think it's quite misleading here. I would suggest naming it
CookiesAndSiteDataCounter, or simply SiteDataCounter (as cookies are also site
data).

[dullweber, 2017/01/09 16:05:46]

Done.

+ public:
+  explicit CookieCounter(Profile* profile);
+  ~CookieCounter() override;
+
+  const char* GetPrefName() const override;
+
+ private:
+  Profile* profile_;

[msramek, 2017/01/09 12:54:43]

style: Methods before attributes (I know, I know :) )

[dullweber, 2017/01/09 16:05:46]

Done.

+
+  base::WeakPtrFactory<CookieCounter> weak_ptr_factory_;
+
+  void Count() override;
+
+  void Done(int origin_count);
+};
+
+#endif  // CHROME_BROWSER_BROWSING_DATA_COOKIE_COUNTER_H_