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

Unified Diff: chrome/browser/content_settings/content_settings_browsertest.cc

Issue 1996863002: media: Use bundled Widevine CDM in encrypted media browser tests (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: comments addressed Created 4 years, 7 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
« no previous file with comments | « no previous file | chrome/browser/media/encrypted_media_browsertest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/content_settings/content_settings_browsertest.cc
diff --git a/chrome/browser/content_settings/content_settings_browsertest.cc b/chrome/browser/content_settings/content_settings_browsertest.cc
index 5539dcf311b183f7632be9b37eea9669a1264863..5e8ed329e45b5d238ba4acd7ffd078a27352184d 100644
--- a/chrome/browser/content_settings/content_settings_browsertest.cc
+++ b/chrome/browser/content_settings/content_settings_browsertest.cc
@@ -19,6 +19,7 @@
#include "chrome/browser/ui/tabs/tab_strip_model.h"
#include "chrome/common/chrome_switches.h"
#include "chrome/test/base/in_process_browser_test.h"
+#include "chrome/test/base/test_launcher_utils.h"
#include "chrome/test/base/ui_test_utils.h"
#include "components/content_settings/core/browser/cookie_settings.h"
#include "components/content_settings/core/browser/host_content_settings_map.h"
@@ -328,13 +329,6 @@ class PepperContentSettingsSpecialCasesTest : public ContentSettingsTest {
base::FilePath::StringType pepper_plugins = BuildPepperCdmRegistration(
kClearKeyCdmBaseDirectory, kClearKeyCdmAdapterFileName,
kClearKeyCdmDisplayName, kClearKeyCdmPepperMimeType);
-#if defined(WIDEVINE_CDM_AVAILABLE) && defined(WIDEVINE_CDM_IS_COMPONENT)
- // The CDM must be registered when it is a component.
- pepper_plugins.append(FILE_PATH_LITERAL(","));
- pepper_plugins.append(BuildPepperCdmRegistration(
- kWidevineCdmBaseDirectory, kWidevineCdmAdapterFileName,
- kWidevineCdmDisplayName, kWidevineCdmPluginMimeType));
-#endif // defined(WIDEVINE_CDM_AVAILABLE) && defined(WIDEVINE_CDM_IS_COMPONENT)
command_line->AppendSwitchNative(switches::kRegisterPepperPlugins,
pepper_plugins);
#endif // defined(ENABLE_PEPPER_CDMS)
@@ -345,6 +339,15 @@ class PepperContentSettingsSpecialCasesTest : public ContentSettingsTest {
#endif
}
+#if defined(ENABLE_PEPPER_CDMS)
+ void SetUpDefaultCommandLine(base::CommandLine* command_line) override {
+ base::CommandLine default_command_line(base::CommandLine::NO_PROGRAM);
+ InProcessBrowserTest::SetUpDefaultCommandLine(&default_command_line);
+ test_launcher_utils::RemoveCommandLineSwitch(
+ default_command_line, switches::kDisableComponentUpdate, command_line);
+ }
+#endif // defined(ENABLE_PEPPER_CDMS)
+
void RunLoadPepperPluginTest(const char* mime_type, bool expect_loaded) {
const char* expected_result = expect_loaded ? "Loaded" : "Not Loaded";
content::WebContents* web_contents =
« no previous file with comments | « no previous file | chrome/browser/media/encrypted_media_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698