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

Unified Diff: chrome/browser/download/download_history_unittest.cc

Issue 2508503002: Fix an issue that temp files are left permanently on storage after chrome crash (Closed)
Patch Set: Created 4 years, 1 month 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/download/download_history_unittest.cc
diff --git a/chrome/browser/download/download_history_unittest.cc b/chrome/browser/download/download_history_unittest.cc
index 04a22bf56d5e498cbb83c5be617422f1f863dfd1..8f8ec4fee421fde97d5c6ecdefcfc6c6250295fe 100644
--- a/chrome/browser/download/download_history_unittest.cc
+++ b/chrome/browser/download/download_history_unittest.cc
@@ -89,7 +89,8 @@ class FakeHistoryAdapter : public DownloadHistory::HistoryAdapter {
create_download_callback_.Reset();
}
- void UpdateDownload(const history::DownloadRow& info) override {
+ void UpdateDownload(const history::DownloadRow& info,
+ bool should_commit_immediately) override {
DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
update_download_ = info;
asanka 2016/11/16 23:15:57 Store should_commit_immediately bit and test that
qinmin 2016/11/17 00:46:30 Done.
}

Powered by Google App Engine
This is Rietveld 408576698