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

Unified Diff: chrome/browser/profile_resetter/profile_resetter_unittest.cc

Issue 23450021: Profile Reset dialog: the new section with detailed feedback information (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed compilation Created 7 years, 3 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/profile_resetter/profile_resetter_unittest.cc
diff --git a/chrome/browser/profile_resetter/profile_resetter_unittest.cc b/chrome/browser/profile_resetter/profile_resetter_unittest.cc
index 34b6110b52b6272958ba1b806231c3963f4607c7..1e6d0a0d12a9465967b559ba3418b47865f21ecb 100644
--- a/chrome/browser/profile_resetter/profile_resetter_unittest.cc
+++ b/chrome/browser/profile_resetter/profile_resetter_unittest.cc
@@ -726,7 +726,8 @@ TEST_F(ProfileResetterTest, CheckSnapshots) {
EXPECT_TRUE(empty_snap.homepage().empty());
EXPECT_TRUE(empty_snap.homepage_is_ntp());
EXPECT_NE(std::string::npos, empty_snap.dse_url().find("{google:baseURL}"));
- EXPECT_EQ(std::vector<std::string>(), empty_snap.enabled_extensions());
+ EXPECT_EQ(ResettableSettingsSnapshot::ExtensionList(),
+ empty_snap.enabled_extensions());
// Reset to organic defaults.
ResetAndWait(ProfileResetter::DEFAULT_SEARCH_ENGINE |
@@ -745,8 +746,9 @@ TEST_F(ProfileResetterTest, CheckSnapshots) {
EXPECT_EQ("http://www.foo.com", nonorganic_snap.homepage());
EXPECT_FALSE(nonorganic_snap.homepage_is_ntp());
EXPECT_EQ("http://www.foo.com/s?q={searchTerms}", nonorganic_snap.dse_url());
- EXPECT_EQ(std::vector<std::string>(1, ext_id),
- nonorganic_snap.enabled_extensions());
+ EXPECT_EQ(ResettableSettingsSnapshot::ExtensionList(
+ 1, std::make_pair(ext_id, "example")),
+ nonorganic_snap.enabled_extensions());
}
TEST_F(ProfileResetterTest, FeedbackSerializtionTest) {

Powered by Google App Engine
This is Rietveld 408576698