Simplification: Remove support for specifying suffix lists when rebaselining.

third_party/WebKit/Tools/Scripts/webkitpy/tool/commands/rebaseline_cl.py

 # Copyright 2016 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.
 
 """A command to fetch new baselines from try jobs for the current CL."""
 
 import json
 import logging
 import optparse
 
 from webkitpy.common.net.git_cl import GitCL
-from webkitpy.layout_tests.models.test_expectations import BASELINE_SUFFIX_LIST
 from webkitpy.tool.commands.rebaseline import AbstractParallelRebaselineCommand
 from webkitpy.w3c.wpt_manifest import WPTManifest
 
 _log = logging.getLogger(__name__)
 
 
 class RebaselineCL(AbstractParallelRebaselineCommand):
     name = 'rebaseline-cl'
     help_text = 'Fetches new baselines for a CL from test runs on try bots.'
     long_help = ('By default, this command will check the latest try job results '
@@ skipping 60 matching lines @@
             return 1
 
         _log.debug('Getting results for issue %d.', issue_number)
         builds_to_results = self._fetch_results(builds)
         if builds_to_results is None:
             return 1
 
         test_prefix_list = {}
         if args:
             for test in args:
-                test_prefix_list[test] = {b: BASELINE_SUFFIX_LIST for b in builds}
+                test_prefix_list[test] = builds
         else:
             test_prefix_list = self._test_prefix_list(
                 builds_to_results,
                 only_changed_tests=options.only_changed_tests)
 
         self._log_test_prefix_list(test_prefix_list)
 
         if not options.dry_run:
             self.rebaseline(options, test_prefix_list)
         return 0
@@ skipping 81 matching lines @@
             # the path separator, and they're always relative to repo root.
             # TODO(qyearsley): Do this without using a hard-coded constant.
             test_base = 'third_party/WebKit/LayoutTests/'
             tests_in_cl = [f[len(test_base):] for f in files_in_cl if f.startswith(test_base)]
         result = {}
         for build, tests in builds_to_tests.iteritems():
             for test in tests:
                 if only_changed_tests and test not in tests_in_cl:
                     continue
                 if test not in result:
-                    result[test] = {}
-                result[test][build] = BASELINE_SUFFIX_LIST
+                    result[test] = []
+                result[test].append(build)
         return result
 
     def _tests_to_rebaseline(self, build, layout_test_results):
         """Fetches a list of tests that should be rebaselined for some build."""
         unexpected_results = layout_test_results.didnt_run_as_expected_results()
         tests = sorted(r.test_name() for r in unexpected_results
                        if r.is_missing_baseline() or r.has_mismatch_result())
 
         new_failures = self._fetch_tests_with_new_failures(build)
         if new_failures is None:
@@ skipping 19 matching lines @@
     def _log_test_prefix_list(test_prefix_list):
         """Logs the tests to download new baselines for."""
         if not test_prefix_list:
             _log.info('No tests to rebaseline; exiting.')
             return
         _log.debug('Tests to rebaseline:')
         for test, builds in test_prefix_list.iteritems():
             _log.debug('  %s:', test)
             for build in sorted(builds):
                 _log.debug('    %s', build)