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

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

Issue 2572883002: Convert USE_PROPRIETARY_CODECS to a buildflag header. (Closed)
Patch Set: Merge Created 4 years 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: chrome/browser/media/encrypted_media_browsertest.cc
diff --git a/chrome/browser/media/encrypted_media_browsertest.cc b/chrome/browser/media/encrypted_media_browsertest.cc
index d3ece5603c5aad24ebe88bec7d28226f80a08e8c..5a63645e922b7ddb71e656fbef280b4236e6ebbf 100644
--- a/chrome/browser/media/encrypted_media_browsertest.cc
+++ b/chrome/browser/media/encrypted_media_browsertest.cc
@@ -20,6 +20,7 @@
#include "content/public/common/content_switches.h"
#include "content/public/test/browser_test_utils.h"
#include "media/base/media_switches.h"
+#include "media/media_features.h"
#include "ppapi/features/features.h"
#include "testing/gtest/include/gtest/gtest-spi.h"
@@ -64,12 +65,12 @@ const char kWebMVP8VideoOnly[] = "video/webm; codecs=\"vp8\"";
const char kWebMVorbisAudioVP8Video[] = "video/webm; codecs=\"vorbis, vp8\"";
const char kWebMOpusAudioVP9Video[] = "video/webm; codecs=\"opus, vp9\"";
const char kWebMVP9VideoOnly[] = "video/webm; codecs=\"vp9\"";
-#if defined(USE_PROPRIETARY_CODECS)
+#if BUILDFLAG(USE_PROPRIETARY_CODECS)
const char kMP4AudioOnly[] = "audio/mp4; codecs=\"mp4a.40.2\"";
const char kMP4VideoOnly[] = "video/mp4; codecs=\"avc1.4D000C\"";
const char kMP4VideoVp9Only[] =
"video/mp4; codecs=\"vp09.00.01.08.02.01.01.00\"";
-#endif // defined(USE_PROPRIETARY_CODECS)
+#endif // BUILDFLAG(USE_PROPRIETARY_CODECS)
// Sessions to load.
const char kNoSessionToLoad[] = "";
@@ -528,7 +529,7 @@ IN_PROC_BROWSER_TEST_P(EncryptedMediaTest, FrameSizeChangeVideo) {
TestFrameSizeChange();
}
-#if defined(USE_PROPRIETARY_CODECS)
+#if BUILDFLAG(USE_PROPRIETARY_CODECS)
// Crashes on Mac only. http://crbug.com/621857
#if defined(OS_MACOSX)
#define MAYBE_Playback_VideoOnly_MP4 DISABLED_Playback_VideoOnly_MP4
@@ -606,7 +607,7 @@ IN_PROC_BROWSER_TEST_P(EncryptedMediaTest,
TestDifferentContainers(EncryptedContainer::ENCRYPTED_WEBM,
EncryptedContainer::ENCRYPTED_MP4);
}
-#endif // defined(USE_PROPRIETARY_CODECS)
+#endif // BUILDFLAG(USE_PROPRIETARY_CODECS)
#if defined(WIDEVINE_CDM_AVAILABLE)
// The parent key system cannot be used when creating MediaKeys.
« no previous file with comments | « chrome/browser/icon_loader_chromeos.cc ('k') | chrome/browser/media/encrypted_media_supported_types_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698