In rebaseline-cl, don't check for local file existence.

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 a Rietveld issue.
 
 This command interacts with the Rietveld API to get information about try jobs
 with layout test results.
 """
 
 import json
 import logging
 import optparse
 
 from webkitpy.common.net.rietveld import Rietveld
 from webkitpy.common.net.web import Web
 from webkitpy.common.net.git_cl import GitCL
-from webkitpy.common.webkit_finder import WebKitFinder
 from webkitpy.layout_tests.models.test_expectations import BASELINE_SUFFIX_LIST
 from webkitpy.tool.commands.rebaseline import AbstractParallelRebaselineCommand
 
 _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 37 matching lines @@
             _log.info('No builds to download baselines from.')
 
         if args:
             test_prefix_list = {}
             for test in args:
                 test_prefix_list[test] = {b: BASELINE_SUFFIX_LIST for b in builds}
         else:
             test_prefix_list = self._test_prefix_list(
                 issue_number, only_changed_tests=options.only_changed_tests)
 
-        # TODO(qyearsley): Fix places where non-existing tests may be added:
-        #  1. Make sure that the tests obtained when passing --only-changed-tests include only existing tests.
-        test_prefix_list = self._filter_existing(test_prefix_list)
-
         self._log_test_prefix_list(test_prefix_list)
 
         if options.dry_run:
             return
         self.rebaseline(options, test_prefix_list)
 
-    def _filter_existing(self, test_prefix_list):
-        """Filters out entries in |test_prefix_list| for tests that don't exist."""
-        new_test_prefix_list = {}
-        port = self._tool.port_factory.get()
-        for test in test_prefix_list:
-            path = port.abspath_for_test(test)
-            if self._tool.filesystem.exists(path):
-                new_test_prefix_list[test] = test_prefix_list[test]
-            else:
-                _log.warning('%s not found, removing from list.', path)
-        return new_test_prefix_list
-
     def _get_issue_number(self, options):
         """Gets the Rietveld CL number from either |options| or from the current local branch."""
         if options.issue:
             return options.issue
         issue_number = self.git_cl().get_issue_number()
         _log.debug('Issue number for current branch: %s', issue_number)
         if not issue_number.isdigit():
             _log.error('No issue number given and no issue for current branch. This tool requires a CL\n'
                        'to operate on; please run `git cl upload` on this branch first, or use the --issue\n'
                        'option to download baselines for another existing CL.')
@@ skipping 113 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)