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

Side by Side Diff: chrome/browser/ui/startup/startup_browser_creator_browsertest.cc

Issue 777143003: Clean up straggler classes to use embedded signin in page in the new profiles world. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: . Created 6 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <algorithm> 5 #include <algorithm>
6 #include <string> 6 #include <string>
7 7
8 #include "base/command_line.h" 8 #include "base/command_line.h"
9 #include "base/files/file_path.h" 9 #include "base/files/file_path.h"
10 #include "base/prefs/pref_service.h" 10 #include "base/prefs/pref_service.h"
(...skipping 490 matching lines...) Expand 10 before | Expand all | Expand 10 after
501 browser()->host_desktop_type())); 501 browser()->host_desktop_type()));
502 502
503 // This should have created a new browser window. 503 // This should have created a new browser window.
504 Browser* new_browser = FindOneOtherBrowser(browser()); 504 Browser* new_browser = FindOneOtherBrowser(browser());
505 ASSERT_TRUE(new_browser); 505 ASSERT_TRUE(new_browser);
506 506
507 TabStripModel* tab_strip = new_browser->tab_strip_model(); 507 TabStripModel* tab_strip = new_browser->tab_strip_model();
508 EXPECT_EQ(1, tab_strip->count()); 508 EXPECT_EQ(1, tab_strip->count());
509 509
510 if (signin::ShouldShowPromoAtStartup(browser()->profile(), true)) { 510 if (signin::ShouldShowPromoAtStartup(browser()->profile(), true)) {
511 EXPECT_EQ(signin::GetPromoURL(signin::SOURCE_START_PAGE, false), 511 EXPECT_EQ(signin::GetPromoURL(signin_metrics::SOURCE_START_PAGE, false),
512 tab_strip->GetWebContentsAt(0)->GetURL()); 512 tab_strip->GetWebContentsAt(0)->GetURL());
513 } else { 513 } else {
514 EXPECT_EQ(GURL(chrome::kChromeUINewTabURL), 514 EXPECT_EQ(GURL(chrome::kChromeUINewTabURL),
515 tab_strip->GetWebContentsAt(0)->GetURL()); 515 tab_strip->GetWebContentsAt(0)->GetURL());
516 } 516 }
517 } 517 }
518 518
519 IN_PROC_BROWSER_TEST_F(StartupBrowserCreatorTest, SyncPromoWithWelcomePage) { 519 IN_PROC_BROWSER_TEST_F(StartupBrowserCreatorTest, SyncPromoWithWelcomePage) {
520 first_run::SetShouldShowWelcomePage(); 520 first_run::SetShouldShowWelcomePage();
521 521
522 // Do a simple non-process-startup browser launch. 522 // Do a simple non-process-startup browser launch.
523 CommandLine dummy(CommandLine::NO_PROGRAM); 523 CommandLine dummy(CommandLine::NO_PROGRAM);
524 StartupBrowserCreatorImpl launch(base::FilePath(), dummy, 524 StartupBrowserCreatorImpl launch(base::FilePath(), dummy,
525 chrome::startup::IS_FIRST_RUN); 525 chrome::startup::IS_FIRST_RUN);
526 ASSERT_TRUE(launch.Launch(browser()->profile(), std::vector<GURL>(), false, 526 ASSERT_TRUE(launch.Launch(browser()->profile(), std::vector<GURL>(), false,
527 browser()->host_desktop_type())); 527 browser()->host_desktop_type()));
528 528
529 // This should have created a new browser window. 529 // This should have created a new browser window.
530 Browser* new_browser = FindOneOtherBrowser(browser()); 530 Browser* new_browser = FindOneOtherBrowser(browser());
531 ASSERT_TRUE(new_browser); 531 ASSERT_TRUE(new_browser);
532 532
533 TabStripModel* tab_strip = new_browser->tab_strip_model(); 533 TabStripModel* tab_strip = new_browser->tab_strip_model();
534 EXPECT_EQ(2, tab_strip->count()); 534 EXPECT_EQ(2, tab_strip->count());
535 535
536 if (signin::ShouldShowPromoAtStartup(browser()->profile(), true)) { 536 if (signin::ShouldShowPromoAtStartup(browser()->profile(), true)) {
537 EXPECT_EQ(signin::GetPromoURL(signin::SOURCE_START_PAGE, false), 537 EXPECT_EQ(signin::GetPromoURL(signin_metrics::SOURCE_START_PAGE, false),
538 tab_strip->GetWebContentsAt(0)->GetURL()); 538 tab_strip->GetWebContentsAt(0)->GetURL());
539 } else { 539 } else {
540 EXPECT_EQ(GURL(chrome::kChromeUINewTabURL), 540 EXPECT_EQ(GURL(chrome::kChromeUINewTabURL),
541 tab_strip->GetWebContentsAt(0)->GetURL()); 541 tab_strip->GetWebContentsAt(0)->GetURL());
542 } 542 }
543 EXPECT_EQ(internals::GetWelcomePageURL(), 543 EXPECT_EQ(internals::GetWelcomePageURL(),
544 tab_strip->GetWebContentsAt(1)->GetURL()); 544 tab_strip->GetWebContentsAt(1)->GetURL());
545 } 545 }
546 546
547 IN_PROC_BROWSER_TEST_F(StartupBrowserCreatorTest, SyncPromoWithFirstRunTabs) { 547 IN_PROC_BROWSER_TEST_F(StartupBrowserCreatorTest, SyncPromoWithFirstRunTabs) {
(...skipping 12 matching lines...) Expand all
560 ASSERT_TRUE(launch.Launch(browser()->profile(), std::vector<GURL>(), false, 560 ASSERT_TRUE(launch.Launch(browser()->profile(), std::vector<GURL>(), false,
561 browser()->host_desktop_type())); 561 browser()->host_desktop_type()));
562 562
563 // This should have created a new browser window. 563 // This should have created a new browser window.
564 Browser* new_browser = FindOneOtherBrowser(browser()); 564 Browser* new_browser = FindOneOtherBrowser(browser());
565 ASSERT_TRUE(new_browser); 565 ASSERT_TRUE(new_browser);
566 566
567 TabStripModel* tab_strip = new_browser->tab_strip_model(); 567 TabStripModel* tab_strip = new_browser->tab_strip_model();
568 if (signin::ShouldShowPromoAtStartup(browser()->profile(), true)) { 568 if (signin::ShouldShowPromoAtStartup(browser()->profile(), true)) {
569 EXPECT_EQ(2, tab_strip->count()); 569 EXPECT_EQ(2, tab_strip->count());
570 EXPECT_EQ(signin::GetPromoURL(signin::SOURCE_START_PAGE, false), 570 EXPECT_EQ(signin::GetPromoURL(signin_metrics::SOURCE_START_PAGE, false),
571 tab_strip->GetWebContentsAt(0)->GetURL()); 571 tab_strip->GetWebContentsAt(0)->GetURL());
572 EXPECT_EQ("title1.html", 572 EXPECT_EQ("title1.html",
573 tab_strip->GetWebContentsAt(1)->GetURL().ExtractFileName()); 573 tab_strip->GetWebContentsAt(1)->GetURL().ExtractFileName());
574 } else { 574 } else {
575 EXPECT_EQ(1, tab_strip->count()); 575 EXPECT_EQ(1, tab_strip->count());
576 EXPECT_EQ("title1.html", 576 EXPECT_EQ("title1.html",
577 tab_strip->GetWebContentsAt(0)->GetURL().ExtractFileName()); 577 tab_strip->GetWebContentsAt(0)->GetURL().ExtractFileName());
578 } 578 }
579 } 579 }
580 580
(...skipping 12 matching lines...) Expand all
593 ASSERT_TRUE(launch.Launch(browser()->profile(), std::vector<GURL>(), false, 593 ASSERT_TRUE(launch.Launch(browser()->profile(), std::vector<GURL>(), false,
594 browser()->host_desktop_type())); 594 browser()->host_desktop_type()));
595 595
596 // This should have created a new browser window. 596 // This should have created a new browser window.
597 Browser* new_browser = FindOneOtherBrowser(browser()); 597 Browser* new_browser = FindOneOtherBrowser(browser());
598 ASSERT_TRUE(new_browser); 598 ASSERT_TRUE(new_browser);
599 599
600 TabStripModel* tab_strip = new_browser->tab_strip_model(); 600 TabStripModel* tab_strip = new_browser->tab_strip_model();
601 if (signin::ShouldShowPromoAtStartup(browser()->profile(), true)) { 601 if (signin::ShouldShowPromoAtStartup(browser()->profile(), true)) {
602 EXPECT_EQ(3, tab_strip->count()); 602 EXPECT_EQ(3, tab_strip->count());
603 EXPECT_EQ(signin::GetPromoURL(signin::SOURCE_START_PAGE, false), 603 EXPECT_EQ(signin::GetPromoURL(signin_metrics::SOURCE_START_PAGE, false),
604 tab_strip->GetWebContentsAt(0)->GetURL()); 604 tab_strip->GetWebContentsAt(0)->GetURL());
605 EXPECT_EQ("title1.html", 605 EXPECT_EQ("title1.html",
606 tab_strip->GetWebContentsAt(1)->GetURL().ExtractFileName()); 606 tab_strip->GetWebContentsAt(1)->GetURL().ExtractFileName());
607 EXPECT_EQ(internals::GetWelcomePageURL(), 607 EXPECT_EQ(internals::GetWelcomePageURL(),
608 tab_strip->GetWebContentsAt(2)->GetURL()); 608 tab_strip->GetWebContentsAt(2)->GetURL());
609 } else { 609 } else {
610 EXPECT_EQ(2, tab_strip->count()); 610 EXPECT_EQ(2, tab_strip->count());
611 EXPECT_EQ("title1.html", 611 EXPECT_EQ("title1.html",
612 tab_strip->GetWebContentsAt(0)->GetURL().ExtractFileName()); 612 tab_strip->GetWebContentsAt(0)->GetURL().ExtractFileName());
613 EXPECT_EQ(internals::GetWelcomePageURL(), 613 EXPECT_EQ(internals::GetWelcomePageURL(),
(...skipping 579 matching lines...) Expand 10 before | Expand all | Expand 10 after
1193 ASSERT_TRUE(launch.Launch(browser()->profile(), std::vector<GURL>(), true, 1193 ASSERT_TRUE(launch.Launch(browser()->profile(), std::vector<GURL>(), true,
1194 browser()->host_desktop_type())); 1194 browser()->host_desktop_type()));
1195 1195
1196 // This should have created a new browser window. 1196 // This should have created a new browser window.
1197 Browser* new_browser = FindOneOtherBrowser(browser()); 1197 Browser* new_browser = FindOneOtherBrowser(browser());
1198 ASSERT_TRUE(new_browser); 1198 ASSERT_TRUE(new_browser);
1199 1199
1200 // Verify that the sync promo and the welcome page are shown. 1200 // Verify that the sync promo and the welcome page are shown.
1201 TabStripModel* tab_strip = new_browser->tab_strip_model(); 1201 TabStripModel* tab_strip = new_browser->tab_strip_model();
1202 ASSERT_EQ(2, tab_strip->count()); 1202 ASSERT_EQ(2, tab_strip->count());
1203 EXPECT_EQ(signin::GetPromoURL(signin::SOURCE_START_PAGE, false), 1203 EXPECT_EQ(signin::GetPromoURL(signin_metrics::SOURCE_START_PAGE, false),
1204 tab_strip->GetWebContentsAt(0)->GetURL()); 1204 tab_strip->GetWebContentsAt(0)->GetURL());
1205 EXPECT_EQ(internals::GetWelcomePageURL(), 1205 EXPECT_EQ(internals::GetWelcomePageURL(),
1206 tab_strip->GetWebContentsAt(1)->GetURL()); 1206 tab_strip->GetWebContentsAt(1)->GetURL());
1207 } 1207 }
1208 1208
1209 #if defined(GOOGLE_CHROME_BUILD) && defined(OS_MACOSX) 1209 #if defined(GOOGLE_CHROME_BUILD) && defined(OS_MACOSX)
1210 // http://crbug.com/314819 1210 // http://crbug.com/314819
1211 #define MAYBE_FirstRunTabsPromoAllowed DISABLED_FirstRunTabsPromoAllowed 1211 #define MAYBE_FirstRunTabsPromoAllowed DISABLED_FirstRunTabsPromoAllowed
1212 #else 1212 #else
1213 #define MAYBE_FirstRunTabsPromoAllowed FirstRunTabsPromoAllowed 1213 #define MAYBE_FirstRunTabsPromoAllowed FirstRunTabsPromoAllowed
(...skipping 21 matching lines...) Expand all
1235 ASSERT_TRUE(launch.Launch(browser()->profile(), std::vector<GURL>(), true, 1235 ASSERT_TRUE(launch.Launch(browser()->profile(), std::vector<GURL>(), true,
1236 browser()->host_desktop_type())); 1236 browser()->host_desktop_type()));
1237 1237
1238 // This should have created a new browser window. 1238 // This should have created a new browser window.
1239 Browser* new_browser = FindOneOtherBrowser(browser()); 1239 Browser* new_browser = FindOneOtherBrowser(browser());
1240 ASSERT_TRUE(new_browser); 1240 ASSERT_TRUE(new_browser);
1241 1241
1242 // Verify that the first-run tab is shown and the sync promo has been added. 1242 // Verify that the first-run tab is shown and the sync promo has been added.
1243 TabStripModel* tab_strip = new_browser->tab_strip_model(); 1243 TabStripModel* tab_strip = new_browser->tab_strip_model();
1244 ASSERT_EQ(2, tab_strip->count()); 1244 ASSERT_EQ(2, tab_strip->count());
1245 EXPECT_EQ(signin::GetPromoURL(signin::SOURCE_START_PAGE, false), 1245 EXPECT_EQ(signin::GetPromoURL(signin_metrics::SOURCE_START_PAGE, false),
1246 tab_strip->GetWebContentsAt(0)->GetURL()); 1246 tab_strip->GetWebContentsAt(0)->GetURL());
1247 EXPECT_EQ("title1.html", 1247 EXPECT_EQ("title1.html",
1248 tab_strip->GetWebContentsAt(1)->GetURL().ExtractFileName()); 1248 tab_strip->GetWebContentsAt(1)->GetURL().ExtractFileName());
1249 } 1249 }
1250 1250
1251 #if defined(GOOGLE_CHROME_BUILD) && defined(OS_MACOSX) 1251 #if defined(GOOGLE_CHROME_BUILD) && defined(OS_MACOSX)
1252 // http://crbug.com/314819 1252 // http://crbug.com/314819
1253 #define MAYBE_FirstRunTabsContainSyncPromo \ 1253 #define MAYBE_FirstRunTabsContainSyncPromo \
1254 DISABLED_FirstRunTabsContainSyncPromo 1254 DISABLED_FirstRunTabsContainSyncPromo
1255 #else 1255 #else
1256 #define MAYBE_FirstRunTabsContainSyncPromo FirstRunTabsContainSyncPromo 1256 #define MAYBE_FirstRunTabsContainSyncPromo FirstRunTabsContainSyncPromo
1257 #endif 1257 #endif
1258 IN_PROC_BROWSER_TEST_F(StartupBrowserCreatorFirstRunTest, 1258 IN_PROC_BROWSER_TEST_F(StartupBrowserCreatorFirstRunTest,
1259 MAYBE_FirstRunTabsContainSyncPromo) { 1259 MAYBE_FirstRunTabsContainSyncPromo) {
1260 // Simulate the following master_preferences: 1260 // Simulate the following master_preferences:
1261 // { 1261 // {
1262 // "first_run_tabs" : [ 1262 // "first_run_tabs" : [
1263 // "files/title1.html", 1263 // "files/title1.html",
1264 // "chrome://signin/?source=0&next_page=chrome%3A%2F%2Fnewtab%2F" 1264 // "chrome://signin/?source=0&next_page=chrome%3A%2F%2Fnewtab%2F"
1265 // ], 1265 // ],
1266 // "sync_promo": { 1266 // "sync_promo": {
1267 // "show_on_first_run_allowed": true 1267 // "show_on_first_run_allowed": true
1268 // } 1268 // }
1269 // } 1269 // }
1270 ASSERT_TRUE(test_server()->Start()); 1270 ASSERT_TRUE(test_server()->Start());
1271 StartupBrowserCreator browser_creator; 1271 StartupBrowserCreator browser_creator;
1272 browser_creator.AddFirstRunTab(test_server()->GetURL("files/title1.html")); 1272 browser_creator.AddFirstRunTab(test_server()->GetURL("files/title1.html"));
1273 browser_creator.AddFirstRunTab(signin::GetPromoURL(signin::SOURCE_START_PAGE, 1273 browser_creator.AddFirstRunTab(
1274 false)); 1274 signin::GetPromoURL(signin_metrics::SOURCE_START_PAGE, false));
1275 browser()->profile()->GetPrefs()->SetBoolean( 1275 browser()->profile()->GetPrefs()->SetBoolean(
1276 prefs::kSignInPromoShowOnFirstRunAllowed, true); 1276 prefs::kSignInPromoShowOnFirstRunAllowed, true);
1277 1277
1278 // Do a process-startup browser launch. 1278 // Do a process-startup browser launch.
1279 CommandLine dummy(CommandLine::NO_PROGRAM); 1279 CommandLine dummy(CommandLine::NO_PROGRAM);
1280 StartupBrowserCreatorImpl launch(base::FilePath(), dummy, &browser_creator, 1280 StartupBrowserCreatorImpl launch(base::FilePath(), dummy, &browser_creator,
1281 chrome::startup::IS_FIRST_RUN); 1281 chrome::startup::IS_FIRST_RUN);
1282 ASSERT_TRUE(launch.Launch(browser()->profile(), std::vector<GURL>(), true, 1282 ASSERT_TRUE(launch.Launch(browser()->profile(), std::vector<GURL>(), true,
1283 browser()->host_desktop_type())); 1283 browser()->host_desktop_type()));
1284 1284
1285 // This should have created a new browser window. 1285 // This should have created a new browser window.
1286 Browser* new_browser = FindOneOtherBrowser(browser()); 1286 Browser* new_browser = FindOneOtherBrowser(browser());
1287 ASSERT_TRUE(new_browser); 1287 ASSERT_TRUE(new_browser);
1288 1288
1289 // Verify that the first-run tabs are shown and no sync promo has been added 1289 // Verify that the first-run tabs are shown and no sync promo has been added
1290 // as the first-run tabs contain it already. 1290 // as the first-run tabs contain it already.
1291 TabStripModel* tab_strip = new_browser->tab_strip_model(); 1291 TabStripModel* tab_strip = new_browser->tab_strip_model();
1292 ASSERT_EQ(2, tab_strip->count()); 1292 ASSERT_EQ(2, tab_strip->count());
1293 EXPECT_EQ("title1.html", 1293 EXPECT_EQ("title1.html",
1294 tab_strip->GetWebContentsAt(0)->GetURL().ExtractFileName()); 1294 tab_strip->GetWebContentsAt(0)->GetURL().ExtractFileName());
1295 EXPECT_EQ(signin::GetPromoURL(signin::SOURCE_START_PAGE, false), 1295 EXPECT_EQ(signin::GetPromoURL(signin_metrics::SOURCE_START_PAGE, false),
1296 tab_strip->GetWebContentsAt(1)->GetURL()); 1296 tab_strip->GetWebContentsAt(1)->GetURL());
1297 } 1297 }
1298 1298
1299 #if defined(GOOGLE_CHROME_BUILD) && defined(OS_MACOSX) 1299 #if defined(GOOGLE_CHROME_BUILD) && defined(OS_MACOSX)
1300 // http://crbug.com/314819 1300 // http://crbug.com/314819
1301 #define MAYBE_FirstRunTabsContainNTPSyncPromoAllowed \ 1301 #define MAYBE_FirstRunTabsContainNTPSyncPromoAllowed \
1302 DISABLED_FirstRunTabsContainNTPSyncPromoAllowed 1302 DISABLED_FirstRunTabsContainNTPSyncPromoAllowed
1303 #else 1303 #else
1304 #define MAYBE_FirstRunTabsContainNTPSyncPromoAllowed \ 1304 #define MAYBE_FirstRunTabsContainNTPSyncPromoAllowed \
1305 FirstRunTabsContainNTPSyncPromoAllowed 1305 FirstRunTabsContainNTPSyncPromoAllowed
(...skipping 24 matching lines...) Expand all
1330 browser()->host_desktop_type())); 1330 browser()->host_desktop_type()));
1331 1331
1332 // This should have created a new browser window. 1332 // This should have created a new browser window.
1333 Browser* new_browser = FindOneOtherBrowser(browser()); 1333 Browser* new_browser = FindOneOtherBrowser(browser());
1334 ASSERT_TRUE(new_browser); 1334 ASSERT_TRUE(new_browser);
1335 1335
1336 // Verify that the first-run tabs are shown but the NTP that they contain has 1336 // Verify that the first-run tabs are shown but the NTP that they contain has
1337 // been replaced by the sync promo. 1337 // been replaced by the sync promo.
1338 TabStripModel* tab_strip = new_browser->tab_strip_model(); 1338 TabStripModel* tab_strip = new_browser->tab_strip_model();
1339 ASSERT_EQ(2, tab_strip->count()); 1339 ASSERT_EQ(2, tab_strip->count());
1340 EXPECT_EQ(signin::GetPromoURL(signin::SOURCE_START_PAGE, false), 1340 EXPECT_EQ(signin::GetPromoURL(signin_metrics::SOURCE_START_PAGE, false),
1341 tab_strip->GetWebContentsAt(0)->GetURL()); 1341 tab_strip->GetWebContentsAt(0)->GetURL());
1342 EXPECT_EQ("title1.html", 1342 EXPECT_EQ("title1.html",
1343 tab_strip->GetWebContentsAt(1)->GetURL().ExtractFileName()); 1343 tab_strip->GetWebContentsAt(1)->GetURL().ExtractFileName());
1344 } 1344 }
1345 1345
1346 #if defined(GOOGLE_CHROME_BUILD) && defined(OS_MACOSX) 1346 #if defined(GOOGLE_CHROME_BUILD) && defined(OS_MACOSX)
1347 // http://crbug.com/314819 1347 // http://crbug.com/314819
1348 #define MAYBE_FirstRunTabsContainNTPSyncPromoForbidden \ 1348 #define MAYBE_FirstRunTabsContainNTPSyncPromoForbidden \
1349 DISABLED_FirstRunTabsContainNTPSyncPromoForbidden 1349 DISABLED_FirstRunTabsContainNTPSyncPromoForbidden
1350 #else 1350 #else
(...skipping 136 matching lines...) Expand 10 before | Expand all | Expand 10 after
1487 ASSERT_EQ(1, tab_strip->count()); 1487 ASSERT_EQ(1, tab_strip->count());
1488 EXPECT_EQ("title1.html", 1488 EXPECT_EQ("title1.html",
1489 tab_strip->GetWebContentsAt(0)->GetURL().ExtractFileName()); 1489 tab_strip->GetWebContentsAt(0)->GetURL().ExtractFileName());
1490 } 1490 }
1491 #endif // defined(ENABLE_CONFIGURATION_POLICY) 1491 #endif // defined(ENABLE_CONFIGURATION_POLICY)
1492 1492
1493 #endif // !defined(OS_LINUX) || !defined(GOOGLE_CHROME_BUILD) || 1493 #endif // !defined(OS_LINUX) || !defined(GOOGLE_CHROME_BUILD) ||
1494 // defined(ENABLE_CONFIGURATION_POLICY) 1494 // defined(ENABLE_CONFIGURATION_POLICY)
1495 1495
1496 #endif // !defined(OS_CHROMEOS) 1496 #endif // !defined(OS_CHROMEOS)
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698