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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/browser/bookmarks/bookmark_model.cc

Issue 10802066: Adds support for saving favicon size into history database. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/bookmarks/bookmark_html_writer.cc ('k') | chrome/browser/extensions/extension_web_ui.cc » ('j') | chrome/browser/history/android/favicon_sql_handler.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/bookmarks/bookmark_model.cc
diff --git a/chrome/browser/bookmarks/bookmark_model.cc b/chrome/browser/bookmarks/bookmark_model.cc
index f1a14c8d6722711b39fd113ee352bf4c68da52b1..5aa795ac8fb1938c9d27cc77067bfab323505eb2 100644
--- a/chrome/browser/bookmarks/bookmark_model.cc
+++ b/chrome/browser/bookmarks/bookmark_model.cc
@@ -779,8 +779,8 @@ void BookmarkModel::OnFaviconDataAvailable(
node->set_favicon_load_handle(0);
if (favicon.is_valid()) {
scoped_ptr<gfx::Image> favicon_image(
- gfx::ImageFromPNGEncodedData(favicon.image_data->front(),
- favicon.image_data->size()));
+ gfx::ImageFromPNGEncodedData(favicon.bitmap_data->front(),
+ favicon.bitmap_data->size()));
if (favicon_image.get()) {
node->set_favicon(*favicon_image.get());
FaviconLoaded(node);
« no previous file with comments | « chrome/browser/bookmarks/bookmark_html_writer.cc ('k') | chrome/browser/extensions/extension_web_ui.cc » ('j') | chrome/browser/history/android/favicon_sql_handler.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698