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

Unified Diff: chrome/browser/profiles/profile_manager_browsertest.cc

Issue 132453004: Add Profile metric counting to new User Manager. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: added new metric information to xml file Created 6 years, 11 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/profiles/profile_manager_browsertest.cc
diff --git a/chrome/browser/profiles/profile_manager_browsertest.cc b/chrome/browser/profiles/profile_manager_browsertest.cc
index 86a8911f126a6c0d89a09ca5db7be0868fd2e4ce..ce258ecba9df4fa47e1fa94b015c4ca45b76ef18 100644
--- a/chrome/browser/profiles/profile_manager_browsertest.cc
+++ b/chrome/browser/profiles/profile_manager_browsertest.cc
@@ -278,21 +278,24 @@ IN_PROC_BROWSER_TEST_F(ProfileManagerBrowserTest,
// Open a browser window for the first profile.
profiles::SwitchToProfile(path_profile1, desktop_type, false,
- kOnProfileSwitchDoNothing);
+ kOnProfileSwitchDoNothing,
+ ProfileMetrics::SWITCH_PROFILE_ICON);
EXPECT_EQ(chrome::GetTotalBrowserCount(), 1U);
EXPECT_EQ(1U, browser_list->size());
EXPECT_EQ(path_profile1, browser_list->get(0)->profile()->GetPath());
// Open a browser window for the second profile.
profiles::SwitchToProfile(path_profile2, desktop_type, false,
- kOnProfileSwitchDoNothing);
+ kOnProfileSwitchDoNothing,
+ ProfileMetrics::SWITCH_PROFILE_ICON);
EXPECT_EQ(chrome::GetTotalBrowserCount(), 2U);
EXPECT_EQ(2U, browser_list->size());
EXPECT_EQ(path_profile2, browser_list->get(1)->profile()->GetPath());
// Switch to the first profile without opening a new window.
profiles::SwitchToProfile(path_profile1, desktop_type, false,
- kOnProfileSwitchDoNothing);
+ kOnProfileSwitchDoNothing,
+ ProfileMetrics::SWITCH_PROFILE_ICON);
EXPECT_EQ(chrome::GetTotalBrowserCount(), 2U);
EXPECT_EQ(2U, browser_list->size());
@@ -338,14 +341,16 @@ IN_PROC_BROWSER_TEST_F(ProfileManagerBrowserTest, EphemeralProfile) {
// Open a browser window for the second profile.
profiles::SwitchToProfile(path_profile2, desktop_type, false,
- kOnProfileSwitchDoNothing);
+ kOnProfileSwitchDoNothing,
+ ProfileMetrics::SWITCH_PROFILE_ICON);
EXPECT_EQ(2U, chrome::GetTotalBrowserCount());
EXPECT_EQ(2U, browser_list->size());
EXPECT_EQ(path_profile2, browser_list->get(1)->profile()->GetPath());
// Create a second window for the ephemeral profile.
profiles::SwitchToProfile(path_profile2, desktop_type, true,
- kOnProfileSwitchDoNothing);
+ kOnProfileSwitchDoNothing,
+ ProfileMetrics::SWITCH_PROFILE_ICON);
EXPECT_EQ(3U, chrome::GetTotalBrowserCount());
EXPECT_EQ(3U, browser_list->size());

Powered by Google App Engine
This is Rietveld 408576698