Recipe changes to move webview perf bots to the android/perf recipe

scripts/slave/recipes/android/perf.py

 # Copyright 2014 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.
 
 from recipe_engine.types import freeze
 
 
 DEPS = [
     'adb',
-    'depot_tools/bot_update',
     'chromium',
     'chromium_android',
     'chromium_tests',
+    'depot_tools/bot_update',
     'depot_tools/gclient',
     'recipe_engine/json',
     'recipe_engine/path',
     'recipe_engine/properties',
     'recipe_engine/python',
     'recipe_engine/step',
     'test_utils',
 ]
 
 
 REPO_URL = 'https://chromium.googlesource.com/chromium/src.git'
 
-
-def _ChromiumPerfTesters():
-  def _AddTestSpec(name, perf_id, num_device_shards=1, num_host_shards=1,
-                   target_bits=64, known_devices_file='.known_devices'):
-    for shard_index in xrange(num_host_shards):
-      builder_name = '%s (%d)' % (name, shard_index + 1)
-      testers[builder_name] = _TestSpec(
-          name, perf_id, num_device_shards, num_host_shards, shard_index,
-          target_bits, known_devices_file)
-
-  def _TestSpec(name, perf_id, num_device_shards, num_host_shards, shard_index,
-                target_bits, known_devices_file):
+def _CreateTestSpec(name, perf_id, required_apks, num_device_shards=1,
+                   num_host_shards=1, target_bits=64,
+                   known_devices_file='.known_devices'):
+  def _CreateShardTestSpec(name, perf_id, required_apks, num_device_shards,
+                num_host_shards, shard_index, target_bits, known_devices_file):
     spec = {
       'perf_id': perf_id,
+      'required_apks': required_apks,
       'bucket': 'chrome-perf',
       'num_device_shards': num_device_shards,
       'num_host_shards': num_host_shards,
       'shard_index': shard_index,
       'test_spec_file': 'chromium.perf.json',
       'max_battery_temp': 350,
       'known_devices_file': known_devices_file,
     }
-
     if target_bits == 32:
       builder_name = 'android_perf_rel'
     elif target_bits == 64:
       builder_name = 'android_perf_rel_arm64'
       spec['recipe_config'] = 'tests_arm64'
     spec['path'] = lambda api: '%s/full-build-linux_%s.zip' % (
         builder_name, api.properties['parent_revision'])
-
     return spec
 
-  testers = {}
+  tester_spec = {}
+  for shard_index in xrange(num_host_shards):
+    builder_name = '%s (%d)' % (name, shard_index + 1)
+    tester_spec[builder_name] = _CreateShardTestSpec(
+        name, perf_id, required_apks, num_device_shards, num_host_shards,
+        shard_index, target_bits, known_devices_file)
+  return tester_spec
 
