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

Unified Diff: content/browser/download/download_manager.cc

Issue 8008021: Add new UMA stats to get a handle on Downloads UI Usage (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: comments Created 9 years, 3 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
Index: content/browser/download/download_manager.cc
diff --git a/content/browser/download/download_manager.cc b/content/browser/download/download_manager.cc
index e3235aea22fd6ecfb8f37603ce14f577d2ef4352..54d6a3cd8e2efe1cb17fdcd0313f051af9cf11ef 100644
--- a/content/browser/download/download_manager.cc
+++ b/content/browser/download/download_manager.cc
@@ -25,6 +25,7 @@
#include "content/browser/download/download_item.h"
#include "content/browser/download/download_manager_delegate.h"
#include "content/browser/download/download_persistent_store_info.h"
+#include "content/browser/download/download_stats.h"
#include "content/browser/download/download_status_updater.h"
#include "content/browser/renderer_host/render_process_host.h"
#include "content/browser/renderer_host/render_view_host.h"
@@ -682,6 +683,7 @@ int DownloadManager::RemoveDownloads(const base::Time remove_begin) {
}
int DownloadManager::RemoveAllDownloads() {
+ download_stats::RecordClearAllSize(history_downloads_.size());
// The null times make the date range unbounded.
return RemoveDownloadsBetween(base::Time(), base::Time());
}
@@ -817,6 +819,8 @@ void DownloadManager::OnPersistentStoreQueryComplete(
// http://crbug.com/84508 is fixed.
largest_db_handle_in_history_ = 0;
+ download_stats::RecordHistorySize(entries->size());
+
for (size_t i = 0; i < entries->size(); ++i) {
DownloadItem* download = new DownloadItem(this, entries->at(i));
// TODO(rdsmith): Remove after http://crbug.com/85408 resolved.
@@ -841,6 +845,8 @@ void DownloadManager::AddDownloadItemToHistory(DownloadItem* download,
// is fixed.
CHECK_NE(DownloadItem::kUninitializedHandle, db_handle);
+ download_stats::RecordHistorySize2(history_downloads_.size());
+
DCHECK(download->db_handle() == DownloadItem::kUninitializedHandle);
download->set_db_handle(db_handle);
@@ -1068,3 +1074,7 @@ void DownloadManager::SavePageDownloadFinished(DownloadItem* download) {
Details<DownloadItem>(download));
}
}
+
+void DownloadManager::MarkDownloadOpened(DownloadItem* download) {
+ delegate_->UpdateItemInPersistentStore(download);
+}

Powered by Google App Engine
This is Rietveld 408576698