Moved BrowsingDataCounter and part of BrowsingDataCounterUtils to components.

chrome/browser/browsing_data/downloads_counter_browsertest.cc

 // 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.
 
 #include "chrome/browser/browsing_data/downloads_counter.h"
 
 #include <set>
 
 #include "base/files/file_path.h"
 #include "base/guid.h"
 #include "base/run_loop.h"
 #include "base/strings/string_util.h"
 #include "chrome/browser/download/download_history.h"
 #include "chrome/browser/download/download_service.h"
 #include "chrome/browser/download/download_service_factory.h"
+#include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/test/base/in_process_browser_test.h"
+#include "components/browsing_data/browsing_data_utils.h"
+#include "components/browsing_data/pref_names.h"
 #include "components/history/core/browser/download_row.h"
 #include "components/prefs/pref_service.h"
 #include "content/public/browser/download_manager.h"
 
 #if defined(ENABLE_EXTENSIONS)
 #include "extensions/common/extension.h"
 #endif
 
 namespace {
 
 class DownloadsCounterTest : public InProcessBrowserTest,
                              public DownloadHistory::Observer {
  public:
   void SetUpOnMainThread() override {
     time_ = base::Time::Now();
     items_count_ = 0;
     manager_ =
         content::BrowserContext::GetDownloadManager(browser()->profile());
     history_ =
         DownloadServiceFactory::GetForBrowserContext(browser()->profile())->
             GetDownloadHistory();
     history_->AddObserver(this);
 
     otr_manager_ =
         content::BrowserContext::GetDownloadManager(
             browser()->profile()->GetOffTheRecordProfile());
     SetDownloadsDeletionPref(true);
-    SetDeletionPeriodPref(BrowsingDataRemover::EVERYTHING);
+    SetDeletionPeriodPref(browsing_data::EVERYTHING);
   }
 
   void TearDownOnMainThread() override {
     history_->RemoveObserver(this);
   }
 
   // Adding and removing download items. ---------------------------------------
 
   std::string AddDownload() {
     std::string guid = AddDownloadInternal(
@@ skipping 101 matching lines @@
     item->Remove();
   }
 
   // Miscellaneous. ------------------------------------------------------------
 
   void SetDownloadsDeletionPref(bool value) {
     browser()->profile()->GetPrefs()->SetBoolean(
         prefs::kDeleteDownloadHistory, value);
   }
 
-  void SetDeletionPeriodPref(BrowsingDataRemover::TimePeriod period) {
+  void SetDeletionPeriodPref(browsing_data::TimePeriod period) {
     browser()->profile()->GetPrefs()->SetInteger(
-        prefs::kDeleteTimePeriod, static_cast<int>(period));
+        browsing_data::prefs::kDeleteTimePeriod, static_cast<int>(period));
   }
 
   void RevertTimeInHours(int days) {
     time_ -= base::TimeDelta::FromHours(days);
   }
 
   // Waiting for downloads to be stored. ---------------------------------------
 
   void OnDownloadStored(
       content::DownloadItem* item,
@@ skipping 26 matching lines @@
     if (guids_to_add_.empty() && ids_to_remove_.empty())
       return;
 
     DCHECK(!run_loop_ || !run_loop_->running());
     run_loop_.reset(new base::RunLoop());
     run_loop_->Run();
   }
 
   // Retrieving the result. ----------------------------------------------------
 
-  BrowsingDataCounter::ResultInt GetResult() {
+  browsing_data::BrowsingDataCounter::ResultInt GetResult() {
     DCHECK(finished_);
     return result_;
   }
 
-  void ResultCallback(std::unique_ptr<BrowsingDataCounter::Result> result) {
+  void ResultCallback(
+      std::unique_ptr<browsing_data::BrowsingDataCounter::Result> result) {
     finished_ = result->Finished();
 
     if (finished_) {
-      result_ = static_cast<BrowsingDataCounter::FinishedResult*>(
-          result.get())->Value();
+      result_ =
+          static_cast<browsing_data::BrowsingDataCounter::FinishedResult*>(
+              result.get())
+              ->Value();
     }
   }
 
  private:
   std::unique_ptr<base::RunLoop> run_loop_;
 
   // GUIDs of download items that were added and for which we expect
   // the OnDownloadStored() callback to be called.
   std::set<std::string> guids_to_add_;
 
   // IDs of download items that are being removed from the download service
   // and for which we expect the OnDownloadsRemoved() callback. Unlike in
   // |guids_to_add_|, we don't store GUIDs, because OnDownloadsRemoved() returns
   // a set of IDs.
   std::set<uint32_t> ids_to_remove_;
 
   content::DownloadManager* manager_;
   content::DownloadManager* otr_manager_;
   DownloadHistory* history_;
   base::Time time_;
 
   int items_count_;
 
   bool finished_;
-  BrowsingDataCounter::ResultInt result_;
+  browsing_data::BrowsingDataCounter::ResultInt result_;
 };
 
 // Tests that we count the total number of downloads correctly.
 IN_PROC_BROWSER_TEST_F(DownloadsCounterTest, Count) {
-  DownloadsCounter counter;
-  counter.Init(browser()->profile(),
+  Profile* profile = browser()->profile();
+  DownloadsCounter counter(profile);
+  counter.Init(profile->GetPrefs(),
                base::Bind(&DownloadsCounterTest::ResultCallback,
                           base::Unretained(this)));
   counter.Restart();
   EXPECT_EQ(0u, GetResult());
 
   std::string first_download = AddDownload();
   AddDownload();
   std::string last_download = AddDownload();
   WaitForDownloadHistory();
   counter.Restart();
   EXPECT_EQ(3, GetResult());
 
   RemoveDownload(last_download);
   RemoveDownload(first_download);
   WaitForDownloadHistory();
   counter.Restart();
   EXPECT_EQ(1, GetResult());
 
   AddDownload();
   WaitForDownloadHistory();
   counter.Restart();
   EXPECT_EQ(2, GetResult());
 }
 
 // Tests that not just standard complete downloads are counted.
 IN_PROC_BROWSER_TEST_F(DownloadsCounterTest, Types) {
-  DownloadsCounter counter;
-  counter.Init(browser()->profile(),
+  Profile* profile = browser()->profile();
+  DownloadsCounter counter(profile);
+  counter.Init(profile->GetPrefs(),
                base::Bind(&DownloadsCounterTest::ResultCallback,
                           base::Unretained(this)));
 
   AddDownload();
   AddDownloadWithProperties(
       content::DownloadItem::COMPLETE,
       content::DOWNLOAD_DANGER_TYPE_DANGEROUS_FILE,
       content::DOWNLOAD_INTERRUPT_REASON_NONE);
   AddDownloadWithProperties(
       content::DownloadItem::COMPLETE,
@@ skipping 12 matching lines @@
       content::DOWNLOAD_DANGER_TYPE_UNCOMMON_CONTENT,
       content::DOWNLOAD_INTERRUPT_REASON_USER_CANCELED);
 
   WaitForDownloadHistory();
   counter.Restart();
   EXPECT_EQ(6u, GetResult());
 }
 
 // Tests that downloads not persisted by DownloadHistory are not counted.
 IN_PROC_BROWSER_TEST_F(DownloadsCounterTest, NotPersisted) {
-  DownloadsCounter counter;
-  counter.Init(browser()->profile(),
+  Profile* profile = browser()->profile();
+  DownloadsCounter counter(profile);
+  counter.Init(profile->GetPrefs(),
                base::Bind(&DownloadsCounterTest::ResultCallback,
                           base::Unretained(this)));
 
   // Extension and user scripts download are not persisted.
   AddDownload();
 #if defined(ENABLE_EXTENSIONS)
   AddUserScriptDownload();
   AddExtensionDownload();
 #endif
 
@@ skipping 26 matching lines @@
   RevertTimeInHours(10 * 24);
   AddDownload();  // 8 items
 
   RevertTimeInHours(30 * 24);
   AddDownload();
   AddDownload();
   AddDownload();  // 11 items
 
   WaitForDownloadHistory();
 
-  DownloadsCounter counter;
-  counter.Init(browser()->profile(),
+  Profile* profile = browser()->profile();
+  DownloadsCounter counter(profile);
+  counter.Init(profile->GetPrefs(),
                base::Bind(&DownloadsCounterTest::ResultCallback,
                           base::Unretained(this)));
 
-  SetDeletionPeriodPref(BrowsingDataRemover::LAST_HOUR);
+  SetDeletionPeriodPref(browsing_data::LAST_HOUR);
   EXPECT_EQ(2u, GetResult());
 
-  SetDeletionPeriodPref(BrowsingDataRemover::LAST_DAY);
+  SetDeletionPeriodPref(browsing_data::LAST_DAY);
   EXPECT_EQ(5u, GetResult());
 
-  SetDeletionPeriodPref(BrowsingDataRemover::LAST_WEEK);
+  SetDeletionPeriodPref(browsing_data::LAST_WEEK);
   EXPECT_EQ(7u, GetResult());
 
-  SetDeletionPeriodPref(BrowsingDataRemover::FOUR_WEEKS);
+  SetDeletionPeriodPref(browsing_data::FOUR_WEEKS);
   EXPECT_EQ(8u, GetResult());
 
-  SetDeletionPeriodPref(BrowsingDataRemover::EVERYTHING);
+  SetDeletionPeriodPref(browsing_data::EVERYTHING);
   EXPECT_EQ(11u, GetResult());
 }
 
 }  // namespace