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

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

Issue 2343363005: [Download Home] Open supported files in Custom Tabs (Closed)
Patch Set: Comments Created 4 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
« no previous file with comments | « chrome/browser/android/download/download_manager_service.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 249a7727797d4ea881d9cc9312c81b6467134f80..ce684ae65caaac7e129c6dbd24671efe56d34100 100644
--- a/chrome/browser/android/download/download_manager_service.cc
+++ b/chrome/browser/android/download/download_manager_service.cc
@@ -14,6 +14,7 @@
#include "chrome/browser/download/download_service_factory.h"
#include "chrome/browser/profiles/profile_manager.h"
#include "chrome/grit/generated_resources.h"
+#include "components/mime_util/mime_util.h"
#include "content/public/browser/browser_context.h"
#include "content/public/browser/download_item.h"
#include "jni/DownloadManagerService_jni.h"
@@ -126,6 +127,23 @@ void DownloadManagerService::RemoveDownload(
EnqueueDownloadAction(download_guid, REMOVE);
}
+bool DownloadManagerService::IsDownloadOpenableInBrowser(
+ JNIEnv* env,
+ jobject obj,
+ const JavaParamRef<jstring>& jdownload_guid,
+ bool is_off_the_record) {
+ std::string download_guid = ConvertJavaStringToUTF8(env, jdownload_guid);
+ content::DownloadManager* manager = GetDownloadManager(is_off_the_record);
+ if (!manager)
+ return false;
+
+ content::DownloadItem* item = manager->GetDownloadByGuid(download_guid);
+ if (!item)
+ return false;
+
+ return mime_util::IsSupportedMimeType(item->GetMimeType());
+}
+
void DownloadManagerService::GetAllDownloads(JNIEnv* env,
const JavaParamRef<jobject>& obj,
bool is_off_the_record) {
« no previous file with comments | « chrome/browser/android/download/download_manager_service.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698