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

Unified Diff: content/browser/media/encrypted_media_browsertest.cc

Issue 2494983002: media: Supports Clear Key key system for mojo CDM/Renderer combo (Closed)
Patch Set: fix comments Created 4 years, 1 month 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 | « no previous file | media/mojo/clients/mojo_cdm_factory.cc » ('j') | media/mojo/clients/mojo_cdm_factory.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/media/encrypted_media_browsertest.cc
diff --git a/content/browser/media/encrypted_media_browsertest.cc b/content/browser/media/encrypted_media_browsertest.cc
index 565bf29acc93fc3aedcc0bb668c90ea24761ca4a..066b81111c60f1861b6651607450d747d2d33c55 100644
--- a/content/browser/media/encrypted_media_browsertest.cc
+++ b/content/browser/media/encrypted_media_browsertest.cc
@@ -17,12 +17,6 @@
#include "base/android/build_info.h"
#endif
-// MojoCdm supports Clear Key, but currently MojoRenderer cannot use it.
-// See http://crbug.com/441957 for details.
-#if !(defined(ENABLE_MOJO_CDM) && defined(ENABLE_MOJO_RENDERER))
-#define SUPPORTS_CLEAR_KEY_IN_CONTENT_SHELL
-#endif
-
#if defined(ENABLE_MOJO_CDM)
// When mojo CDM is enabled, External Clear Key is supported in //content/shell/
// by using mojo CDM with AesDecryptor running in the remote (e.g. GPU or
@@ -33,9 +27,7 @@
#endif
// Available key systems.
-#if defined(SUPPORTS_CLEAR_KEY_IN_CONTENT_SHELL)
const char kClearKeyKeySystem[] = "org.w3.clearkey";
-#endif
#if defined(SUPPORTS_EXTERNAL_CLEAR_KEY_IN_CONTENT_SHELL)
const char kExternalClearKeyKeySystem[] = "org.chromium.externalclearkey";
@@ -149,13 +141,11 @@ class EncryptedMediaTest : public content::MediaBrowserTest,
using ::testing::Combine;
using ::testing::Values;
-#if defined(SUPPORTS_CLEAR_KEY_IN_CONTENT_SHELL)
INSTANTIATE_TEST_CASE_P(SRC_ClearKey, EncryptedMediaTest,
Combine(Values(kClearKeyKeySystem), Values(SRC)));
INSTANTIATE_TEST_CASE_P(MSE_ClearKey, EncryptedMediaTest,
Combine(Values(kClearKeyKeySystem), Values(MSE)));
-#endif
#if defined(SUPPORTS_EXTERNAL_CLEAR_KEY_IN_CONTENT_SHELL)
INSTANTIATE_TEST_CASE_P(SRC_ExternalClearKey,
« no previous file with comments | « no previous file | media/mojo/clients/mojo_cdm_factory.cc » ('j') | media/mojo/clients/mojo_cdm_factory.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698