-  _AddTestSpec('Android Galaxy S5 Perf', 'android-galaxy-s5',
-      num_device_shards=7, num_host_shards=3, target_bits=32)
-  _AddTestSpec('Android Nexus5 Perf', 'android-nexus5',
-      num_device_shards=7, num_host_shards=3, target_bits=32)
-  _AddTestSpec('Android Nexus5X Perf', 'android-nexus5X',
-      num_device_shards=7, num_host_shards=3)
-  _AddTestSpec('Android Nexus6 Perf', 'android-nexus6',
-      num_device_shards=7, num_host_shards=3, target_bits=32)
-  _AddTestSpec('Android Nexus7v2 Perf', 'android-nexus7v2',
-      num_device_shards=7, num_host_shards=3, target_bits=32)
-  _AddTestSpec('Android Nexus9 Perf', 'android-nexus9',
-      num_device_shards=7, num_host_shards=3)
-  _AddTestSpec('Android One Perf', 'android-one',
-      num_device_shards=7, num_host_shards=3, target_bits=32)
+def _ChromiumPerfTesters():
+  testers = [
+    _CreateTestSpec('Android Galaxy S5 Perf', 'android-galaxy-s5',
+        required_apks=['ChromePublic.apk'], num_device_shards=7,
+        num_host_shards=3, target_bits=32),
+    _CreateTestSpec('Android Nexus5 Perf', 'android-nexus5',
+        required_apks=['ChromePublic.apk'], num_device_shards=7,
+        num_host_shards=3, target_bits=32),
+    _CreateTestSpec('Android Nexus5X Perf', 'android-nexus5X',
+        required_apks=['ChromePublic.apk'], num_device_shards=7,
+        num_host_shards=3),
+    _CreateTestSpec('Android Nexus6 Perf', 'android-nexus6',
+        required_apks=['ChromePublic.apk'], num_device_shards=7,
+        num_host_shards=3, target_bits=32),
+    _CreateTestSpec('Android Nexus7v2 Perf', 'android-nexus7v2',
+        required_apks=['ChromePublic.apk'], num_device_shards=7,
+        num_host_shards=3, target_bits=32),
+    _CreateTestSpec('Android Nexus9 Perf', 'android-nexus9',
+        required_apks=['ChromePublic.apk'], num_device_shards=7,
+        num_host_shards=3),
+    _CreateTestSpec('Android One Perf', 'android-one',
+        required_apks=['ChromePublic.apk'], num_device_shards=7,
+        num_host_shards=3, target_bits=32),
+  ]
+  master_spec = {}
+  for spec in testers:
+    master_spec.update(spec)
+  return master_spec
 
-  return testers
+def _ChromiumPerfFYITesters():
+  testers = [
+    _CreateTestSpec('Android Nexus5 WebView Perf', 'android-webview',
+        required_apks=['SystemWebview.apk', 'SystemWebViewShell.apk'],
+        num_device_shards=5, num_host_shards=1, target_bits=32),
+    _CreateTestSpec('Android Nexus5x WebView Perf', 'android-webview-nexus5X',
+        required_apks=['SystemWebview.apk', 'SystemWebViewShell.apk'],
+        num_device_shards=7, num_host_shards=2, target_bits=32),
+  ]
+  master_spec = {}
+  for spec in testers:
+    master_spec.update(spec)
+  return master_spec
 
 
 BUILDERS = freeze({
   'chromium.perf': _ChromiumPerfTesters(),
+  'chromium.perf.fyi': _ChromiumPerfFYITesters(),
 })
 
 
 def RunSteps(api):
   mastername = api.properties['mastername']
   buildername = api.properties['buildername']
   # TODO(akuegel): Move the configs in builders.py in chromium_tests to this
   # recipe, and get rid of duplications.
   builder = dict(BUILDERS[mastername][buildername])
   builder_config = builder.get('recipe_config', 'base_config')
@@ skipping 26 matching lines @@
         bot_update_step)
 
   api.path['checkout'] = api.path['slave_build'].join('src')
   api.chromium_android.clean_local_files()
 
   api.chromium_android.download_build(bucket=builder['bucket'],
     path=builder['path'](api))
 
   api.chromium_android.common_tests_setup_steps(perf_setup=True)
 
-  api.chromium_android.adb_install_apk('ChromePublic.apk')
+  required_apks = builder.get('required_apks', [])
+  for apk in required_apks:
+    api.chromium_android.adb_install_apk(apk)
 
   test_runner = api.chromium_tests.create_test_runner(
       api, builder.get('tests', []))
 
   try:
     failures = []
     if test_runner:
       try:
         test_runner()
       except api.step.StepFailure as f:
@@ skipping 107 matching lines @@
           mastername='chromium.perf',
           buildername='Android Nexus5 Perf (1)',
           parent_buildername='parent_buildername',
           parent_buildnumber='1729',
           parent_revision='deadbeef',
           revision='deadbeef',
           slavename='slavename',
           target='Release') +
       api.override_step_data(
           'perf_test.foo', retcode=87))