Create a separate src for colorspace tests

Create a separate src for colorspace tests

This will allow me to run these tests in sRGB mode, while
leaving most of the image tests disabled.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2206953006

Committed: https://skia.googlesource.com/skia/+/b8d1aac87a6e3821c7ed435a17a550e6e87d2044

[msarett, 2016-08-04 14:55:23]

Description was changed from

==========
Create a separate src for colorspace tests

This will allow me to run these tests in sRGB mode, while
leaving most of the image tests disabled.

BUG=skia:
==========

to

==========
Create a separate src for colorspace tests

This will allow me to run these tests in sRGB mode, while
leaving most of the image tests disabled.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2206953006
==========

[msarett, 2016-08-04 14:56:04]

msarett@google.com changed reviewers:
+ borenet@google.com

[msarett, 2016-08-04 15:00:36]

The CQ bit was checked by msarett@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-08-04 15:00:44]

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

[borenet, 2016-08-04 15:06:22]

How will this affect run times on the bots?

I'm in the process of writing a series of changes to dramatically simplify how
this type of change will work.  The first is here:
https://codereview.chromium.org/2215803002/  If you land this CL before mine,
you also need to run:

$ python tools/buildbot_spec.py test
$ python infra/bots/gen_buildbot_specs.py
$ python infra/bots/recipes.py simulation_test train

[msarett, 2016-08-04 15:11:16]

Will have no noticeable effect on runtime.  This adds an extra config for 5
(colorImage) tests that run on a small set of images.

The (image) config has hundreds of tests that run on a much larger set of
images.

I'm fine to wait for your change if it will land soon (like today?).  I've
unintentionally disabled all of my tests (by switching them to sRGB) and I want
to get them back up and running.

[borenet, 2016-08-04 15:14:32]

On 2016/08/04 15:11:16, msarett wrote:
> Will have no noticeable effect on runtime.  This adds an extra config for 5
> (colorImage) tests that run on a small set of images.
> 
> The (image) config has hundreds of tests that run on a much larger set of
> images.
> 
> I'm fine to wait for your change if it will land soon (like today?).  I've
> unintentionally disabled all of my tests (by switching them to sRGB) and I
want
> to get them back up and running.

Ok, please go ahead and generate the remaining expectations files. I'll try to
get my CL landed ASAP, but if it doesn't we can land this one and I'll rebase.

[msarett, 2016-08-04 15:14:33]

msarett@google.com changed reviewers:
+ jcgregorio@google.com

[msarett, 2016-08-04 15:14:34]

Adding Joe.  If I make this change, will the new src (colorImages) be uploaded
to Gold?  Or do I need to do something else?

[msarett, 2016-08-04 15:20:10]

Adding buildbot_spec.json

[borenet, 2016-08-04 18:12:33]

Okay, my CL has landed.  Please rebase and apply your dm_flags change here:
https://skia.googlesource.com/skia/+/master/infra/bots/recipe_modules/vars/ap...

[msarett, 2016-08-04 18:25:59]

Alright I think this looks ready.

[msarett, 2016-08-04 18:29:18]

The CQ bit was checked by msarett@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-08-04 18:29:26]

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, 2016-08-04 18:39:18]

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

[commit-bot: I haz the power, 2016-08-04 18:39:19]

Dry run: Try jobs failed on following builders:
  Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Debug-GN-Trybot on master.client.skia
(JOB_FAILED,
http://build.chromium.org/p/client.skia/builders/Test-Ubuntu-GCC-GCE-CPU-AVX2...)

[borenet, 2016-08-04 18:48:50]

LGTM but looks like you have an assertion failure.

[msarett, 2016-08-04 18:50:26]

On 2016/08/04 18:48:50, borenet wrote:
> LGTM but looks like you have an assertion failure.

Yup, will fix.  One of my new images...

[msarett, 2016-08-04 18:58:24]

The CQ bit was checked by msarett@google.com

[msarett, 2016-08-04 18:58:24]

The patchset sent to the CQ was uploaded after l-g-t-m from borenet@google.com
Link to the patchset: https://codereview.chromium.org/2206953006/#ps80001
(title: "FIXME")

[commit-bot: I haz the power, 2016-08-04 18:58:32]

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, 2016-08-04 19:22:09]

Description was changed from

==========
Create a separate src for colorspace tests

This will allow me to run these tests in sRGB mode, while
leaving most of the image tests disabled.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2206953006
==========

to

==========
Create a separate src for colorspace tests

This will allow me to run these tests in sRGB mode, while
leaving most of the image tests disabled.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2206953006

Committed:
https://skia.googlesource.com/skia/+/b8d1aac87a6e3821c7ed435a17a550e6e87d2044
==========

[commit-bot: I haz the power, 2016-08-04 19:22:10]

Committed patchset #5 (id:80001) as
https://skia.googlesource.com/skia/+/b8d1aac87a6e3821c7ed435a17a550e6e87d2044