Add new UMA stats to get a handle on Downloads UI Usage

chrome/browser/history/history_unittest.cc

Index: chrome/browser/history/history_unittest.cc
diff --git a/chrome/browser/history/history_unittest.cc b/chrome/browser/history/history_unittest.cc
index d5ce21a7820915157eb4eae99c3b3f3d78d0889d..684eed5b06a83d3f582d5055a4d855440edcfc88 100644
--- a/chrome/browser/history/history_unittest.cc
+++ b/chrome/browser/history/history_unittest.cc
@@ -193,9 +193,11 @@ class HistoryTest : public testing::Test {
         GURL("foo-url"),
         GURL(""),
         time,
+        time,
         0,
         512,
         state,
+        0,
         0);
     return db_->CreateDownload(download);
   }
@@ -369,8 +371,16 @@ TEST_F(HistoryTest, ClearBrowsingData_Downloads) {
             downloads[2].start_time.ToInternalValue());
 
   // Change state so we can delete the downloads.
-  EXPECT_TRUE(db_->UpdateDownload(512, DownloadItem::COMPLETE, in_progress));
-  EXPECT_TRUE(db_->UpdateDownload(512, DownloadItem::CANCELLED, removing));
+  DownloadPersistentStoreInfo data;
+  data.received_bytes = 512;
+  data.state = DownloadItem::COMPLETE;
+  data.end_time = base::Time::Now();
+  data.opened = false;
+  data.db_handle = in_progress;
+  EXPECT_TRUE(db_->UpdateDownload(data));
+  data.state = DownloadItem::CANCELLED;
+  data.db_handle = removing;
+  EXPECT_TRUE(db_->UpdateDownload(data));
 
   // Try removing from Time=0. This should delete all.
   db_->RemoveDownloadsBetween(Time(), Time());