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

Unified Diff: content/renderer/media/render_media_client_unittest.cc

Issue 2746013006: use VideoColorSpace in decoder configuration (Closed)
Patch Set: oops, win compile fix reapplied 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 | media/base/hdr_metadata.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/media/render_media_client_unittest.cc
diff --git a/content/renderer/media/render_media_client_unittest.cc b/content/renderer/media/render_media_client_unittest.cc
index d88b26f4b7fb873df6c9c2f700dff0695c9b52ad..77a6669b4042489dc085dcaa443b1f3ad8ffed8e 100644
--- a/content/renderer/media/render_media_client_unittest.cc
+++ b/content/renderer/media/render_media_client_unittest.cc
@@ -183,7 +183,7 @@ TEST_F(RenderMediaClientTest, IsKeySystemsUpdateNeeded) {
TEST_F(RenderMediaClientTest, IsSupportedVideoConfigBasics) {
// Default to common 709.
- const media::VideoColorSpace kColorSpace = media::VideoColorSpace::BT709();
+ const media::VideoColorSpace kColorSpace = media::VideoColorSpace::REC709();
// Some codecs do not have a notion of level.
const int kUnspecifiedLevel = 0;
@@ -234,7 +234,7 @@ TEST_F(RenderMediaClientTest, IsSupportedVideoConfig_VP9TransferFunctions) {
for (int i = 0; i <= (1 << (8 * sizeof(media::VideoColorSpace::TransferID)));
i++) {
- media::VideoColorSpace color_space = media::VideoColorSpace::BT709();
+ media::VideoColorSpace color_space = media::VideoColorSpace::REC709();
color_space.transfer = media::VideoColorSpace::GetTransferID(i);
bool found = kSupportedTransfers.find(color_space.transfer) !=
kSupportedTransfers.end();
@@ -260,7 +260,7 @@ TEST_F(RenderMediaClientTest, IsSupportedVideoConfig_VP9Primaries) {
for (int i = 0; i <= (1 << (8 * sizeof(media::VideoColorSpace::PrimaryID)));
i++) {
- media::VideoColorSpace color_space = media::VideoColorSpace::BT709();
+ media::VideoColorSpace color_space = media::VideoColorSpace::REC709();
color_space.primaries = media::VideoColorSpace::GetPrimaryID(i);
bool found = kSupportedPrimaries.find(color_space.primaries) !=
kSupportedPrimaries.end();
@@ -286,7 +286,7 @@ TEST_F(RenderMediaClientTest, IsSupportedVideoConfig_VP9Matrix) {
for (int i = 0; i <= (1 << (8 * sizeof(media::VideoColorSpace::MatrixID)));
i++) {
- media::VideoColorSpace color_space = media::VideoColorSpace::BT709();
+ media::VideoColorSpace color_space = media::VideoColorSpace::REC709();
color_space.matrix = media::VideoColorSpace::GetMatrixID(i);
bool found =
kSupportedMatrix.find(color_space.matrix) != kSupportedMatrix.end();
« no previous file with comments | « no previous file | media/base/hdr_metadata.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698