summaryrefslogtreecommitdiffstats
path: root/chrome/browser/ui/toolbar/back_forward_menu_model.cc
diff options
context:
space:
mode:
authorjam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2011-12-25 03:59:51 +0000
committerjam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2011-12-25 03:59:51 +0000
commit36fc039f12944d61df900690238a5d0303f457dd (patch)
tree1f53b1b2a137bb60bf4e696dd5e6cb706833cc2d /chrome/browser/ui/toolbar/back_forward_menu_model.cc
parent12f75ab37393f57fcac33c39673793f1579b665c (diff)
downloadchromium_src-36fc039f12944d61df900690238a5d0303f457dd.zip
chromium_src-36fc039f12944d61df900690238a5d0303f457dd.tar.gz
chromium_src-36fc039f12944d61df900690238a5d0303f457dd.tar.bz2
Create a Content API around NavigationEntry that's in content/public/browser. NavigationEntry will be renamed to NavigationEntryImpl in a later change. It now derives from content::NavigationEntry. Most of this change is changing unix_hacker functions to CamelCase and updating callers.
BUG=98716 TBR=brettw Review URL: http://codereview.chromium.org/9007043 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115766 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/ui/toolbar/back_forward_menu_model.cc')
-rw-r--r--chrome/browser/ui/toolbar/back_forward_menu_model.cc16
1 files changed, 8 insertions, 8 deletions
diff --git a/chrome/browser/ui/toolbar/back_forward_menu_model.cc b/chrome/browser/ui/toolbar/back_forward_menu_model.cc
index c3b30d2..410fd47 100644
--- a/chrome/browser/ui/toolbar/back_forward_menu_model.cc
+++ b/chrome/browser/ui/toolbar/back_forward_menu_model.cc
@@ -225,20 +225,20 @@ void BackForwardMenuModel::SetMenuModelDelegate(
void BackForwardMenuModel::FetchFavicon(NavigationEntry* entry) {
// If the favicon has already been requested for this menu, don't do
// anything.
- if (requested_favicons_.find(entry->unique_id()) !=
+ if (requested_favicons_.find(entry->GetUniqueID()) !=
requested_favicons_.end()) {
return;
}
- requested_favicons_.insert(entry->unique_id());
+ requested_favicons_.insert(entry->GetUniqueID());
FaviconService* favicon_service =
browser_->profile()->GetFaviconService(Profile::EXPLICIT_ACCESS);
if (!favicon_service)
return;
FaviconService::Handle handle = favicon_service->GetFaviconForURL(
- entry->url(), history::FAVICON, &load_consumer_,
+ entry->GetURL(), history::FAVICON, &load_consumer_,
base::Bind(&BackForwardMenuModel::OnFavIconDataAvailable,
base::Unretained(this)));
- load_consumer_.SetClientData(favicon_service, handle, entry->unique_id());
+ load_consumer_.SetClientData(favicon_service, handle, entry->GetUniqueID());
}
void BackForwardMenuModel::OnFavIconDataAvailable(
@@ -252,7 +252,7 @@ void BackForwardMenuModel::OnFavIconDataAvailable(
for (int i = 0; i < GetItemCount() - 1; i++) {
if (IsSeparator(i))
continue;
- if (GetNavigationEntry(i)->unique_id() == unique_id) {
+ if (GetNavigationEntry(i)->GetUniqueID() == unique_id) {
model_index = i;
entry = GetNavigationEntry(i);
break;
@@ -348,14 +348,14 @@ int BackForwardMenuModel::GetIndexOfNextChapterStop(int start_from,
}
NavigationEntry* start_entry = controller.GetEntryAtIndex(start_from);
- const GURL& url = start_entry->url();
+ const GURL& url = start_entry->GetURL();
if (!forward) {
// When going backwards we return the first entry we find that has a
// different domain.
for (int i = start_from - 1; i >= 0; --i) {
if (!net::RegistryControlledDomainService::SameDomainOrHost(url,
- controller.GetEntryAtIndex(i)->url()))
+ controller.GetEntryAtIndex(i)->GetURL()))
return i;
}
// We have reached the beginning without finding a chapter stop.
@@ -365,7 +365,7 @@ int BackForwardMenuModel::GetIndexOfNextChapterStop(int start_from,
// different domain.
for (int i = start_from + 1; i < max_count; ++i) {
if (!net::RegistryControlledDomainService::SameDomainOrHost(url,
- controller.GetEntryAtIndex(i)->url()))
+ controller.GetEntryAtIndex(i)->GetURL()))
return i - 1;
}
// Last entry is always considered a chapter stop.