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

Unified Diff: chrome/test/base/ui_test_utils.cc

Issue 8956050: Rename TabContents::controller() to GetController and put it into the WebContents interface. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 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 side-by-side diff with in-line comments
Download patch
Index: chrome/test/base/ui_test_utils.cc
===================================================================
--- chrome/test/base/ui_test_utils.cc (revision 115228)
+++ chrome/test/base/ui_test_utils.cc (working copy)
@@ -301,7 +301,7 @@
TabContents* tab_contents = browser->GetSelectedTabContents();
if (!tab_contents)
return false;
- NavigationEntry* last_entry = tab_contents->controller().GetActiveEntry();
+ NavigationEntry* last_entry = tab_contents->GetController().GetActiveEntry();
if (!last_entry)
return false;
title->assign(last_entry->GetTitleForDisplay(""));
@@ -335,7 +335,7 @@
void WaitForLoadStop(TabContents* tab) {
WindowedNotificationObserver load_stop_observer(
content::NOTIFICATION_LOAD_STOP,
- content::Source<NavigationController>(&tab->controller()));
+ content::Source<NavigationController>(&tab->GetController()));
// In many cases, the load may have finished before we get here. Only wait if
// the tab still has a pending navigation.
if (!tab->IsLoading())
@@ -365,7 +365,7 @@
Browser::OpenURLOffTheRecord(profile, url);
Browser* browser = BrowserList::FindTabbedBrowser(
profile->GetOffTheRecordProfile(), false);
- WaitForNavigations(&browser->GetSelectedTabContents()->controller(), 1);
+ WaitForNavigations(&browser->GetSelectedTabContents()->GetController(), 1);
}
void NavigateToURL(browser::NavigateParams* params) {
@@ -398,7 +398,7 @@
WaitForLoadStop(browser->GetSelectedTabContents());
TestNavigationObserver same_tab_observer(
content::Source<NavigationController>(
- &browser->GetSelectedTabContents()->controller()),
+ &browser->GetSelectedTabContents()->GetController()),
NULL,
number_of_navigations);
@@ -443,7 +443,7 @@
base::Unretained(MessageLoopForUI::current())));
return;
} else if (tab_contents) {
- NavigationController* controller = &tab_contents->controller();
+ NavigationController* controller = &tab_contents->GetController();
WaitForNavigations(controller, number_of_navigations);
return;
}
@@ -922,7 +922,7 @@
notification_registrar_.Add(
this,
content::NOTIFICATION_LOAD_STOP,
- content::Source<NavigationController>(&tab_contents->controller()));
+ content::Source<NavigationController>(&tab_contents->GetController()));
}
void TitleWatcher::AlsoWaitForTitle(const string16& expected_title) {
@@ -949,7 +949,7 @@
} else if (type == content::NOTIFICATION_LOAD_STOP) {
NavigationController* controller =
content::Source<NavigationController>(source).ptr();
- ASSERT_EQ(&tab_contents_->controller(), controller);
+ ASSERT_EQ(&tab_contents_->GetController(), controller);
} else {
FAIL() << "Unexpected notification received.";
}
« no previous file with comments | « chrome/test/base/test_tab_strip_model_observer.cc ('k') | content/browser/child_process_security_policy_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698