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

Side by Side Diff: components/bookmarks/browser/bookmark_model.cc

Issue 331163003: [Refactor] Change FaviconService API to take in a desired pixel size (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 5 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "components/bookmarks/browser/bookmark_model.h" 5 #include "components/bookmarks/browser/bookmark_model.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <functional> 8 #include <functional>
9 9
10 #include "base/bind.h" 10 #include "base/bind.h"
(...skipping 941 matching lines...) Expand 10 before | Expand all | Expand 10 after
952 } 952 }
953 953
954 void BookmarkModel::LoadFavicon( 954 void BookmarkModel::LoadFavicon(
955 BookmarkNode* node, 955 BookmarkNode* node,
956 favicon_base::IconType icon_type) { 956 favicon_base::IconType icon_type) {
957 if (node->is_folder()) 957 if (node->is_folder())
958 return; 958 return;
959 959
960 DCHECK(node->url().is_valid()); 960 DCHECK(node->url().is_valid());
961 node->set_favicon_state(BookmarkNode::LOADING_FAVICON); 961 node->set_favicon_state(BookmarkNode::LOADING_FAVICON);
962 base::CancelableTaskTracker::TaskId taskId = client_->GetFaviconImageForURL( 962 base::CancelableTaskTracker::TaskId taskId =
963 node->url(), 963 client_->GetFaviconImageForPageURL(
964 icon_type, 964 node->url(),
965 icon_type == favicon_base::FAVICON ? gfx::kFaviconSize : 0, 965 icon_type,
966 base::Bind( 966 base::Bind(
967 &BookmarkModel::OnFaviconDataAvailable, 967 &BookmarkModel::OnFaviconDataAvailable,
968 base::Unretained(this), 968 base::Unretained(this),
969 node, 969 node,
970 icon_type), 970 icon_type),
971 &cancelable_task_tracker_); 971 &cancelable_task_tracker_);
972 if (taskId != base::CancelableTaskTracker::kBadTaskId) 972 if (taskId != base::CancelableTaskTracker::kBadTaskId)
973 node->set_favicon_load_task_id(taskId); 973 node->set_favicon_load_task_id(taskId);
974 } 974 }
975 975
976 void BookmarkModel::FaviconLoaded(const BookmarkNode* node) { 976 void BookmarkModel::FaviconLoaded(const BookmarkNode* node) {
977 FOR_EACH_OBSERVER(BookmarkModelObserver, observers_, 977 FOR_EACH_OBSERVER(BookmarkModelObserver, observers_,
978 BookmarkNodeFaviconChanged(this, node)); 978 BookmarkNodeFaviconChanged(this, node));
979 } 979 }
980 980
981 void BookmarkModel::CancelPendingFaviconLoadRequests(BookmarkNode* node) { 981 void BookmarkModel::CancelPendingFaviconLoadRequests(BookmarkNode* node) {
(...skipping 25 matching lines...) Expand all
1007 BookmarkPermanentNode* mobile_node = 1007 BookmarkPermanentNode* mobile_node =
1008 CreatePermanentNode(BookmarkNode::MOBILE); 1008 CreatePermanentNode(BookmarkNode::MOBILE);
1009 return scoped_ptr<BookmarkLoadDetails>(new BookmarkLoadDetails( 1009 return scoped_ptr<BookmarkLoadDetails>(new BookmarkLoadDetails(
1010 bb_node, 1010 bb_node,
1011 other_node, 1011 other_node,
1012 mobile_node, 1012 mobile_node,
1013 client_->GetLoadExtraNodesCallback(), 1013 client_->GetLoadExtraNodesCallback(),
1014 new BookmarkIndex(client_, index_urls_, accept_languages), 1014 new BookmarkIndex(client_, index_urls_, accept_languages),
1015 next_node_id_)); 1015 next_node_id_));
1016 } 1016 }
OLDNEW
« no previous file with comments | « components/bookmarks/browser/bookmark_client.cc ('k') | ui/message_center/message_center_style.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698