Reland: Change the video color space default.
commit7cb4525761b68e41671af9006920d568be3c5405
authorwatk <watk@chromium.org>
Tue, 14 Jul 2015 00:46:04 +0000 (13 17:46 -0700)
committerCommit bot <commit-bot@chromium.org>
Tue, 14 Jul 2015 00:46:52 +0000 (14 00:46 +0000)
tree934a3bb2909d6ddb02c8a731e1d890af65487c58
parent26affa0d7dcc5203d9ec17b4c027c8f19208a951
Reland: Change the video color space default.

Reverted by: e735fb0c4b13b72561b8f9931a8788e97deced33
Original CL: https://codereview.chromium.org/1228843003
The new change is to media/cast/sender/h264_vt_encoder_unittest.cc.
The rest of the changes are due to rebasing on top of
11edd34b171b78462c1a27ac6587d977833122ea, which moves the color space
enum.

Previously video without color space metadata was assumed to be in
Rec601. Now the default depends on the kind of playback. Normal src=
defaults to Rec601 for SD sized video (<720 pixels high), and Rec709 for
HD. MSE will always default to Rec709. Using a size based heuristic
doesn't make sense for MSE where it is common for the resolution to
change mid playback.

This CL doesn't change the meaning of COLOR_SPACE_UNSPECIFIED. Instead,
it adds a color space field to VideoDecoderConfig, and updates the video
decoders to use this as the default if they don't find a more
authoritative value in the bitstream.

This also fixes a (year old!) bug causing the blackwhite
tests to always succeed, renames the rec709 blackwhite test
file to match the name in blackwhite.html, and re-encodes it
to contain the color space metadata (previously it had none).

BUG=333619

Review URL: https://codereview.chromium.org/1230593005

Cr-Commit-Position: refs/heads/master@{#338607}
25 files changed:
chromecast/common/media/cma_param_traits.cc
chromecast/common/media/cma_param_traits_macros.h
chromecast/media/cma/ipc_streamer/video_decoder_config_marshaller.cc
chromecast/media/cma/pipeline/audio_video_pipeline_impl_unittest.cc
chromecast/media/cma/test/demuxer_stream_for_test.cc
chromecast/media/cma/test/mock_frame_provider.cc
content/renderer/pepper/video_decoder_shim.cc
media/base/test_helpers.cc
media/base/video_decoder_config.cc
media/base/video_decoder_config.h
media/base/video_types.h
media/cast/sender/h264_vt_encoder_unittest.cc
media/ffmpeg/ffmpeg_common.cc
media/ffmpeg/ffmpeg_common.h
media/filters/decrypting_demuxer_stream.cc
media/filters/ffmpeg_video_decoder.cc
media/filters/ffmpeg_video_decoder_unittest.cc
media/filters/vpx_video_decoder.cc
media/formats/mp2t/es_adapter_video_unittest.cc
media/formats/mp2t/es_parser_h264.cc
media/formats/mp4/mp4_stream_parser.cc
media/formats/webm/webm_video_client.cc
media/mojo/interfaces/media_types.mojom
media/mojo/services/media_type_converters.cc
media/test/data/blackwhite_yuv420p_rec709.mp4 [new file with mode: 0644]