webrtc: drop dependency on chromium_tests; extract chromium_swarming

scripts/slave/recipe_modules/chromium_tests/api.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.
 
-import collections
 import contextlib
 import copy
 import itertools
 import json
 
 from recipe_engine.types import freeze
 from recipe_engine import recipe_api
 from recipe_engine import util as recipe_util
 
 from . import bot_config_and_test_db as bdb_module
 from . import builders
 from . import steps
 from . import trybots
 
 
 # Paths which affect recipe config and behavior in a way that survives
 # deapplying user's patch.
 RECIPE_CONFIG_PATHS = [
     'testing/buildbot',
 ]
 
 
-PER_TARGET_SWARMING_DIMS = collections.defaultdict(dict)
-PER_TARGET_SWARMING_DIMS.update({
-    'android': {
-      'android_devices': '6',
-      'cpu': None,
-      'gpu': None,
-      'os': 'Android',
-    }
-})
-
-
-MASTER_SWARMING_PRIORITIES = collections.defaultdict(lambda: 25)
-MASTER_SWARMING_PRIORITIES.update({
-    'chromium.fyi': 35,  # This should be lower than the CQ.
-    'chromium.memory.fyi': 27,
-})
-
-
 class ChromiumTestsApi(recipe_api.RecipeApi):
   def __init__(self, *args, **kwargs):
     super(ChromiumTestsApi, self).__init__(*args, **kwargs)
     self._builders = {}
     self.add_builders(builders.BUILDERS)
     self._precommit_mode = False
 
     # Keep track of working directory (which contains the checkout).
     # None means "default value".
     self._working_dir = None
@@ skipping 662 matching lines @@
         'self.m.filter.test_targets': self.m.filter.test_targets,
         'compile_targets': compile_targets,
     }
     with contextlib.closing(recipe_util.StringListIO()) as listio:
       json.dump(analyze_details, listio, indent=2, sort_keys=True)
     step_result = self.m.step.active_result
     step_result.presentation.logs['analyze_details'] = listio.lines
 
     return self.m.filter.test_targets, compile_targets
 
+  # TODO(phajdan.jr): fix callers and remove chromium_tests.configure_swarming.
   def configure_swarming(self, project_name, precommit, mastername=None):
-    """Configures default swarming dimensions and tags.
-
-    Uses the 'chromium' global config to determine target platform defaults,
-    make sure something like chromium_tests.configure_build() has been called
-    beforehand.
-
-    Args:
-      project_name: Lowercase name of the project, e.g. "blink", "chromium".
-      precommit: Boolean flag to indicate whether the tests are running before
-          the changes are commited.
-    """
-
-    # Set platform-specific default dims.
-    target_platform = self.m.chromium.c.TARGET_PLATFORM
-    swarming_dims = PER_TARGET_SWARMING_DIMS[target_platform]
-    for k, v in swarming_dims.iteritems():
-      self.m.swarming.set_default_dimension(k, v)
-
-    self.m.swarming.set_default_dimension('pool', 'Chrome')
-    self.m.swarming.add_default_tag('project:%s' % project_name)
-    self.m.swarming.default_idempotent = True
-
-    if precommit:
-      self.m.swarming.default_priority = 30
-      self.m.swarming.add_default_tag('purpose:pre-commit')
-      requester = self.m.properties.get('requester')
-      if requester == 'commit-bot@chromium.org':
-        self.m.swarming.add_default_tag('purpose:CQ')
-        blamelist = self.m.properties.get('blamelist')
-        if len(blamelist) == 1:
-          requester = blamelist[0]
-      else:
-        self.m.swarming.add_default_tag('purpose:ManualTS')
-      self.m.swarming.default_user = requester
-
-      patch_project = self.m.properties.get('patch_project')
-      if patch_project:
-        self.m.swarming.add_default_tag('patch_project:%s' % patch_project)
-    else:
-      self.m.swarming.default_priority = MASTER_SWARMING_PRIORITIES[mastername]
-      self.m.swarming.add_default_tag('purpose:post-commit')
-      self.m.swarming.add_default_tag('purpose:CI')
+    return self.m.chromium_swarming.configure_swarming(  # pragma: no cover
+        project_name, precommit, mastername)
 
   def _build_gs_archive_url(self, mastername, master_config, buildername):
     """Returns the archive URL to pass to self.m.archive.zip_and_upload_build.
 
     Most builders on most masters use a standard format for the build archive
     URL, but some builders on some masters may specify custom places to upload
     builds to. These special cases include:
       'chromium.perf' or 'chromium.perf.fyi':
         Exclude the name of the master from the url.
       'tryserver.chromium.perf', or
@@ skipping 55 matching lines @@
   def get_compile_targets_for_scripts(self):
     return self.m.python(
         name='get compile targets for scripts',
         script=self.m.path['checkout'].join(
             'testing', 'scripts', 'get_compile_targets.py'),
         args=[
             '--output', self.m.json.output(),
             '--',
         ] + self.get_common_args_for_scripts(),
         step_test_data=lambda: self.m.json.test_api.output({}))