diff options
author | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-12-21 05:09:04 +0000 |
---|---|---|
committer | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-12-21 05:09:04 +0000 |
commit | 69cda6547de3c2bc25e6476607e7550ba548cac9 (patch) | |
tree | e54f39838f04e0e61921cd7457711019be8b545f /content/browser/tab_contents/tab_contents_unittest.cc | |
parent | c691a593ac199d774a43f239342cd85c80ca1a2a (diff) | |
download | chromium_src-69cda6547de3c2bc25e6476607e7550ba548cac9.zip chromium_src-69cda6547de3c2bc25e6476607e7550ba548cac9.tar.gz chromium_src-69cda6547de3c2bc25e6476607e7550ba548cac9.tar.bz2 |
Rename TabContents::controller() to GetController and put it into the WebContents namespace.
BUG=98716
TBR=dpranke
Review URL: http://codereview.chromium.org/8956050
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115276 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'content/browser/tab_contents/tab_contents_unittest.cc')
-rw-r--r-- | content/browser/tab_contents/tab_contents_unittest.cc | 30 |
1 files changed, 15 insertions, 15 deletions
diff --git a/content/browser/tab_contents/tab_contents_unittest.cc b/content/browser/tab_contents/tab_contents_unittest.cc index 8d6470e..d014cd5 100644 --- a/content/browser/tab_contents/tab_contents_unittest.cc +++ b/content/browser/tab_contents/tab_contents_unittest.cc @@ -464,9 +464,9 @@ TEST_F(TabContentsTest, NavigateTwoTabsCrossSite) { // Open a new tab with the same SiteInstance, navigated to the same site. TestTabContents contents2(browser_context_.get(), instance1); contents2.transition_cross_site = true; - contents2.controller().LoadURL(url, content::Referrer(), - content::PAGE_TRANSITION_TYPED, - std::string()); + contents2.GetController().LoadURL(url, content::Referrer(), + content::PAGE_TRANSITION_TYPED, + std::string()); // Need this page id to be 2 since the site instance is the same (which is the // scope of page IDs) and we want to consider this a new page. contents2.TestDidNavigate( @@ -486,9 +486,9 @@ TEST_F(TabContentsTest, NavigateTwoTabsCrossSite) { // Navigate second tab to the same site as the first tab const GURL url2b("http://mail.yahoo.com"); - contents2.controller().LoadURL(url2b, content::Referrer(), - content::PAGE_TRANSITION_TYPED, - std::string()); + contents2.GetController().LoadURL(url2b, content::Referrer(), + content::PAGE_TRANSITION_TYPED, + std::string()); TestRenderViewHost* rvh2 = static_cast<TestRenderViewHost*>(contents2.GetRenderViewHost()); rvh2->SendShouldCloseACK(true); @@ -526,9 +526,9 @@ TEST_F(TabContentsTest, CrossSiteComparesAgainstCurrentPage) { TestTabContents contents2(browser_context_.get(), instance1); contents2.transition_cross_site = true; const GURL url2("http://www.yahoo.com"); - contents2.controller().LoadURL(url2, content::Referrer(), - content::PAGE_TRANSITION_TYPED, - std::string()); + contents2.GetController().LoadURL(url2, content::Referrer(), + content::PAGE_TRANSITION_TYPED, + std::string()); // The first RVH in contents2 isn't live yet, so we shortcut the cross site // pending. TestRenderViewHost* rvh2 = static_cast<TestRenderViewHost*>( @@ -1640,7 +1640,7 @@ TEST_F(TabContentsTest, NewInterstitialDoesNotCancelPendingEntry) { const GURL kGURL(kUrl); // Start a navigation to a page - contents()->controller().LoadURL( + contents()->GetController().LoadURL( kGURL, content::Referrer(), content::PAGE_TRANSITION_TYPED, std::string()); @@ -1656,7 +1656,7 @@ TEST_F(TabContentsTest, NewInterstitialDoesNotCancelPendingEntry) { // Initiate a new navigation from the browser that also triggers an // interstitial. - contents()->controller().LoadURL( + contents()->GetController().LoadURL( kGURL, content::Referrer(), content::PAGE_TRANSITION_TYPED, std::string()); TestInterstitialPage::InterstitialState state2 = @@ -1669,7 +1669,7 @@ TEST_F(TabContentsTest, NewInterstitialDoesNotCancelPendingEntry) { interstitial2->TestDidNavigate(1, kGURL); // Make sure we still have an entry. - NavigationEntry* entry = contents()->controller().pending_entry(); + NavigationEntry* entry = contents()->GetController().pending_entry(); ASSERT_TRUE(entry); EXPECT_EQ(kUrl, entry->url().spec()); @@ -1687,7 +1687,7 @@ TEST_F(TabContentsTest, NoJSMessageOnInterstitials) { const GURL kGURL(kUrl); // Start a navigation to a page - contents()->controller().LoadURL( + contents()->GetController().LoadURL( kGURL, content::Referrer(), content::PAGE_TRANSITION_TYPED, std::string()); // DidNavigate from the page @@ -1742,7 +1742,7 @@ TEST_F(TabContentsTest, CopyStateFromAndPruneSourceInterstitial) { // Create another NavigationController. GURL url3("http://foo2"); scoped_ptr<TestTabContents> other_contents(CreateTestTabContents()); - NavigationController& other_controller = other_contents->controller(); + NavigationController& other_controller = other_contents->GetController(); other_contents->NavigateAndCommit(url3); other_contents->ExpectSetHistoryLengthAndPrune( other_controller.GetEntryAtIndex(0)->site_instance(), 1, @@ -1768,7 +1768,7 @@ TEST_F(TabContentsTest, CopyStateFromAndPruneTargetInterstitial) { // Create another NavigationController. scoped_ptr<TestTabContents> other_contents(CreateTestTabContents()); - NavigationController& other_controller = other_contents->controller(); + NavigationController& other_controller = other_contents->GetController(); // Navigate it to url2. GURL url2("http://foo2"); |