diff options
author | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-12-25 03:59:51 +0000 |
---|---|---|
committer | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-12-25 03:59:51 +0000 |
commit | 36fc039f12944d61df900690238a5d0303f457dd (patch) | |
tree | 1f53b1b2a137bb60bf4e696dd5e6cb706833cc2d /chrome/browser/ui/toolbar/back_forward_menu_model.cc | |
parent | 12f75ab37393f57fcac33c39673793f1579b665c (diff) | |
download | chromium_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.cc | 16 |
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. |