summaryrefslogtreecommitdiffstats
path: root/chrome/browser/browser_commands_unittest.cc
diff options
context:
space:
mode:
authorjam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2012-01-03 05:53:20 +0000
committerjam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2012-01-03 05:53:20 +0000
commit4ca153033f022dc6e11d6094daed41b1ed0a995f (patch)
treef4720517b10953166f84707ca3ec8b82de860884 /chrome/browser/browser_commands_unittest.cc
parent08bd3d71d969061b1588e6a4210f6d74b0ec0c6d (diff)
downloadchromium_src-4ca153033f022dc6e11d6094daed41b1ed0a995f.zip
chromium_src-4ca153033f022dc6e11d6094daed41b1ed0a995f.tar.gz
chromium_src-4ca153033f022dc6e11d6094daed41b1ed0a995f.tar.bz2
Replace most of Browser::GetSelectedTabContents calls into Browser::GetSelectedWebContents. I've converted the easy ones, I'll do the ones with more dependencies in a separate change to keep things trivial to review.
I considered taking out GetSelectedTabContents altogether and having people just use GetSelectedTabContentsWrapper()->web_contents() per the existing comment in browser.h, but there are a lot of callers and it seemed too long to type. BUG=98716 TBR=joi Review URL: http://codereview.chromium.org/9015022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116122 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/browser_commands_unittest.cc')
-rw-r--r--chrome/browser/browser_commands_unittest.cc12
1 files changed, 6 insertions, 6 deletions
diff --git a/chrome/browser/browser_commands_unittest.cc b/chrome/browser/browser_commands_unittest.cc
index b2cd8da..60ccb42 100644
--- a/chrome/browser/browser_commands_unittest.cc
+++ b/chrome/browser/browser_commands_unittest.cc
@@ -161,14 +161,14 @@ TEST_F(BrowserCommandsTest, BackForwardInNewTab) {
CommitPendingLoad(&second->GetController());
browser()->GoBack(NEW_FOREGROUND_TAB);
ASSERT_EQ(3, browser()->active_index());
- ASSERT_EQ(url1, browser()->GetSelectedTabContents()->GetURL());
+ ASSERT_EQ(url1, browser()->GetSelectedWebContents()->GetURL());
// Same thing again for forward.
// TODO(brettw) bug 11055: see the comment above about why we need this.
- CommitPendingLoad(&browser()->GetSelectedTabContents()->GetController());
+ CommitPendingLoad(&browser()->GetSelectedWebContents()->GetController());
browser()->GoForward(NEW_FOREGROUND_TAB);
ASSERT_EQ(4, browser()->active_index());
- ASSERT_EQ(url2, browser()->GetSelectedTabContents()->GetURL());
+ ASSERT_EQ(url2, browser()->GetSelectedWebContents()->GetURL());
}
// Tests IDC_SEARCH (the Search key on Chrome OS devices).
@@ -178,13 +178,13 @@ TEST_F(BrowserCommandsTest, Search) {
GURL non_ntp_url("http://foo/");
AddTab(browser(), non_ntp_url);
ASSERT_EQ(1, browser()->tab_count());
- EXPECT_EQ(non_ntp_url, browser()->GetSelectedTabContents()->GetURL());
+ EXPECT_EQ(non_ntp_url, browser()->GetSelectedWebContents()->GetURL());
// Pressing the Search key should open a new tab containing the NTP.
browser()->Search();
ASSERT_EQ(2, browser()->tab_count());
ASSERT_EQ(1, browser()->active_index());
- GURL current_url = browser()->GetSelectedTabContents()->GetURL();
+ GURL current_url = browser()->GetSelectedWebContents()->GetURL();
EXPECT_TRUE(current_url.SchemeIs(chrome::kChromeUIScheme));
EXPECT_EQ(chrome::kChromeUINewTabHost, current_url.host());
@@ -192,7 +192,7 @@ TEST_F(BrowserCommandsTest, Search) {
browser()->Search();
ASSERT_EQ(2, browser()->tab_count());
ASSERT_EQ(1, browser()->active_index());
- current_url = browser()->GetSelectedTabContents()->GetURL();
+ current_url = browser()->GetSelectedWebContents()->GetURL();
EXPECT_TRUE(current_url.SchemeIs(chrome::kChromeUIScheme));
EXPECT_EQ(chrome::kChromeUINewTabHost, current_url.host());
}