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

Unified Diff: chrome/browser/ui/startup/startup_browser_creator_browsertest.cc

Issue 1473543002: Implement newly designed sign-in related histograms for desktop platorms. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: add comments Created 5 years 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/ui/startup/startup_browser_creator_browsertest.cc
diff --git a/chrome/browser/ui/startup/startup_browser_creator_browsertest.cc b/chrome/browser/ui/startup/startup_browser_creator_browsertest.cc
index e0fadeee2f46f6970dbd7a50bd9edbdb35631edb..1bf1d125cb107d3d32de259d76f49c3939848ef5 100644
--- a/chrome/browser/ui/startup/startup_browser_creator_browsertest.cc
+++ b/chrome/browser/ui/startup/startup_browser_creator_browsertest.cc
@@ -614,7 +614,9 @@ IN_PROC_BROWSER_TEST_F(StartupBrowserCreatorTest, SyncPromoNoWelcomePage) {
if (signin::ShouldShowPromoAtStartup(browser()->profile(), true)) {
// The browser should show only the promo.
ASSERT_EQ(1, tab_strip->count());
- EXPECT_EQ(signin::GetPromoURL(signin_metrics::SOURCE_START_PAGE, false),
+ EXPECT_EQ(signin::GetPromoURL(signin_metrics::ACCESS_POINT_START_PAGE,
+ signin_metrics::REASON_SIGNIN_PRIMARY_ACCOUNT,
+ false),
tab_strip->GetWebContentsAt(0)->GetURL());
} else if (IsWindows10OrNewer()) {
// The browser should show the welcome page and the NTP.
@@ -655,7 +657,9 @@ IN_PROC_BROWSER_TEST_F(StartupBrowserCreatorTest, SyncPromoWithWelcomePage) {
tab_strip->GetWebContentsAt(1)->GetURL());
} else {
if (signin::ShouldShowPromoAtStartup(browser()->profile(), true)) {
- EXPECT_EQ(signin::GetPromoURL(signin_metrics::SOURCE_START_PAGE, false),
+ EXPECT_EQ(signin::GetPromoURL(
+ signin_metrics::ACCESS_POINT_START_PAGE,
+ signin_metrics::REASON_SIGNIN_PRIMARY_ACCOUNT, false),
tab_strip->GetWebContentsAt(0)->GetURL());
} else {
EXPECT_EQ(GURL(chrome::kChromeUINewTabURL),
@@ -691,7 +695,9 @@ IN_PROC_BROWSER_TEST_F(StartupBrowserCreatorTest, SyncPromoWithFirstRunTabs) {
TabStripModel* tab_strip = new_browser->tab_strip_model();
if (signin::ShouldShowPromoAtStartup(browser()->profile(), true)) {
EXPECT_EQ(2, tab_strip->count());
- EXPECT_EQ(signin::GetPromoURL(signin_metrics::SOURCE_START_PAGE, false),
+ EXPECT_EQ(signin::GetPromoURL(signin_metrics::ACCESS_POINT_START_PAGE,
+ signin_metrics::REASON_SIGNIN_PRIMARY_ACCOUNT,
+ false),
tab_strip->GetWebContentsAt(0)->GetURL());
EXPECT_EQ("title1.html",
tab_strip->GetWebContentsAt(1)->GetURL().ExtractFileName());
@@ -726,7 +732,9 @@ IN_PROC_BROWSER_TEST_F(StartupBrowserCreatorTest,
TabStripModel* tab_strip = new_browser->tab_strip_model();
if (signin::ShouldShowPromoAtStartup(browser()->profile(), true)) {
EXPECT_EQ(3, tab_strip->count());
- EXPECT_EQ(signin::GetPromoURL(signin_metrics::SOURCE_START_PAGE, false),
+ EXPECT_EQ(signin::GetPromoURL(signin_metrics::ACCESS_POINT_START_PAGE,
+ signin_metrics::REASON_SIGNIN_PRIMARY_ACCOUNT,
+ false),
tab_strip->GetWebContentsAt(0)->GetURL());
EXPECT_EQ("title1.html",
tab_strip->GetWebContentsAt(1)->GetURL().ExtractFileName());
@@ -1407,8 +1415,10 @@ IN_PROC_BROWSER_TEST_F(StartupBrowserCreatorFirstRunTest,
// Verify that the sync promo and the welcome page are shown.
TabStripModel* tab_strip = new_browser->tab_strip_model();
ASSERT_EQ(2, tab_strip->count());
- EXPECT_EQ(signin::GetPromoURL(signin_metrics::SOURCE_START_PAGE, false),
- tab_strip->GetWebContentsAt(0)->GetURL());
+ EXPECT_EQ(
+ signin::GetPromoURL(signin_metrics::ACCESS_POINT_START_PAGE,
+ signin_metrics::REASON_SIGNIN_PRIMARY_ACCOUNT, false),
+ tab_strip->GetWebContentsAt(0)->GetURL());
EXPECT_EQ(internals::GetWelcomePageURL(),
tab_strip->GetWebContentsAt(1)->GetURL());
}
@@ -1453,8 +1463,10 @@ IN_PROC_BROWSER_TEST_F(StartupBrowserCreatorFirstRunTest,
// Verify that the first-run tab is shown and the sync promo has been added.
TabStripModel* tab_strip = new_browser->tab_strip_model();
ASSERT_EQ(2, tab_strip->count());
- EXPECT_EQ(signin::GetPromoURL(signin_metrics::SOURCE_START_PAGE, false),
- tab_strip->GetWebContentsAt(0)->GetURL());
+ EXPECT_EQ(
+ signin::GetPromoURL(signin_metrics::ACCESS_POINT_START_PAGE,
+ signin_metrics::REASON_SIGNIN_PRIMARY_ACCOUNT, false),
+ tab_strip->GetWebContentsAt(0)->GetURL());
EXPECT_EQ("title1.html",
tab_strip->GetWebContentsAt(1)->GetURL().ExtractFileName());
}
@@ -1484,8 +1496,9 @@ IN_PROC_BROWSER_TEST_F(StartupBrowserCreatorFirstRunTest,
StartupBrowserCreator browser_creator;
browser_creator.AddFirstRunTab(
embedded_test_server()->GetURL("/title1.html"));
- browser_creator.AddFirstRunTab(
- signin::GetPromoURL(signin_metrics::SOURCE_START_PAGE, false));
+ browser_creator.AddFirstRunTab(signin::GetPromoURL(
+ signin_metrics::ACCESS_POINT_START_PAGE,
+ signin_metrics::REASON_SIGNIN_PRIMARY_ACCOUNT, false));
browser()->profile()->GetPrefs()->SetBoolean(
prefs::kSignInPromoShowOnFirstRunAllowed, true);
@@ -1506,8 +1519,10 @@ IN_PROC_BROWSER_TEST_F(StartupBrowserCreatorFirstRunTest,
ASSERT_EQ(2, tab_strip->count());
EXPECT_EQ("title1.html",
tab_strip->GetWebContentsAt(0)->GetURL().ExtractFileName());
- EXPECT_EQ(signin::GetPromoURL(signin_metrics::SOURCE_START_PAGE, false),
- tab_strip->GetWebContentsAt(1)->GetURL());
+ EXPECT_EQ(
+ signin::GetPromoURL(signin_metrics::ACCESS_POINT_START_PAGE,
+ signin_metrics::REASON_SIGNIN_PRIMARY_ACCOUNT, false),
+ tab_strip->GetWebContentsAt(1)->GetURL());
}
#if defined(GOOGLE_CHROME_BUILD) && defined(OS_MACOSX)
@@ -1555,8 +1570,10 @@ IN_PROC_BROWSER_TEST_F(StartupBrowserCreatorFirstRunTest,
// been replaced by the sync promo.
TabStripModel* tab_strip = new_browser->tab_strip_model();
ASSERT_EQ(2, tab_strip->count());
- EXPECT_EQ(signin::GetPromoURL(signin_metrics::SOURCE_START_PAGE, false),
- tab_strip->GetWebContentsAt(0)->GetURL());
+ EXPECT_EQ(
+ signin::GetPromoURL(signin_metrics::ACCESS_POINT_START_PAGE,
+ signin_metrics::REASON_SIGNIN_PRIMARY_ACCOUNT, false),
+ tab_strip->GetWebContentsAt(0)->GetURL());
EXPECT_EQ("title1.html",
tab_strip->GetWebContentsAt(1)->GetURL().ExtractFileName());
}

Powered by Google App Engine
This is Rietveld 408576698