[Downloads] Expose basic download history to UI

chrome/browser/android/download/download_manager_service.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/android/download/download_manager_service.h"
 
 #include "base/android/jni_string.h"
 #include "base/location.h"
 #include "base/single_thread_task_runner.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "base/time/time.h"
 #include "chrome/browser/android/download/download_controller.h"
 #include "chrome/browser/download/download_service.h"
 #include "chrome/browser/download/download_service_factory.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/grit/generated_resources.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/download_item.h"
 #include "jni/DownloadManagerService_jni.h"
 #include "ui/base/l10n/l10n_util.h"
 
 using base::android::JavaParamRef;
 using base::android::ConvertJavaStringToUTF8;
 using base::android::ConvertUTF8ToJavaString;
 
+namespace {
+
+bool ShouldShowDownloadItem(content::DownloadItem* item) {
+  return !item->IsTemporary() &&
+      !item->GetFileNameToReportUser().empty() &&
+      !item->GetTargetFilePath().empty();
+}
+
+}  // namespace
+
 // static
 bool DownloadManagerService::RegisterDownloadManagerService(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 static jlong Init(JNIEnv* env, const JavaParamRef<jobject>& jobj) {
   Profile* profile = ProfileManager::GetActiveUserProfile();
   DownloadManagerService* service =
       new DownloadManagerService(env, jobj);
   DownloadService* download_service =
@@ skipping 30 matching lines @@
     JNIEnv* env,
     jobject obj,
     const JavaParamRef<jstring>& jdownload_guid) {
   std::string download_guid = ConvertJavaStringToUTF8(env, jdownload_guid);
   if (is_history_query_complete_)
     PauseDownloadInternal(download_guid);
   else
     EnqueueDownloadAction(download_guid, PAUSE);
 }
 
+void DownloadManagerService::GetAllDownloads(JNIEnv* env,
+                                             const JavaParamRef<jobject>& obj) {
+  if (is_history_query_complete_)
+    GetAllDownloadsInternal();
+  else
+    EnqueueDownloadAction(std::string(), INITIALIZE_UI);
+}
+
+void DownloadManagerService::GetAllDownloadsInternal() {
+  content::DownloadManager* manager = GetDownloadManager(false);
+  if (java_ref_.is_null() || !manager)
+    return;
+
+  content::DownloadManager::DownloadVector all_items;
+  manager->GetAllDownloads(&all_items);
+
+  // Determine how many items will be displayed.
+  int num_to_display = 0;
+  for (size_t i = 0; i < all_items.size(); i++) {
+    content::DownloadItem* item = all_items[i];
+    num_to_display += (ShouldShowDownloadItem(item) ? 1 : 0);
+  }
+
+  // Create a Java array of all of the visible DownloadItems.
+  JNIEnv* env = base::android::AttachCurrentThread();
+  ScopedJavaLocalRef<jclass> j_download_item_class = base::android::GetClass(

[qinmin, 2016/07/14 19:48:18]

This looks a little worse than the previous patch, i forgot to take jni array
creation into consideration. 
Sorry for all the inconvenience, revert this to the old patch would be better.

[gone, 2016/07/14 20:22:58]

No worries; I reverted most of it, but kept the function for
ShouldShowDownloadItem.  It might be helpful later.

+      env, "org/chromium/chrome/browser/download/DownloadItem");
+  ScopedJavaLocalRef<jobjectArray> j_item_array(
+      env,
+      env->NewObjectArray(
+          num_to_display, j_download_item_class.obj(), nullptr));
+
+  for (size_t i = 0, index = 0; i < all_items.size(); i++) {
+    content::DownloadItem* item = all_items[i];
+    if (!ShouldShowDownloadItem(item))
+      continue;
+
+    ScopedJavaLocalRef<jobject> j_item =
+        Java_DownloadManagerService_createDownloadItem(
+            env,
+            java_ref_.obj(),
+            ConvertUTF8ToJavaString(env, item->GetGuid()).obj(),
+            ConvertUTF8ToJavaString(
+                env,
+                item->GetFileNameToReportUser().BaseName().value()).obj(),
+            ConvertUTF8ToJavaString(env, item->GetTabUrl().spec()).obj(),
+            ConvertUTF8ToJavaString(env, item->GetMimeType()).obj(),
+            item->GetStartTime().ToJavaTime(),
+            item->GetTotalBytes());
+    env->SetObjectArrayElement(j_item_array.obj(), index++, j_item.obj());
+  }
+
+  Java_DownloadManagerService_onAllDownloadsRetrieved(
+      env, java_ref_.obj(), j_item_array.obj());
+}
+
+
 void DownloadManagerService::CancelDownload(
     JNIEnv* env,
     jobject obj,
     const JavaParamRef<jstring>& jdownload_guid,
     bool is_off_the_record,
     bool is_notification_dismissed) {
   std::string download_guid = ConvertJavaStringToUTF8(env, jdownload_guid);
 
   DownloadController::RecordDownloadCancelReason(
       is_notification_dismissed ?
@@ skipping 21 matching lines @@
     switch (action) {
       case RESUME:
         ResumeDownloadInternal(download_guid);
         break;
       case PAUSE:
         PauseDownloadInternal(download_guid);
         break;
       case CANCEL:
         CancelDownloadInternal(download_guid, false);
         break;
+      case INITIALIZE_UI:
+        GetAllDownloadsInternal();

[qinmin, 2016/07/14 19:48:18]

Do you need to check whether user has closed the download home activity at this
moment?

[gone, 2016/07/14 20:22:58]

Should be fine: the Adapter is owned by the DownloadManagerService, so checking
if that java_ref is valid is OK.

+        break;
       default:
         NOTREACHED();
         break;
     }
   }
   pending_actions_.clear();
 }
 
 void DownloadManagerService::ResumeDownloadInternal(
     const std::string& download_guid) {
@@ skipping 54 matching lines @@
       if (iter->second == PAUSE)
         iter->second = action;
       break;
     case PAUSE:
       if (iter->second == RESUME)
         iter->second = action;
       break;
     case CANCEL:
       iter->second = action;
       break;
+    case INITIALIZE_UI:
+      iter->second = action;
+      break;
     default:
       NOTREACHED();
       break;
   }
 }
 
 void DownloadManagerService::OnResumptionFailed(
     const std::string& download_guid) {
   base::ThreadTaskRunnerHandle::Get()->PostTask(
       FROM_HERE, base::Bind(&DownloadManagerService::OnResumptionFailedInternal,
@@ skipping 14 matching lines @@
     resume_callback_for_testing_.Run(false);
 }
 
 content::DownloadManager* DownloadManagerService::GetDownloadManager(
     bool is_off_the_record) {
   Profile* profile = ProfileManager::GetActiveUserProfile();
   if (is_off_the_record)
     profile = profile->GetOffTheRecordProfile();
   return content::BrowserContext::GetDownloadManager(profile);
 }