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

Unified Diff: chrome/browser/history/history.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: fixed tests 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: chrome/browser/history/history.cc
diff --git a/chrome/browser/history/history.cc b/chrome/browser/history/history.cc
index 6957e3a402fbf86d9f66abcde4eb2bf18753a260..126cd4fcb8877b37fbb60a642253fef17c1f44e1 100644
--- a/chrome/browser/history/history.cc
+++ b/chrome/browser/history/history.cc
@@ -541,11 +541,8 @@ void HistoryService::CleanUpInProgressEntries() {
// Handle updates for a particular download. This is a 'fire and forget'
// operation, so we don't need to be called back.
-void HistoryService::UpdateDownload(int64 received_bytes,
- int32 state,
- int64 db_handle) {
- ScheduleAndForget(PRIORITY_NORMAL, &HistoryBackend::UpdateDownload,
- received_bytes, state, db_handle);
+void HistoryService::UpdateDownload(history::DownloadItemData data) {
brettw 2011/10/01 16:45:45 This should be a const ref instead of a copy.
benjhayden 2011/10/03 20:54:39 Done.
+ ScheduleAndForget(PRIORITY_NORMAL, &HistoryBackend::UpdateDownload, data);
}
void HistoryService::UpdateDownloadPath(const FilePath& path,

Powered by Google App Engine
This is Rietveld 408576698