| Index: chrome/browser/bookmarks/bookmark_html_writer.cc
|
| diff --git a/chrome/browser/bookmarks/bookmark_html_writer.cc b/chrome/browser/bookmarks/bookmark_html_writer.cc
|
| index 8b0e9790af88c57c32da174d326a05129c650d4d..ba70ed09c731e5bb38d39a5a37b864d7d096cade 100644
|
| --- a/chrome/browser/bookmarks/bookmark_html_writer.cc
|
| +++ b/chrome/browser/bookmarks/bookmark_html_writer.cc
|
| @@ -439,7 +439,8 @@ bool BookmarkFaviconFetcher::FetchNextFavicon() {
|
| if (favicons_map_->end() == iter) {
|
| FaviconService* favicon_service =
|
| profile_->GetFaviconService(Profile::EXPLICIT_ACCESS);
|
| - favicon_service->GetFaviconForURL(GURL(url), &favicon_consumer_,
|
| + favicon_service->GetFaviconForURL(GURL(url), history::FAVICON,
|
| + &favicon_consumer_,
|
| NewCallback(this, &BookmarkFaviconFetcher::OnFaviconDataAvailable));
|
| return true;
|
| } else {
|
| @@ -451,17 +452,14 @@ bool BookmarkFaviconFetcher::FetchNextFavicon() {
|
|
|
| void BookmarkFaviconFetcher::OnFaviconDataAvailable(
|
| FaviconService::Handle handle,
|
| - bool know_favicon,
|
| - scoped_refptr<RefCountedMemory> data,
|
| - bool expired,
|
| - GURL icon_url) {
|
| + history::FaviconData favicon) {
|
| GURL url;
|
| if (!bookmark_urls_.empty()) {
|
| url = GURL(bookmark_urls_.front());
|
| bookmark_urls_.pop_front();
|
| }
|
| - if (know_favicon && data.get() && data->size() && !url.is_empty()) {
|
| - favicons_map_->insert(make_pair(url.spec(), data));
|
| + if (favicon.is_valid() && !url.is_empty()) {
|
| + favicons_map_->insert(make_pair(url.spec(), favicon.image_data));
|
| }
|
|
|
| if (FetchNextFavicon()) {
|
|
|