Skia: Add --colorImages to DM

scripts/slave/recipe_modules/skia/api.py

Index: scripts/slave/recipe_modules/skia/api.py
diff --git a/scripts/slave/recipe_modules/skia/api.py b/scripts/slave/recipe_modules/skia/api.py
index 5df1998d91125fe7e8385649b59086b13df77b14..3192476be3aa3db8bfaae37b41721996e5597b25 100644
--- a/scripts/slave/recipe_modules/skia/api.py
+++ b/scripts/slave/recipe_modules/skia/api.py
@@ -712,6 +712,8 @@ print json.dumps({'ccache': ccache})
       '--skps', self.device_dirs.skp_dir,
       '--images', self.flavor.device_path_join(
           self.device_dirs.images_dir, 'dm'),
+      '--colorImages', self.flavor.device_path_join(
+          self.device_dirs.images_dir, 'colorspace'),

[borenet, 2016/05/04 12:09:55]

I have a couple of concerns:
1. This will break for backfills before this flag was added to DM.
2. Does DM properly handle this directory being missing? AFAICT it hasn't yet
been added to GS.

[scroggo, 2016/05/04 12:50:32]

Isn't this a more general problem any time we add a new flag to DM? Has this not
happened before? (I suppose we have not added any since we started backfilling?)
I believe CollectImages only causes a problem if a named directory/file is
missing/empty, but if no directories/files were listed, so not being there yet
should not be a problem.

[borenet, 2016/05/04 12:56:16]

Yeah, it's a general problem.  I just feel I need to point it out when it
happens and think briefly about whether there's any way to avoid it.  I think
"version >= 5" conditional would prevent any issues.

       '--nameByHash',
       '--properties'
     ] + properties