Run skimage on the bots.

slave/skia_slave_scripts/download_skimage_files.py

+#!/usr/bin/env python
+# Copyright (c) 2013 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+""" Download the image files needed to run skimage tool. """

[epoger, 2013/07/08 16:03:05]

Wouldn't it be simpler to keep the source image files in source control (e.g.
https://code.google.com/p/skia/source/browse/#svn%2Ftrunk%2Fresources ), rather
than downloading them from Google Storage?

Or are there a very large number of these images, or maybe they change often? 
(Those are reasons I could imagine not wanting to have them in the repo)

One thing that's nice about keeping files in our repo is that it makes it easier
for individual developers to run the test locally...

[scroggo, 2013/07/08 22:36:44]

Yes, it would be easier to keep them in resources.
There are currently a "small" number of images. My understanding was that we
wanted NO images under git source control, but perhaps I took the warning too
far. My plan is to check in more images when we find and fix bugs in our image
decoders, and continue to have them for regression testing. The images
themselves will not change (i.e. '<image1>.png' will always be the same encoded
image). I don't know where the line into "very large" is, or when we will cross
it.

[epoger, 2013/07/12 15:09:01]

I don't know where the line is either. :-)  You get to decide.

Since you've already got this working pulling images from Google Storage, that's
fine.  If it ends up being trouble for a developer who wants to run locally (or
for some other reason), and thus you decide to move the images into our repo,
that's fine too.

+
+from build_step import BuildStep
+from utils import gs_utils
+from utils import sync_bucket_subdir
+import os
+import sys
+
+class DownloadSKImageFiles(BuildStep):
+  def __init__(self, timeout=12800, no_output_timeout=9600, **kwargs):
+    super (DownloadSKImageFiles, self).__init__(
+        timeout=timeout,
+        no_output_timeout=no_output_timeout,
+        **kwargs)
+
+  def _DownloadSKImagesFromStorage(self):
+    """Copies over image files from Google Storage if the timestamps differ."""
+    dest_gsbase = (self._args.get('dest_gsbase') or
+                   sync_bucket_subdir.DEFAULT_PERFDATA_GS_BASE)
+    print '\n\n========Downloading image files from Google Storage========\n\n'
+    gs_relative_dir = os.path.join('skimage', 'input')
+    gs_utils.DownloadDirectoryContentsIfChanged(
+        gs_base=dest_gsbase,
+        gs_relative_dir=gs_relative_dir,
+        local_dir=self._skimage_in_dir)
+
+  def _Run(self):
+    # Locally copy image files from GoogleStorage.
+    self._DownloadSKImagesFromStorage()
+
+
+if '__main__' == __name__:
+  sys.exit(BuildStep.RunBuildStep(DownloadSKImageFiles))