| Index: chrome/browser/history/history_browsertest.cc
|
| diff --git a/chrome/browser/history/history_browsertest.cc b/chrome/browser/history/history_browsertest.cc
|
| index e78d95ae460a87e007790a641246d78b40fbd453..29ee9f36773c73dbe38f8f0ce9cb4e9339c9ea52 100644
|
| --- a/chrome/browser/history/history_browsertest.cc
|
| +++ b/chrome/browser/history/history_browsertest.cc
|
| @@ -15,7 +15,6 @@
|
| #include "chrome/browser/profiles/profile.h"
|
| #include "chrome/browser/ui/browser.h"
|
| #include "chrome/browser/ui/browser_commands.h"
|
| -#include "chrome/browser/ui/browser_tabstrip.h"
|
| #include "chrome/browser/ui/tabs/tab_strip_model.h"
|
| #include "chrome/common/chrome_switches.h"
|
| #include "chrome/common/pref_names.h"
|
| @@ -103,7 +102,7 @@ class HistoryBrowserTest : public InProcessBrowserTest {
|
| void LoadAndWaitForURL(const GURL& url) {
|
| string16 expected_title(ASCIIToUTF16("OK"));
|
| content::TitleWatcher title_watcher(
|
| - chrome::GetActiveWebContents(browser()), expected_title);
|
| + browser()->tab_strip_model()->GetActiveWebContents(), expected_title);
|
| title_watcher.AlsoWaitForTitle(ASCIIToUTF16("FAIL"));
|
| ui_test_utils::NavigateToURL(browser(), url);
|
| EXPECT_EQ(expected_title, title_watcher.WaitAndGetTitle());
|
| @@ -246,7 +245,8 @@ IN_PROC_BROWSER_TEST_F(HistoryBrowserTest,
|
| // Therefore, Page 11 should be in the history in addition to Page 12.
|
| LoadAndWaitForFile("history_length_test_page_11.html");
|
|
|
| - content::SimulateMouseClick(chrome::GetActiveWebContents(browser()), 0,
|
| + content::SimulateMouseClick(
|
| + browser()->tab_strip_model()->GetActiveWebContents(), 0,
|
| WebKit::WebMouseEvent::ButtonLeft);
|
| LoadAndWaitForFile("history_length_test_page_11.html");
|
| }
|
| @@ -277,11 +277,12 @@ IN_PROC_BROWSER_TEST_F(HistoryBrowserTest, MAYBE_HistorySearchXSS) {
|
| // so that we're not susceptible (less susceptible?) to a race condition.
|
| // Should a race condition ever trigger, it won't result in flakiness.
|
| int num = ui_test_utils::FindInPage(
|
| - chrome::GetActiveWebContents(browser()), ASCIIToUTF16("<img"), true,
|
| + browser()->tab_strip_model()->GetActiveWebContents(),
|
| + ASCIIToUTF16("<img"), true,
|
| true, NULL, NULL);
|
| EXPECT_GT(num, 0);
|
| EXPECT_EQ(ASCIIToUTF16("History"),
|
| - chrome::GetActiveWebContents(browser())->GetTitle());
|
| + browser()->tab_strip_model()->GetActiveWebContents()->GetTitle());
|
| }
|
|
|
| // Verify that history persists after session restart.
|
| @@ -375,7 +376,8 @@ IN_PROC_BROWSER_TEST_F(HistoryBrowserTest, RedirectHistory) {
|
| FilePath().AppendASCII("landing.html"));
|
| ui_test_utils::NavigateToURLBlockUntilNavigationsComplete(
|
| browser(), redirector, 2);
|
| - ASSERT_EQ(landing_url, chrome::GetActiveWebContents(browser())->GetURL());
|
| + ASSERT_EQ(landing_url,
|
| + browser()->tab_strip_model()->GetActiveWebContents()->GetURL());
|
| std::vector<GURL> urls(GetHistoryContents());
|
| ASSERT_EQ(2u, urls.size());
|
| ASSERT_EQ(landing_url, urls[0]);
|
| @@ -413,7 +415,8 @@ IN_PROC_BROWSER_TEST_F(HistoryBrowserTest, ReloadBringPageToTop) {
|
| ASSERT_EQ(url2, urls[0]);
|
| ASSERT_EQ(url1, urls[1]);
|
|
|
| - content::WebContents* tab = chrome::GetActiveWebContents(browser());
|
| + content::WebContents* tab =
|
| + browser()->tab_strip_model()->GetActiveWebContents();
|
| tab->GetController().Reload(false);
|
| content::WaitForLoadStop(tab);
|
|
|
| @@ -432,7 +435,8 @@ IN_PROC_BROWSER_TEST_F(HistoryBrowserTest, BackForwardBringPageToTop) {
|
| ui_test_utils::NavigateToURL(browser(), url1);
|
| ui_test_utils::NavigateToURL(browser(), url2);
|
|
|
| - content::WebContents* tab = chrome::GetActiveWebContents(browser());
|
| + content::WebContents* tab =
|
| + browser()->tab_strip_model()->GetActiveWebContents();
|
| chrome::GoBack(browser(), CURRENT_TAB);
|
| content::WaitForLoadStop(tab);
|
|
|
| @@ -459,10 +463,11 @@ IN_PROC_BROWSER_TEST_F(HistoryBrowserTest, SubmitFormAddsTargetPage) {
|
| FilePath().AppendASCII("target.html"));
|
| ui_test_utils::NavigateToURL(browser(), form);
|
|
|
| - content::WebContents* web_contents = chrome::GetActiveWebContents(browser());
|
| + content::WebContents* web_contents =
|
| + browser()->tab_strip_model()->GetActiveWebContents();
|
| string16 expected_title(ASCIIToUTF16("Target Page"));
|
| content::TitleWatcher title_watcher(
|
| - chrome::GetActiveWebContents(browser()), expected_title);
|
| + browser()->tab_strip_model()->GetActiveWebContents(), expected_title);
|
| ASSERT_TRUE(content::ExecuteScript(
|
| web_contents, "document.getElementById('form').submit()"));
|
| EXPECT_EQ(expected_title, title_watcher.WaitAndGetTitle());
|
| @@ -480,7 +485,8 @@ IN_PROC_BROWSER_TEST_F(HistoryBrowserTest, OneHistoryTabPerWindow) {
|
|
|
| // Even after navigate completes, the currently-active tab title is
|
| // 'Loading...' for a brief time while the history page loads.
|
| - content::WebContents* web_contents = chrome::GetActiveWebContents(browser());
|
| + content::WebContents* web_contents =
|
| + browser()->tab_strip_model()->GetActiveWebContents();
|
| string16 expected_title(ASCIIToUTF16("History"));
|
| content::TitleWatcher title_watcher(web_contents, expected_title);
|
| chrome::ExecuteCommand(browser(), IDC_SHOW_HISTORY);
|
|
|