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

Unified Diff: media/base/android/media_drm_bridge.cc

Issue 2796843002: [Clank] Load/Remove persistent license (Closed)
Patch Set: Bug fix Created 3 years, 8 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/media_drm_bridge.cc
diff --git a/media/base/android/media_drm_bridge.cc b/media/base/android/media_drm_bridge.cc
index 83b14ae185b0d04bb7ccc4063daeb70fc25f4035..cb8855bf0cd343d4e621571835781893dc9f8a68 100644
--- a/media/base/android/media_drm_bridge.cc
+++ b/media/base/android/media_drm_bridge.cc
@@ -124,12 +124,14 @@ ContentDecryptionModule::MessageType GetMessageType(RequestType request_type) {
return ContentDecryptionModule::LICENSE_REQUEST;
}
-CdmKeyInformation::KeyStatus ConvertKeyStatus(KeyStatus key_status) {
+CdmKeyInformation::KeyStatus ConvertKeyStatus(KeyStatus key_status,
+ bool is_key_release) {
switch (key_status) {
case KeyStatus::KEY_STATUS_USABLE:
return CdmKeyInformation::USABLE;
case KeyStatus::KEY_STATUS_EXPIRED:
- return CdmKeyInformation::EXPIRED;
+ return is_key_release ? CdmKeyInformation::RELEASED
+ : CdmKeyInformation::EXPIRED;
case KeyStatus::KEY_STATUS_OUTPUT_NOT_ALLOWED:
return CdmKeyInformation::OUTPUT_RESTRICTED;
case KeyStatus::KEY_STATUS_PENDING:
@@ -462,9 +464,19 @@ void MediaDrmBridge::LoadSession(
DCHECK(base::FeatureList::IsEnabled(kMediaDrmPersistentLicense));
- NOTIMPLEMENTED() << "EME persistent sessions not yet supported on Android.";
- promise->reject(CdmPromise::NOT_SUPPORTED_ERROR, 0,
- "LoadSession() is not supported.");
+ if (session_type != CdmSessionType::PERSISTENT_LICENSE_SESSION) {
+ promise->reject(
+ CdmPromise::NOT_SUPPORTED_ERROR, 0,
+ "LoadSession() is only supported for 'persistent-license'.");
+ return;
+ }
+
+ JNIEnv* env = AttachCurrentThread();
+ ScopedJavaLocalRef<jbyteArray> j_session_id = base::android::ToJavaByteArray(
+ env, reinterpret_cast<const uint8_t*>(session_id.data()),
+ session_id.size());
+ uint32_t promise_id = cdm_promise_adapter_.SavePromise(std::move(promise));
+ Java_MediaDrmBridge_loadSession(env, j_media_drm_, j_session_id, promise_id);
}
void MediaDrmBridge::UpdateSession(
@@ -505,9 +517,13 @@ void MediaDrmBridge::RemoveSession(
DCHECK(task_runner_->BelongsToCurrentThread());
DVLOG(2) << __func__;
- NOTIMPLEMENTED() << "EME persistent sessions not yet supported on Android.";
- promise->reject(CdmPromise::NOT_SUPPORTED_ERROR, 0,
- "RemoveSession() is not supported.");
+ JNIEnv* env = AttachCurrentThread();
+ ScopedJavaLocalRef<jbyteArray> j_session_id = base::android::ToJavaByteArray(
+ env, reinterpret_cast<const uint8_t*>(session_id.data()),
+ session_id.size());
+ uint32_t promise_id = cdm_promise_adapter_.SavePromise(std::move(promise));
+ Java_MediaDrmBridge_removeSession(env, j_media_drm_, j_session_id,
+ promise_id);
}
CdmContext* MediaDrmBridge::GetCdmContext() {
@@ -698,7 +714,8 @@ void MediaDrmBridge::OnSessionKeysChange(
const JavaParamRef<jobject>& j_media_drm,
const JavaParamRef<jbyteArray>& j_session_id,
const JavaParamRef<jobjectArray>& j_keys_info,
- bool has_additional_usable_key) {
+ bool has_additional_usable_key,
+ bool is_key_release) {
DVLOG(2) << __func__;
CdmKeysInfo cdm_keys_info;
@@ -718,7 +735,7 @@ void MediaDrmBridge::OnSessionKeysChange(
jint j_status_code = Java_KeyStatus_getStatusCode(env, j_key_status);
CdmKeyInformation::KeyStatus key_status =
- ConvertKeyStatus(static_cast<KeyStatus>(j_status_code));
+ ConvertKeyStatus(static_cast<KeyStatus>(j_status_code), is_key_release);
DVLOG(2) << __func__ << "Key status change: "
<< base::HexEncode(&key_id[0], key_id.size()) << ", "
« media/base/android/media_drm_bridge.h ('K') | « media/base/android/media_drm_bridge.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698