Change logging statements to not use the "%" operator.

third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/update_test_expectations.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.
 
 """Updates TestExpectations based on results in builder bots.
 
 Scans the TestExpectations file and uses results from actual builder bots runs
 to remove tests that are marked as flaky but don't fail in the specified way.
 
 E.g. If a test has this expectation:
@@ skipping 23 matching lines @@
 def main(host, bot_test_expectations_factory, argv):
     parser = argparse.ArgumentParser(epilog=__doc__, formatter_class=argparse.RawTextHelpFormatter)
     parser.parse_args(argv)
 
     port = host.port_factory.get()
 
     logging.basicConfig(level=logging.INFO, format="%(message)s")
 
     expectations_file = port.path_to_generic_test_expectations_file()
     if not host.filesystem.isfile(expectations_file):
-        _log.warn("Didn't find generic expectations file at: " + expectations_file)
+        _log.warning("Didn't find generic expectations file at: " + expectations_file)
         return 1
 
     remove_flakes_o_matic = RemoveFlakesOMatic(host,
                                                port,
                                                bot_test_expectations_factory)
 
     test_expectations = remove_flakes_o_matic.get_updated_test_expectations()
 
     remove_flakes_o_matic.write_test_expectations(test_expectations,
                                                   expectations_file)
@@ skipping 42 matching lines @@
 
             if not builder_name:
                 _log.error('Failed to get builder for config [%s, %s, %s]',
                            config.version, config.architecture, config.build_type)
                 # TODO(bokan): Matching configurations often give us bots that don't have a
                 # builder in builders.py's exact_matches. Should we ignore those or be conservative
                 # and assume we need these expectations to make a decision?
                 return False
 
             if builder_name not in self._builder_results_by_path.keys():
-                _log.error('Failed to find results for builder "%s"' % builder_name)
+                _log.error('Failed to find results for builder "%s"', builder_name)
                 return False
 
             results_by_path = self._builder_results_by_path[builder_name]
 
             # No results means the tests were all skipped or all results are passing.
             if test_expectation_line.path not in results_by_path.keys():
                 continue
 
             results_for_single_test = results_by_path[test_expectation_line.path]
 
@@ skipping 71 matching lines @@
         builder_results_by_path = {}
         for builder_name in self._host.builders.all_builder_names():
             expectations_for_builder = (
                 self._expectations_factory.expectations_for_builder(builder_name)
             )
 
             if not expectations_for_builder:
                 # This is not fatal since we may not need to check these
                 # results. If we do need these results we'll log an error later
                 # when trying to check against them.
-                _log.warn('Downloaded results are missing results for builder "%s"' % builder_name)
+                _log.warning('Downloaded results are missing results for builder "%s"', builder_name)
                 continue
 
             builder_results_by_path[builder_name] = (
                 expectations_for_builder.all_results_by_path()
             )
         return builder_results_by_path
 
     def _remove_associated_comments_and_whitespace(self, expectations, removed_index):
         """Removes comments and whitespace from an empty expectation block.
 
@@ skipping 72 matching lines @@
         """Writes the given TestExpectations object to the filesystem.
 
         Args:
             test_expectations: The TestExpectations object to write.
             test_expectations_file: The full file path of the Blink
                 TestExpectations file. This file will be overwritten.
         """
         self._host.filesystem.write_text_file(
             test_expectations_file,
             TestExpectations.list_to_string(test_expectations, reconstitute_only_these=[]))