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

Unified Diff: chrome/browser/favicon/favicon_tab_helper.cc

Issue 9018016: Revert r115276, it broke PanelDownloadTest.Download in interactive_ui_tests (Closed) Base URL: svn://svn.chromium.org/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/browser/favicon/favicon_tab_helper.cc
diff --git a/chrome/browser/favicon/favicon_tab_helper.cc b/chrome/browser/favicon/favicon_tab_helper.cc
index 8df87aab261818c159b0ca76e5437655023abe7c..b6402d458af31cf34ca05cde7a65e2f6fd3312f8 100644
--- a/chrome/browser/favicon/favicon_tab_helper.cc
+++ b/chrome/browser/favicon/favicon_tab_helper.cc
@@ -42,7 +42,7 @@ void FaviconTabHelper::FetchFavicon(const GURL& url) {
SkBitmap FaviconTabHelper::GetFavicon() const {
// Like GetTitle(), we also want to use the favicon for the last committed
// entry rather than a pending navigation entry.
- const NavigationController& controller = tab_contents()->GetController();
+ const NavigationController& controller = tab_contents()->controller();
NavigationEntry* entry = controller.GetTransientEntry();
if (entry)
return entry->favicon().bitmap();
@@ -54,7 +54,7 @@ SkBitmap FaviconTabHelper::GetFavicon() const {
}
bool FaviconTabHelper::FaviconIsValid() const {
- const NavigationController& controller = tab_contents()->GetController();
+ const NavigationController& controller = tab_contents()->controller();
NavigationEntry* entry = controller.GetTransientEntry();
if (entry)
return entry->favicon().is_valid();
@@ -68,7 +68,7 @@ bool FaviconTabHelper::FaviconIsValid() const {
bool FaviconTabHelper::ShouldDisplayFavicon() {
// Always display a throbber during pending loads.
- const NavigationController& controller = tab_contents()->GetController();
+ const NavigationController& controller = tab_contents()->controller();
if (controller.GetLastCommittedEntry() && controller.pending_entry())
return true;
@@ -79,7 +79,7 @@ bool FaviconTabHelper::ShouldDisplayFavicon() {
}
void FaviconTabHelper::SaveFavicon() {
- NavigationEntry* entry = tab_contents()->GetController().GetActiveEntry();
+ NavigationEntry* entry = tab_contents()->controller().GetActiveEntry();
if (!entry || entry->url().is_empty())
return;
@@ -128,7 +128,7 @@ void FaviconTabHelper::OnUpdateFaviconURL(
}
NavigationEntry* FaviconTabHelper::GetActiveEntry() {
- return tab_contents()->GetController().GetActiveEntry();
+ return tab_contents()->controller().GetActiveEntry();
}
void FaviconTabHelper::StartDownload(int id, const GURL& url, int image_size) {
« no previous file with comments | « chrome/browser/external_tab_container_win.cc ('k') | chrome/browser/geolocation/chrome_geolocation_permission_context.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698