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

Unified Diff: chrome/browser/android/download/download_manager_service.cc

Issue 2720613002: Downloads: Added transient flag to download item and download database (Closed)
Patch Set: Fixed migration unit test Created 3 years, 10 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/android/download/download_manager_service.cc
diff --git a/chrome/browser/android/download/download_manager_service.cc b/chrome/browser/android/download/download_manager_service.cc
index 5c825b5ae6ec26a57de1fbc6727e0ff90a0ea42c..f748c0cb110c6c19e3fe1944eec321fafafdc6fc 100644
--- a/chrome/browser/android/download/download_manager_service.cc
+++ b/chrome/browser/android/download/download_manager_service.cc
@@ -122,7 +122,7 @@ ScopedJavaLocalRef<jobject> DownloadManagerService::CreateJavaDownloadInfo(
ConvertUTF8ToJavaString(env, item->GetReferrerUrl().spec()),
time_remaining_known ? time_delta.InMilliseconds()
: kUnknownRemainingTime,
- item->GetLastAccessTime().ToJavaTime());
+ item->GetLastAccessTime().ToJavaTime(), item->IsVisible());
David Trainor- moved to gerrit 2017/03/06 18:28:39 Should visible items make it past this point? May
shaktisahu 2017/03/08 06:51:30 Removed.
}
static jlong Init(JNIEnv* env, const JavaParamRef<jobject>& jobj) {

Powered by Google App Engine
This is Rietveld 408576698