Conditionally build support for Pepper-based CDMs.

content/browser/media/encrypted_media_browsertest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/basictypes.h"
 #include "base/command_line.h"
 #include "base/file_util.h"
 #include "base/path_service.h"
 #include "base/string16.h"
 #include "base/string_util.h"
@@ skipping 11 matching lines @@
 #include "webkit/media/crypto/key_systems.h"
 
 #include "widevine_cdm_version.h"  // In SHARED_INTERMEDIATE_DIR.
 
 #if defined(WIDEVINE_CDM_AVAILABLE) && defined(OS_LINUX)
 #include <gnu/libc-version.h>
 #endif  // defined(WIDEVINE_CDM_AVAILABLE) && defined(OS_LINUX)
 
 // Platform-specific filename relative to the chrome executable.
 #if defined(OS_WIN)
-static const char kClearKeyLibraryName[] = "clearkeycdmadapter.dll";
-static const char kWidevineLibraryName[] = "widevinecdmadapter.dll";
+static const wchar_t  kClearKeyLibraryName[] = L"clearkeycdmadapter.dll";
+static const wchar_t  kWidevineLibraryName[] = L"widevinecdmadapter.dll";
 #elif defined(OS_MACOSX)
 static const char kClearKeyLibraryName[] = "clearkeycdmadapter.plugin";
 static const char kWidevineLibraryName[] = "widevinecdmadapter.plugin";
 #elif defined(OS_POSIX)
 static const char kClearKeyLibraryName[] = "libclearkeycdmadapter.so";
 static const char kWidevineLibraryName[] = "libwidevinecdmadapter.so";

[xhwang, 2013/05/10 00:26:44]

For WV, can we use kWidevineCdmPluginFileName in widevine_cdm_common.h?

[ddorwin, 2013/05/14 00:17:10]

Done.

 #endif
 
 // Available key systems.
 static const char kClearKeyKeySystem[] = "webkit-org.w3.clearkey";
 static const char kExternalClearKeyKeySystem[] =
     "org.chromium.externalclearkey";
 
 // Supported media types.
 static const char kWebMAudioOnly[] = "audio/webm; codecs=\"vorbis\"";
 static const char kWebMVideoOnly[] = "video/webm; codecs=\"vp8\"";
@@ skipping 48 matching lines @@
       std::string fail_message;
       EXPECT_TRUE(ExecuteScriptAndExtractString(
           shell()->web_contents(),
           "window.domAutomationController.send(failMessage);",
           &fail_message));
       LOG(INFO) << "Test failed: " << fail_message;
     }
   }
 
  protected:
-  // Registers any CDM plugins not registered by default.
   virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
-    RegisterPepperPlugin(command_line, kClearKeyLibraryName,
-                         kExternalClearKeyKeySystem);
+    RegisterPepperCdm(command_line, kClearKeyLibraryName,

[xhwang, 2013/05/10 00:26:44]

what's the reason these pepper related code isn't in the ENABLE_PEPPER_CDMS
scope.

[ddorwin, 2013/05/14 00:17:10]

We can handle compatibility with a single ifdef at 116 instead of adding 2 or
more ifdef blocks. It seems cleaner this way, though I see how this might
confuse someone.

UPDATE: After I ended up surrounding these as well since the |library_name| is
specific to the Pepper case.

+                      kExternalClearKeyKeySystem);
   }
 
