Add Matrix4x3 and tristimulus class to color utiliies.

Add Matrix4x3 and tristimulus class to color utiliies.

The TriStim class represents a color in any color space, it will be used as input/output to the
ColorTransform class. The Matrix4x3 class will be used both by the ColorTransform class and to
store the primary color transofmration in the ColorSpace class if primary_ == PrimaryID::CUSTOM.

BUG=622133

[hubbe, 2016-07-27 21:38:28]

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

[commit-bot: I haz the power, 2016-07-27 21:38:50]

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

[hubbe, 2016-07-27 21:38:56]

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

[ccameron, 2016-07-27 21:57:05]

I don't think that we should add these structures -- we already have a robust
4x4 matrix library in gfx::Transform aka SkMatrix44 -- we should use that, and
explicitly set the bottom row to [0 0 0 1].

IMO gfx::ColorSpace should just have a
  float custom_matrix[12];

And then gfx::ColorTransform can internally convert that into and manipulate via
SkMatrix44 (or gfx::Transform).

[hubbe, 2016-07-27 22:03:32]

On 2016/07/27 21:57:05, ccameron wrote:
> I don't think that we should add these structures -- we already have a robust
> 4x4 matrix library in gfx::Transform aka SkMatrix44 -- we should use that, and
> explicitly set the bottom row to [0 0 0 1].
> 
> IMO gfx::ColorSpace should just have a
>   float custom_matrix[12];
> 
> And then gfx::ColorTransform can internally convert that into and manipulate
via
> SkMatrix44 (or gfx::Transform).

Ah, missed that one, will delete this CL soon then.

[commit-bot: I haz the power, 2016-07-27 22:10:20]

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

[commit-bot: I haz the power, 2016-07-27 22:10:21]

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

[hubbe, 2016-08-02 22:16:20]

Description was changed from

==========
Add Matrix4x3 and tristimulus class to color utiliies.

The TriStim class represents a color in any color space, it will be used as
input/output to the
ColorTransform class. The Matrix4x3 class will be used both by the
ColorTransform class and to
store the primary color transofmration in the ColorSpace class if primary_ ==
PrimaryID::CUSTOM.

BUG=622133
==========

to

==========
Add Matrix4x3 and tristimulus class to color utiliies.

The TriStim class represents a color in any color space, it will be used as
input/output to the
ColorTransform class. The Matrix4x3 class will be used both by the
ColorTransform class and to
store the primary color transofmration in the ColorSpace class if primary_ ==
PrimaryID::CUSTOM.

BUG=622133
==========