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

Unified Diff: chrome/browser/sync/glue/favicon_cache.cc

Issue 137263007: Move CancelableTaskTracker to //base/task/CancelableTaskTracker. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Move to base/task/cancelable_task_tracker* Created 6 years, 10 months 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
« no previous file with comments | « chrome/browser/sync/glue/favicon_cache.h ('k') | chrome/browser/ui/android/navigation_popup.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/sync/glue/favicon_cache.cc
diff --git a/chrome/browser/sync/glue/favicon_cache.cc b/chrome/browser/sync/glue/favicon_cache.cc
index f250d9b4c5c88f1a95c7e75f84f5a08994912748..7c9256b75fd7d49cf836ba27a49e4661a20e3a1c 100644
--- a/chrome/browser/sync/glue/favicon_cache.cc
+++ b/chrome/browser/sync/glue/favicon_cache.cc
@@ -467,11 +467,12 @@ void FaviconCache::OnPageFaviconUpdated(const GURL& page_url) {
// TODO(zea): This appears to only fetch one favicon (best match based on
// desired_size_in_dip). Figure out a way to fetch all favicons we support.
// See crbug.com/181068.
- CancelableTaskTracker::TaskId id = favicon_service->GetFaviconForURL(
- FaviconService::FaviconForURLParams(page_url, SupportedFaviconTypes(),
- kMaxFaviconResolution),
+ base::CancelableTaskTracker::TaskId id = favicon_service->GetFaviconForURL(
+ FaviconService::FaviconForURLParams(
+ page_url, SupportedFaviconTypes(), kMaxFaviconResolution),
base::Bind(&FaviconCache::OnFaviconDataAvailable,
- weak_ptr_factory_.GetWeakPtr(), page_url),
+ weak_ptr_factory_.GetWeakPtr(),
+ page_url),
&cancelable_task_tracker_);
page_task_map_[page_url] = id;
}
« no previous file with comments | « chrome/browser/sync/glue/favicon_cache.h ('k') | chrome/browser/ui/android/navigation_popup.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698