Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(504)

Unified Diff: chrome/browser/browsing_data/hosted_apps_counter_unittest.cc

Issue 2671743002: Separate state of basic and advanced tab in CBD dialog (Closed)
Patch Set: rebase and fix compilation Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/browsing_data/hosted_apps_counter_unittest.cc
diff --git a/chrome/browser/browsing_data/hosted_apps_counter_unittest.cc b/chrome/browser/browsing_data/hosted_apps_counter_unittest.cc
index c997eaf2626c3b88bed08a92bc08eca878154c46..1aa7b6ceb6a9346832401f9906f1f9c106708f20 100644
--- a/chrome/browser/browsing_data/hosted_apps_counter_unittest.cc
+++ b/chrome/browser/browsing_data/hosted_apps_counter_unittest.cc
@@ -35,7 +35,7 @@ class HostedAppsCounterTest : public testing::Test {
extension_registry_ = extensions::ExtensionRegistry::Get(profile_.get());
SetHostedAppsDeletionPref(true);
- SetDeletionPeriodPref(browsing_data::ALL_TIME);
+ SetDeletionPeriodPref(browsing_data::TimePeriod::ALL_TIME);
}
// Adding and removing apps and extensions. ----------------------------------
@@ -152,8 +152,9 @@ class HostedAppsCounterTest : public testing::Test {
TEST_F(HostedAppsCounterTest, Count) {
Profile* profile = GetProfile();
HostedAppsCounter counter(profile);
- counter.Init(profile->GetPrefs(), base::Bind(&HostedAppsCounterTest::Callback,
- base::Unretained(this)));
+ counter.Init(
+ profile->GetPrefs(), browsing_data::ClearBrowsingDataTab::ADVANCED,
+ base::Bind(&HostedAppsCounterTest::Callback, base::Unretained(this)));
counter.Restart();
EXPECT_EQ(0u, GetNumHostedApps());
@@ -178,8 +179,9 @@ TEST_F(HostedAppsCounterTest, Count) {
TEST_F(HostedAppsCounterTest, OnlyHostedApps) {
Profile* profile = GetProfile();
HostedAppsCounter counter(profile);
- counter.Init(profile->GetPrefs(), base::Bind(&HostedAppsCounterTest::Callback,
- base::Unretained(this)));
+ counter.Init(
+ profile->GetPrefs(), browsing_data::ClearBrowsingDataTab::ADVANCED,
+ base::Bind(&HostedAppsCounterTest::Callback, base::Unretained(this)));
AddHostedApp(); // 1
AddExtension();
@@ -215,8 +217,9 @@ TEST_F(HostedAppsCounterTest, OnlyHostedApps) {
TEST_F(HostedAppsCounterTest, Examples) {
Profile* profile = GetProfile();
HostedAppsCounter counter(profile);
- counter.Init(profile->GetPrefs(), base::Bind(&HostedAppsCounterTest::Callback,
- base::Unretained(this)));
+ counter.Init(
+ profile->GetPrefs(), browsing_data::ClearBrowsingDataTab::ADVANCED,
+ base::Bind(&HostedAppsCounterTest::Callback, base::Unretained(this)));
counter.Restart();
EXPECT_EQ(0u, GetExamples().size());

Powered by Google App Engine
This is Rietveld 408576698