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

Side by Side Diff: chrome/browser/history/history_browsertest.cc

Issue 2248873002: Convert WindowOpenDisposition to an enum class. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase. Created 4 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 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 <utility> 5 #include <utility>
6 #include <vector> 6 #include <vector>
7 7
8 #include "base/bind.h" 8 #include "base/bind.h"
9 #include "base/command_line.h" 9 #include "base/command_line.h"
10 #include "base/files/file_path.h" 10 #include "base/files/file_path.h"
(...skipping 366 matching lines...) Expand 10 before | Expand all | Expand 10 after
377 base::FilePath(), base::FilePath(FILE_PATH_LITERAL("title1.html"))); 377 base::FilePath(), base::FilePath(FILE_PATH_LITERAL("title1.html")));
378 GURL url3 = ui_test_utils::GetTestUrl( 378 GURL url3 = ui_test_utils::GetTestUrl(
379 base::FilePath(), base::FilePath(FILE_PATH_LITERAL("title3.html"))); 379 base::FilePath(), base::FilePath(FILE_PATH_LITERAL("title3.html")));
380 GURL url4 = ui_test_utils::GetTestUrl( 380 GURL url4 = ui_test_utils::GetTestUrl(
381 base::FilePath(), base::FilePath(FILE_PATH_LITERAL("simple.html"))); 381 base::FilePath(), base::FilePath(FILE_PATH_LITERAL("simple.html")));
382 382
383 ui_test_utils::NavigateToURL(browser(), url1); 383 ui_test_utils::NavigateToURL(browser(), url1);
384 Browser* browser2 = CreateBrowser(browser()->profile()); 384 Browser* browser2 = CreateBrowser(browser()->profile());
385 ui_test_utils::NavigateToURL(browser2, url2); 385 ui_test_utils::NavigateToURL(browser2, url2);
386 ui_test_utils::NavigateToURLWithDisposition( 386 ui_test_utils::NavigateToURLWithDisposition(
387 browser2, url3, NEW_FOREGROUND_TAB, 387 browser2, url3, WindowOpenDisposition::NEW_FOREGROUND_TAB,
388 ui_test_utils::BROWSER_TEST_WAIT_FOR_NAVIGATION); 388 ui_test_utils::BROWSER_TEST_WAIT_FOR_NAVIGATION);
389 ui_test_utils::NavigateToURLWithDisposition( 389 ui_test_utils::NavigateToURLWithDisposition(
390 browser2, url4, NEW_FOREGROUND_TAB, 390 browser2, url4, WindowOpenDisposition::NEW_FOREGROUND_TAB,
391 ui_test_utils::BROWSER_TEST_WAIT_FOR_NAVIGATION); 391 ui_test_utils::BROWSER_TEST_WAIT_FOR_NAVIGATION);
392 392
393 std::vector<GURL> urls(GetHistoryContents()); 393 std::vector<GURL> urls(GetHistoryContents());
394 ASSERT_EQ(4u, urls.size()); 394 ASSERT_EQ(4u, urls.size());
395 ASSERT_EQ(url4, urls[0]); 395 ASSERT_EQ(url4, urls[0]);
396 ASSERT_EQ(url3, urls[1]); 396 ASSERT_EQ(url3, urls[1]);
397 ASSERT_EQ(url2, urls[2]); 397 ASSERT_EQ(url2, urls[2]);
398 ASSERT_EQ(url1, urls[3]); 398 ASSERT_EQ(url1, urls[3]);
399 } 399 }
400 400
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
441 } 441 }
442 442
443 // Verify that reloading a page brings it to top of history list. 443 // Verify that reloading a page brings it to top of history list.
444 IN_PROC_BROWSER_TEST_F(HistoryBrowserTest, ReloadBringPageToTop) { 444 IN_PROC_BROWSER_TEST_F(HistoryBrowserTest, ReloadBringPageToTop) {
445 GURL url1 = GetTestUrl(); 445 GURL url1 = GetTestUrl();
446 GURL url2 = ui_test_utils::GetTestUrl( 446 GURL url2 = ui_test_utils::GetTestUrl(
447 base::FilePath(), base::FilePath(FILE_PATH_LITERAL("title3.html"))); 447 base::FilePath(), base::FilePath(FILE_PATH_LITERAL("title3.html")));
448 448
449 ui_test_utils::NavigateToURL(browser(), url1); 449 ui_test_utils::NavigateToURL(browser(), url1);
450 ui_test_utils::NavigateToURLWithDisposition( 450 ui_test_utils::NavigateToURLWithDisposition(
451 browser(), url2, NEW_BACKGROUND_TAB, 451 browser(), url2, WindowOpenDisposition::NEW_BACKGROUND_TAB,
452 ui_test_utils::BROWSER_TEST_WAIT_FOR_NAVIGATION); 452 ui_test_utils::BROWSER_TEST_WAIT_FOR_NAVIGATION);
453 453
454 std::vector<GURL> urls(GetHistoryContents()); 454 std::vector<GURL> urls(GetHistoryContents());
455 ASSERT_EQ(2u, urls.size()); 455 ASSERT_EQ(2u, urls.size());
456 ASSERT_EQ(url2, urls[0]); 456 ASSERT_EQ(url2, urls[0]);
457 ASSERT_EQ(url1, urls[1]); 457 ASSERT_EQ(url1, urls[1]);
458 458
459 content::WebContents* tab = 459 content::WebContents* tab =
460 browser()->tab_strip_model()->GetActiveWebContents(); 460 browser()->tab_strip_model()->GetActiveWebContents();
461 tab->GetController().Reload(false); 461 tab->GetController().Reload(false);
462 content::WaitForLoadStop(tab); 462 content::WaitForLoadStop(tab);
463 463
464 urls = GetHistoryContents(); 464 urls = GetHistoryContents();
465 ASSERT_EQ(2u, urls.size()); 465 ASSERT_EQ(2u, urls.size());
466 ASSERT_EQ(url1, urls[0]); 466 ASSERT_EQ(url1, urls[0]);
467 ASSERT_EQ(url2, urls[1]); 467 ASSERT_EQ(url2, urls[1]);
468 } 468 }
469 469
470 // Verify that back/forward brings current page to top of history list. 470 // Verify that back/forward brings current page to top of history list.
471 IN_PROC_BROWSER_TEST_F(HistoryBrowserTest, BackForwardBringPageToTop) { 471 IN_PROC_BROWSER_TEST_F(HistoryBrowserTest, BackForwardBringPageToTop) {
472 GURL url1 = GetTestUrl(); 472 GURL url1 = GetTestUrl();
473 GURL url2 = ui_test_utils::GetTestUrl( 473 GURL url2 = ui_test_utils::GetTestUrl(
474 base::FilePath(), base::FilePath(FILE_PATH_LITERAL("title3.html"))); 474 base::FilePath(), base::FilePath(FILE_PATH_LITERAL("title3.html")));
475 475
476 ui_test_utils::NavigateToURL(browser(), url1); 476 ui_test_utils::NavigateToURL(browser(), url1);
477 ui_test_utils::NavigateToURL(browser(), url2); 477 ui_test_utils::NavigateToURL(browser(), url2);
478 478
479 content::WebContents* tab = 479 content::WebContents* tab =
480 browser()->tab_strip_model()->GetActiveWebContents(); 480 browser()->tab_strip_model()->GetActiveWebContents();
481 chrome::GoBack(browser(), CURRENT_TAB); 481 chrome::GoBack(browser(), WindowOpenDisposition::CURRENT_TAB);
482 content::WaitForLoadStop(tab); 482 content::WaitForLoadStop(tab);
483 483
484 std::vector<GURL> urls(GetHistoryContents()); 484 std::vector<GURL> urls(GetHistoryContents());
485 ASSERT_EQ(2u, urls.size()); 485 ASSERT_EQ(2u, urls.size());
486 ASSERT_EQ(url1, urls[0]); 486 ASSERT_EQ(url1, urls[0]);
487 ASSERT_EQ(url2, urls[1]); 487 ASSERT_EQ(url2, urls[1]);
488 488
489 chrome::GoForward(browser(), CURRENT_TAB); 489 chrome::GoForward(browser(), WindowOpenDisposition::CURRENT_TAB);
490 content::WaitForLoadStop(tab); 490 content::WaitForLoadStop(tab);
491 urls = GetHistoryContents(); 491 urls = GetHistoryContents();
492 ASSERT_EQ(2u, urls.size()); 492 ASSERT_EQ(2u, urls.size());
493 ASSERT_EQ(url2, urls[0]); 493 ASSERT_EQ(url2, urls[0]);
494 ASSERT_EQ(url1, urls[1]); 494 ASSERT_EQ(url1, urls[1]);
495 } 495 }
496 496
497 // Verify that submitting form adds target page to history list. 497 // Verify that submitting form adds target page to history list.
498 IN_PROC_BROWSER_TEST_F(HistoryBrowserTest, SubmitFormAddsTargetPage) { 498 IN_PROC_BROWSER_TEST_F(HistoryBrowserTest, SubmitFormAddsTargetPage) {
499 GURL form = ui_test_utils::GetTestUrl( 499 GURL form = ui_test_utils::GetTestUrl(
(...skipping 27 matching lines...) Expand all
527 // Even after navigate completes, the currently-active tab title is 527 // Even after navigate completes, the currently-active tab title is
528 // 'Loading...' for a brief time while the history page loads. 528 // 'Loading...' for a brief time while the history page loads.
529 content::WebContents* web_contents = 529 content::WebContents* web_contents =
530 browser()->tab_strip_model()->GetActiveWebContents(); 530 browser()->tab_strip_model()->GetActiveWebContents();
531 base::string16 expected_title(base::ASCIIToUTF16("History")); 531 base::string16 expected_title(base::ASCIIToUTF16("History"));
532 content::TitleWatcher title_watcher(web_contents, expected_title); 532 content::TitleWatcher title_watcher(web_contents, expected_title);
533 chrome::ExecuteCommand(browser(), IDC_SHOW_HISTORY); 533 chrome::ExecuteCommand(browser(), IDC_SHOW_HISTORY);
534 EXPECT_EQ(expected_title, title_watcher.WaitAndGetTitle()); 534 EXPECT_EQ(expected_title, title_watcher.WaitAndGetTitle());
535 535
536 ui_test_utils::NavigateToURLWithDisposition( 536 ui_test_utils::NavigateToURLWithDisposition(
537 browser(), 537 browser(), GURL(url::kAboutBlankURL),
538 GURL(url::kAboutBlankURL), 538 WindowOpenDisposition::NEW_FOREGROUND_TAB,
539 NEW_FOREGROUND_TAB,
540 ui_test_utils::BROWSER_TEST_WAIT_FOR_NAVIGATION); 539 ui_test_utils::BROWSER_TEST_WAIT_FOR_NAVIGATION);
541 chrome::ExecuteCommand(browser(), IDC_SHOW_HISTORY); 540 chrome::ExecuteCommand(browser(), IDC_SHOW_HISTORY);
542 541
543 content::WebContents* active_web_contents = 542 content::WebContents* active_web_contents =
544 browser()->tab_strip_model()->GetActiveWebContents(); 543 browser()->tab_strip_model()->GetActiveWebContents();
545 ASSERT_EQ(web_contents, active_web_contents); 544 ASSERT_EQ(web_contents, active_web_contents);
546 ASSERT_EQ(history_url, active_web_contents->GetURL()); 545 ASSERT_EQ(history_url, active_web_contents->GetURL());
547 546
548 content::WebContents* second_tab = 547 content::WebContents* second_tab =
549 browser()->tab_strip_model()->GetWebContentsAt(1); 548 browser()->tab_strip_model()->GetWebContentsAt(1);
550 ASSERT_NE(history_url, second_tab->GetURL()); 549 ASSERT_NE(history_url, second_tab->GetURL());
551 } 550 }
OLDNEW
« no previous file with comments | « chrome/browser/favicon/content_favicon_driver_browsertest.cc ('k') | chrome/browser/history/redirect_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698