Gate support for certain EOTFs/primaries/matrices on color management / hdr flags

Gate support for certain EOTFs/primaries/matrices on color management / hdr flags

Creates a new class VideoColorSpace specifically for encoding color spaces
according to ISO/IEC 23001-8.

Plumb this color space through from parsing to validation.

BUG=687627
Review-Url: https://codereview.chromium.org/2742113002
Cr-Commit-Position: refs/heads/master@{#456518}
Committed: https://chromium.googlesource.com/chromium/src/+/2fb4f2c9c461551d43fdfa28ef4c960da81a47dd

[hubbe, 2017-03-10 23:46:00]

The CQ bit was checked by hubbe@chromium.org to run a CQ dry run

[hubbe, 2017-03-10 23:46:01]

hubbe@chromium.org changed reviewers:
+ ccameron@chromium.org, chcunningham@chromium.org

[commit-bot: I haz the power, 2017-03-10 23:46:38]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-10 23:54:26]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-10 23:54:28]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[chcunningham, 2017-03-11 02:50:33]

LGTM! Awesome that color_space.h is doing this for us. I didn't realize the
tables were the same from the h264 spec. Great cleanup.

[ccameron, 2017-03-13 12:49:49]

lgtm2

[ccameron, 2017-03-13 12:50:01]

On 2017/03/13 12:49:49, ccameron wrote:
> lgtm2

err, lgtm

[hubbe, 2017-03-13 19:40:32]

The CQ bit was checked by hubbe@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-13 19:41:21]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[hubbe, 2017-03-13 20:57:38]

The CQ bit was unchecked by hubbe@chromium.org

[hubbe, 2017-03-13 20:57:39]

The CQ bit was checked by hubbe@chromium.org

[hubbe, 2017-03-13 20:57:39]

The patchset sent to the CQ was uploaded after l-g-t-m from
chcunningham@chromium.org, ccameron@chromium.org
Link to the patchset: https://codereview.chromium.org/2742113002/#ps20001
(title: "missed updating some tests")

[commit-bot: I haz the power, 2017-03-13 20:58:00]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-13 22:52:58]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1489438659110100,
"parent_rev": "1a2563dcc211e71fa8c32f52c9bc4d64d71a42cc", "commit_rev":
"2fb4f2c9c461551d43fdfa28ef4c960da81a47dd"}

[commit-bot: I haz the power, 2017-03-13 22:53:48]

Description was changed from

==========
Gate support for certain EOTFs/primaries/matrices on color management / hdr
flags

Creates a new class VideoColorSpace specifically for encoding color spaces
according to ISO/IEC 23001-8.

Plumb this color space through from parsing to validation.

BUG=687627
==========

to

==========
Gate support for certain EOTFs/primaries/matrices on color management / hdr
flags

Creates a new class VideoColorSpace specifically for encoding color spaces
according to ISO/IEC 23001-8.

Plumb this color space through from parsing to validation.

BUG=687627

Review-Url: https://codereview.chromium.org/2742113002
Cr-Commit-Position: refs/heads/master@{#456518}
Committed:
https://chromium.googlesource.com/chromium/src/+/2fb4f2c9c461551d43fdfa28ef4c...
==========

[commit-bot: I haz the power, 2017-03-13 22:53:53]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/2fb4f2c9c461551d43fdfa28ef4c...