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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/download/DownloadUtils.java

Issue 2640023008: Enabling autoplay and fullscreen for downloaded media (Closed)
Patch Set: removed include 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/android/java/src/org/chromium/chrome/browser/download/DownloadUtils.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadUtils.java b/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadUtils.java
index f6bfd8eb47488f34ed14db22d72e1518857e0a4d..95e15539b20546aa8b97e1b408a2ed3439161748 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadUtils.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadUtils.java
@@ -73,6 +73,7 @@ public class DownloadUtils {
private static final String DEFAULT_MIME_TYPE = "*/*";
private static final String MIME_TYPE_DELIMITER = "/";
+ private static final String MIME_TYPE_VIDEO = "video";
private static final String EXTRA_IS_OFF_THE_RECORD =
"org.chromium.chrome.browser.download.IS_OFF_THE_RECORD";
@@ -420,6 +421,8 @@ public class DownloadUtils {
intent.setData(contentUri);
intent.putExtra(CustomTabIntentDataProvider.EXTRA_IS_MEDIA_VIEWER, true);
intent.putExtra(CustomTabIntentDataProvider.EXTRA_MEDIA_VIEWER_URL, fileUri.toString());
+ intent.putExtra(CustomTabIntentDataProvider.EXTRA_ENABLE_EMBEDDED_MEDIA_EXPERIENCE,
+ isMimeTypeVideo(mimeType));
intent.putExtra(
CustomTabIntentDataProvider.EXTRA_INITIAL_BACKGROUND_COLOR, mediaColor);
intent.putExtra(
@@ -699,4 +702,13 @@ public class DownloadUtils {
return ApiCompatibilityUtils.getColorStateList(
context.getResources(), R.color.white_mode_tint);
}
+
+ private static boolean isMimeTypeVideo(String mimeType) {
+ if (TextUtils.isEmpty(mimeType)) return false;
+
+ String[] pieces = mimeType.split(MIME_TYPE_DELIMITER);
+ if (pieces.length != 2) return false;
+
+ return MIME_TYPE_VIDEO.equals(pieces[0]);
+ }
}

Powered by Google App Engine
This is Rietveld 408576698