Index: media/filters/h264_parser.cc |
diff --git a/media/filters/h264_parser.cc b/media/filters/h264_parser.cc |
index 052f441d258bc77a38f9e313ceb84bcffd5e584c..9b178081fbb31a49dd837a23679601ccd913db4d 100644 |
--- a/media/filters/h264_parser.cc |
+++ b/media/filters/h264_parser.cc |
@@ -125,17 +125,17 @@ base::Optional<gfx::Rect> H264SPS::GetVisibleRect() const { |
// available from http://www.itu.int/rec/T-REC-H.264. |
gfx::ColorSpace H264SPS::GetColorSpace() const { |
if (colour_description_present_flag) { |
- return gfx::ColorSpace( |
+ return gfx::ColorSpace::CreateVideo( |
colour_primaries, transfer_characteristics, matrix_coefficients, |
video_full_range_flag ? gfx::ColorSpace::RangeID::FULL |
: gfx::ColorSpace::RangeID::LIMITED); |
} else { |
- return gfx::ColorSpace(gfx::ColorSpace::PrimaryID::UNSPECIFIED, |
- gfx::ColorSpace::TransferID::UNSPECIFIED, |
- gfx::ColorSpace::MatrixID::UNSPECIFIED, |
- video_full_range_flag |
- ? gfx::ColorSpace::RangeID::FULL |
- : gfx::ColorSpace::RangeID::LIMITED); |
+ // TODO(ccameron/hubbe): Add a uniform way to handle default video frames. |
+ return gfx::ColorSpace( |
+ gfx::ColorSpace::PrimaryID::BT709, gfx::ColorSpace::TransferID::BT709, |
+ gfx::ColorSpace::MatrixID::BT709, |
+ video_full_range_flag ? gfx::ColorSpace::RangeID::FULL |
+ : gfx::ColorSpace::RangeID::LIMITED); |
} |
} |