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

Unified Diff: media/base/android/java/src/org/chromium/media/MediaDrmBridge.java

Issue 969883002: Address NewApi Android lint warnings in src/media. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 5 years, 9 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: media/base/android/java/src/org/chromium/media/MediaDrmBridge.java
diff --git a/media/base/android/java/src/org/chromium/media/MediaDrmBridge.java b/media/base/android/java/src/org/chromium/media/MediaDrmBridge.java
index 8f411ddea87e70cad7d7cc7dc60c016b00df8f23..ea97ca0e4452d2b9ccc9c093b061ef9fbd8cba7f 100644
--- a/media/base/android/java/src/org/chromium/media/MediaDrmBridge.java
+++ b/media/base/android/java/src/org/chromium/media/MediaDrmBridge.java
@@ -4,9 +4,11 @@
package org.chromium.media;
+import android.annotation.TargetApi;
import android.media.MediaCrypto;
import android.media.MediaDrm;
import android.os.AsyncTask;
+import android.os.Build;
import android.os.Handler;
import android.util.Log;
@@ -32,6 +34,7 @@ import java.util.UUID;
* sessions for a single MediaSourcePlayer.
*/
@JNINamespace("media")
+@TargetApi(Build.VERSION_CODES.KITKAT)
public class MediaDrmBridge {
// Implementation Notes:
// - A media crypto session (mMediaCryptoSession) is opened after MediaDrm
@@ -198,6 +201,7 @@ public class MediaDrmBridge {
// Create MediaCrypto object.
try {
+ // TODO: This requires KitKat. Is this class used on pre-KK devices?
if (MediaCrypto.isCryptoSchemeSupported(mSchemeUUID)) {
mMediaCrypto = new MediaCrypto(mSchemeUUID, mMediaCryptoSession);
Log.d(TAG, "MediaCrypto successfully created!");

Powered by Google App Engine
This is Rietveld 408576698