-  virtual void RegisterPepperPlugin(CommandLine* command_line,
-                                    const std::string& library_name,
-                                    const std::string& key_system) {
+  virtual void RegisterPepperCdm(CommandLine* command_line,
+                                 const base::FilePath::StringType& library_name,
+                                 const std::string& key_system) {
+#if defined(ENABLE_PEPPER_CDMS)
     // Append the switch to register the Clear Key CDM Adapter.
     base::FilePath plugin_dir;
     EXPECT_TRUE(PathService::Get(base::DIR_MODULE, &plugin_dir));
-#if defined(OS_WIN)
-    base::FilePath plugin_lib = plugin_dir.Append(ASCIIToWide(library_name));
-#else
     base::FilePath plugin_lib = plugin_dir.Append(library_name);
-#endif
     EXPECT_TRUE(file_util::PathExists(plugin_lib));
     base::FilePath::StringType pepper_plugin = plugin_lib.value();
     pepper_plugin.append(FILE_PATH_LITERAL("#CDM#0.1.0.0;"));
 #if defined(OS_WIN)
-    pepper_plugin.append(ASCIIToWide(webkit_media::GetPluginType(key_system)));
+    pepper_plugin.append(ASCIIToWide(webkit_media::GetPepperType(key_system)));
 #else
-    pepper_plugin.append(webkit_media::GetPluginType(key_system));
+    pepper_plugin.append(webkit_media::GetPepperType(key_system));
 #endif
     command_line->AppendSwitchNative(switches::kRegisterPepperPlugins,
                                      pepper_plugin);
+#endif  // defined(ENABLE_PEPPER_CDMS)
   }
 };
 
 #if defined(WIDEVINE_CDM_AVAILABLE)
 class WVEncryptedMediaTest : public EncryptedMediaTest {
  public:
   // Tests that the following happen after trying to play encrypted media:
   // - webkitneedkey event is fired.
   // - webkitGenerateKeyRequest() does not fail.
   // - webkitkeymessage is fired
@@ skipping 13 matching lines @@
                                            key_system, kExpectedWebKitError);
     bool receivedKeyMessage = false;
     EXPECT_TRUE(ExecuteScriptAndExtractBool(
         shell()->web_contents(),
         "window.domAutomationController.send(video.receivedKeyMessage);",
         &receivedKeyMessage));
     ASSERT_TRUE(receivedKeyMessage);
   }
 
  protected:
-  // Registers any CDM plugins not registered by default.
   virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
     command_line->AppendSwitch(switches::kAllowFileAccessFromFiles);
-    RegisterPepperPlugin(command_line, kWidevineLibraryName,
-                         kWidevineKeySystem);
+    RegisterPepperCdm(command_line, kWidevineLibraryName, kWidevineKeySystem);
   }
 };
 #endif  // defined(WIDEVINE_CDM_AVAILABLE)
 
 INSTANTIATE_TEST_CASE_P(ClearKey, EncryptedMediaTest,
                         ::testing::Values(kClearKeyKeySystem));
 
+// External Clear Key is currently only used on platforms that use Pepper CDMs.
+#if defined(ENABLE_PEPPER_CDMS)
 INSTANTIATE_TEST_CASE_P(ExternalClearKey, EncryptedMediaTest,
                         ::testing::Values(kExternalClearKeyKeySystem));
+#endif
 
 IN_PROC_BROWSER_TEST_F(EncryptedMediaTest, InvalidKeySystem) {
   const string16 kExpected = ASCIIToUTF16(
       StringToUpperASCII(std::string("GenerateKeyRequestException")));
   TestSimplePlayback("bear-320x240-av-enc_av.webm", kWebMAudioVideo,
                      "com.example.invalid", kExpected);
 }
 
 IN_PROC_BROWSER_TEST_P(EncryptedMediaTest, Playback_AudioOnly_WebM) {
   TestSimplePlayback("bear-a-enc_a.webm", kWebMAudioOnly, GetParam(),
@@ skipping 77 matching lines @@
 
 IN_PROC_BROWSER_TEST_F(WVEncryptedMediaTest, Playback_AudioOnly_MP4) {
   TestSimplePlayback("bear-640x360-a_frag-cenc.mp4", kMP4AudioOnly,
                      kWidevineKeySystem);
 }
 #endif  // defined(GOOGLE_CHROME_BUILD) || defined(USE_PROPRIETARY_CODECS)
 #endif  // !defined(DISABLE_WIDEVINE_CDM_BROWSERTESTS)
 #endif  // defined(WIDEVINE_CDM_AVAILABLE)
 
 }  // namespace content