Save large icons to a new LARGE_ICON type

components/history/core/browser/thumbnail_database.cc

Index: components/history/core/browser/thumbnail_database.cc
diff --git a/components/history/core/browser/thumbnail_database.cc b/components/history/core/browser/thumbnail_database.cc
index 7fc68d43b7bb7e5437db4a18d79a51e88d223129..258a3b076bd2577c10dd1a0518c278f420311450 100644
--- a/components/history/core/browser/thumbnail_database.cc
+++ b/components/history/core/browser/thumbnail_database.cc
@@ -660,7 +660,7 @@ void ThumbnailDatabase::ComputeDatabaseMetrics() {
             SQL_FROM_HERE,
             "SELECT COUNT(*) FROM favicons WHERE icon_type IN (?, ?)"));
     touch_icon_count.BindInt64(0, favicon_base::TOUCH_ICON);
-    touch_icon_count.BindInt64(0, favicon_base::TOUCH_PRECOMPOSED_ICON);
+    touch_icon_count.BindInt64(1, favicon_base::TOUCH_PRECOMPOSED_ICON);

[huangs, 2015/05/04 05:17:30]

This looks like an independent bug fix. I'd recommend having this as a separate
CL, and having it sped through the review process.  Also please see how this
changes UMA, and ping michaelbai@ (or whoever wrote this).

[Roger McFarlane (Chromium), 2015/05/04 18:48:15]

Moved to a separate CL.

This is a new histogram I added a couple of weeks back. This was just an error
that made it through review. No one else is consuming this histogram yet.

     UMA_HISTOGRAM_COUNTS_10000(
         "History.NumTouchIconsInDB",
         touch_icon_count.Step() ? touch_icon_count.ColumnInt(0) : 0);