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

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

Issue 2760893003: Update new VP9 codec string parsing (Closed)
Patch Set: So many tests... Created 3 years, 9 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_supported_types_browsertest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 5ae78f1ede711b65081b65e98f4443d37295f2a6..553ddca2cfc867f7d95f6a77ac3e2fb23909d87c 100644
--- a/chrome/browser/media/encrypted_media_browsertest.cc
+++ b/chrome/browser/media/encrypted_media_browsertest.cc
@@ -80,8 +80,12 @@ const char kWebMVP9VideoOnly[] = "video/webm; codecs=\"vp9\"";
#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\"";
+
+// NOTE: This string originally signalled the SMPTEST2084 EOTF, but is now
+// signalling BT709. At this time 2084 is only allowed when run with
+// --enable-hdr, and thats not really the focus of this test.
+// TODO(kqyang): update content to not use HDR transfer.
+const char kMP4VideoVp9Only[] = "video/mp4; codecs=\"vp09.00.10.08.01.01.01\"";
#endif // BUILDFLAG(USE_PROPRIETARY_CODECS)
// Sessions to load.
« no previous file with comments | « no previous file | chrome/browser/media/encrypted_media_supported_types_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698