Adds support for saving favicon size into history database.

chrome/browser/bookmarks/bookmark_model.cc

Index: chrome/browser/bookmarks/bookmark_model.cc
diff --git a/chrome/browser/bookmarks/bookmark_model.cc b/chrome/browser/bookmarks/bookmark_model.cc
index b6497a09f1d37fd89e5368c29a2713a97c0f0798..af0992ccd1ba15e522e804e36757a0e3a2b3b857 100644
--- a/chrome/browser/bookmarks/bookmark_model.cc
+++ b/chrome/browser/bookmarks/bookmark_model.cc
@@ -776,16 +776,18 @@ BookmarkPermanentNode* BookmarkModel::CreatePermanentNode(
 
 void BookmarkModel::OnFaviconDataAvailable(
     FaviconService::Handle handle,
-    history::FaviconData favicon) {
+    history::FaviconData favicon_data) {
   BookmarkNode* node =
       load_consumer_.GetClientData(
           profile_->GetFaviconService(Profile::EXPLICIT_ACCESS), handle);
   DCHECK(node);
   node->set_favicon_load_handle(0);
-  if (favicon.is_valid()) {
+  if (favicon_data.is_valid()) {
+    scoped_refptr<base::RefCountedMemory> bitmap_data =
+        favicon_data.elements[0].bitmap_data;

[stevenjb, 2012/08/15 22:59:18]

nit: use constructor instead of assignment

     scoped_ptr<gfx::Image> favicon_image(
-        gfx::ImageFromPNGEncodedData(favicon.image_data->front(),
-                                     favicon.image_data->size()));
+        gfx::ImageFromPNGEncodedData(bitmap_data->front(),
+                                     bitmap_data->size()));
     if (favicon_image.get()) {
       node->set_favicon(*favicon_image.get());
       FaviconLoaded(node);