Switch Android GPU bot to use cloud storage for pixel reference images.

build/android/buildbot/bb_device_steps.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.
 
 import collections
 import glob
 import hashlib
 import json
 import multiprocessing
@@ skipping 434 matching lines @@
 
 def RunWebRTCNativeTests(options):
   RunTestSuites(options, gtest_config.WEBRTC_NATIVE_TEST_SUITES)
 
 
 def RunGPUTests(options):
   InstallApk(options, INSTRUMENTATION_TESTS['ContentShell'], False)
 
   bb_annotations.PrintNamedStep('gpu_tests')
   revision = _GetRevision(options)
-  data_dir = os.path.join(DIR_BUILD_ROOT, 'content_gpu_data', 'telemetry')
   RunCmd(['content/test/gpu/run_gpu_test',
-          '--browser=android-content-shell',
-          '--generated-dir=%s' % os.path.join(data_dir, 'generated'),
-          '--reference-dir=%s' % os.path.join(data_dir, 'reference'),
-          '--build-revision=%s' % revision,
-          'pixel'])
+          'pixel',
+          '--browser',
+          'android-content-shell',
+          '--build-revision',
+          str(revision),
+          '--upload-refimg-to-cloud-storage',
+          '--refimg-cloud-storage-bucket',
+          'chromium-gpu-archive/reference-images',
+          '--os-type',
+          'android',
+          '--test-machine-name',
+          EscapeBuilderName(
+              options.build_properties.get('buildername', 'noname'))])

[Sami, 2014/01/03 13:20:20]

Probably not worth going into in this patch, but it's odd that we can't assume
the builder name is always there.

[navabi, 2014/01/06 21:35:33]

agreed.

[Ken Russell (switch to Gerrit), 2014/01/07 00:48:28]

I suspect that we can assume it, but having a fallback reduces the possibility
of breakage in my opinion.

 
   bb_annotations.PrintNamedStep('webgl_conformance_tests')
   RunCmd(['content/test/gpu/run_gpu_test',
           '--browser=android-content-shell', 'webgl_conformance',
           '--webgl-conformance-version=1.0.1'])
 
 
 def GetTestStepCmds():
   return [
       ('chromedriver', RunChromeDriverTests),
@@ skipping 140 matching lines @@
   setattr(options, 'target', options.factory_properties.get('target', 'Debug'))
   if options.coverage_bucket:
     setattr(options, 'coverage_dir',
             os.path.join(CHROME_OUT_DIR, options.target, 'coverage'))
 
   MainTestWrapper(options)
 
 
 if __name__ == '__main__':
   sys.exit(main(sys.argv))