Move counters for passwords, history and autofill to components

components/browsing_data/core/counters/history_counter.h

 // Copyright (c) 2015 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_HISTORY_COUNTER_H_
-#define CHROME_BROWSER_BROWSING_DATA_HISTORY_COUNTER_H_
+#ifndef COMPONENTS_BROWSING_DATA_CORE_COUNTERS_HISTORY_COUNTER_H_
+#define COMPONENTS_BROWSING_DATA_CORE_COUNTERS_HISTORY_COUNTER_H_
 
-#include "base/memory/weak_ptr.h"
 #include "base/task/cancelable_task_tracker.h"
 #include "base/timer/timer.h"
 #include "components/browsing_data/core/counters/browsing_data_counter.h"
 #include "components/history/core/browser/history_service.h"
 #include "components/history/core/browser/web_history_service.h"
 #include "components/sync_driver/sync_service_observer.h"
 
-class Profile;
-
 class ProfileSyncService;
+namespace browsing_data {
 
 class HistoryCounter : public browsing_data::BrowsingDataCounter,
                        public sync_driver::SyncServiceObserver {
  public:
+  typedef base::Callback<history::WebHistoryService*()>
+      GetUpdatedWebHistoryServiceCallback;
+
   class HistoryResult : public FinishedResult {
    public:
     HistoryResult(const HistoryCounter* source,
                   ResultInt value,
                   bool has_synced_visits);
     ~HistoryResult() override;
 
     bool has_synced_visits() const { return has_synced_visits_; }
 
    private:
     bool has_synced_visits_;
   };
 
-  explicit HistoryCounter(Profile* profile);
+  explicit HistoryCounter(ProfileSyncService* profile_sync_service,
+                          const GetUpdatedWebHistoryServiceCallback& callback,
+                          history::HistoryService* history_service);
   ~HistoryCounter() override;
 
   void OnInitialized() override;
 
   // Whether there are counting tasks in progress. Only used for testing.
   bool HasTrackedTasks();
 
-  // Make the history counter use a custom WebHistoryService instance. Only
-  // used for testing.
-  void SetWebHistoryServiceForTesting(history::WebHistoryService* service);
+  const std::string& GetPrefName() const override;
 
  private:
-  Profile* profile_;
-
-  BrowsingDataCounter::ResultInt local_result_;
-  bool has_synced_visits_;
-
-  bool local_counting_finished_;
-  bool web_counting_finished_;
-
-  history::WebHistoryService* testing_web_history_service_;
-
-  base::CancelableTaskTracker cancelable_task_tracker_;
-  std::unique_ptr<history::WebHistoryService::Request> web_history_request_;
-  base::OneShotTimer web_history_timeout_;
-
-  base::ThreadChecker thread_checker_;
-
-  ProfileSyncService* sync_service_;
-  bool history_sync_enabled_;
-
-  base::WeakPtrFactory<HistoryCounter> weak_ptr_factory_;
-
   void Count() override;
 
   void OnGetLocalHistoryCount(history::HistoryCountResult result);
   void OnGetWebHistoryCount(history::WebHistoryService::Request* request,
                             const base::DictionaryValue* result);
   void OnWebHistoryTimeout();
   void MergeResults();
 
+  bool IsHistorySyncEnabled();
+
   // SyncServiceObserver implementation.
   void OnStateChanged() override;
+
+  const std::string pref_name_;
+
+  ProfileSyncService* profile_sync_service_;
+
+  GetUpdatedWebHistoryServiceCallback web_history_service_callback_;
+
+  history::HistoryService* history_service_;
+
+  bool has_synced_visits_;
+
+  bool local_counting_finished_;
+  bool web_counting_finished_;
+
+  base::CancelableTaskTracker cancelable_task_tracker_;
+  std::unique_ptr<history::WebHistoryService::Request> web_history_request_;
+  base::OneShotTimer web_history_timeout_;
+
+  base::ThreadChecker thread_checker_;
+
+  BrowsingDataCounter::ResultInt local_result_;
+
+  bool history_sync_enabled_;
+
+  base::WeakPtrFactory<HistoryCounter> weak_ptr_factory_;
 };
 
-#endif  // CHROME_BROWSER_BROWSING_DATA_HISTORY_COUNTER_H_
+}  // namespace browsing_data

[msramek, 2016/07/20 13:41:16]

nit: Empty line.

[ioanap, 2016/07/20 17:50:37]

Done.

+#endif  // COMPONENTS_BROWSING_DATA_CORE_COUNTERS_HISTORY_COUNTER_H_