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

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

Issue 1920433003: EME: Address TODOs related to the removal of prefixed EME (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 4 years, 6 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 add05056accd59fbb60462ee059763266c32c228..25ffb1959d52985456972ff9fa1131370cac9bb5 100644
--- a/media/base/android/java/src/org/chromium/media/MediaDrmBridge.java
+++ b/media/base/android/java/src/org/chromium/media/MediaDrmBridge.java
@@ -752,9 +752,8 @@ public class MediaDrmBridge {
return;
}
- // TODO(xhwang): DCHECK this now that prefixed EME is deprecated.
ddorwin 2016/06/23 01:06:47 Sessiond IDs are handled internally now, so we sho
- // https://crbug.com/249976
if (!sessionExists(sessionId)) {
+ assert false; // Should never happen.
onPromiseRejected(
promiseId, "Invalid session in updateSession: " + bytesToHexString(sessionId));
return;

Powered by Google App Engine
This is Rietveld 408576698