Fix git cl upload invocation in deps_updater.py.

third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater.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.
 
 """Fetches a copy of the latest state of a W3C test repository and commits.
 
 If this script is given the argument --auto-update, it will also attempt to
 upload a CL, triggery try jobs, and make any changes that are required for
 new failing tests before committing.
 """
@@ skipping 41 matching lines @@
         if options.target == 'wpt':
             import_commitish = self.update(WPT_DEST_NAME, WPT_REPO_URL, options.keep_w3c_repos_around)
             self._copy_resources_to_wpt()
         elif options.target == 'css':
             import_commitish = self.update(CSS_DEST_NAME, CSS_REPO_URL, options.keep_w3c_repos_around)
         else:
             raise AssertionError("Unsupported target %s" % options.target)
 
         has_changes = self.commit_changes_if_needed(chromium_commitish, import_commitish)
         if options.auto_update and has_changes:
-            commit_successful = self.do_auto_update(options.auth_refresh_token_json)
+            commit_successful = self.do_auto_update()
             if not commit_successful:
                 return 1
         return 0
 
     def parse_args(self, argv):
         parser = argparse.ArgumentParser()
         parser.description = __doc__
         parser.add_argument('-v', '--verbose', action='store_true',
                             help='log what we are doing')
         parser.add_argument('--allow-local-commits', action='store_true',
@@ skipping 180 matching lines @@
         if self.verbose:
             self.print_('rm -fr %s' % dest)
         self.fs.rmtree(dest)
 
     def path_from_webkit_base(self, *comps):
         return self.finder.path_from_webkit_base(*comps)
 
     def print_(self, msg):
         self.host.print_(msg)
 
-    def do_auto_update(self, auth_refresh_token_json):
+    def do_auto_update(self):
         """Attempts to upload a CL, make any required adjustments, and commit.
 
         This function assumes that the imported repo has already been updated,
         and that change has been committed. There may be newly-failing tests,
         so before being able to commit these new changes, we may need to update
         TestExpectations or download new baselines.
 
         Returns:
             True if successfully committed, False otherwise.
         """
         self.print_('## Uploading change list.')
         cc_list = self.get_directory_owners_to_cc()
         self.git_cl.run([
-            'upload', '-f', '--rietveld'
+            'upload', '-f', '--rietveld',
             '-m', 'W3C auto test import CL.\n\nTBR=qyearsley@chromium.org',
         ] + ['--cc=' + email for email in cc_list])
 
         self.print_('## Triggering try jobs.')
         for try_bot in self.host.builders.all_try_builder_names():
             self.git_cl.run(['try', '-b', try_bot])
         if self.git_cl.has_failing_try_results():
             self.write_test_expectations()
 
         self.run(['set-commit', '--rietveld'])
@@ skipping 42 matching lines @@
                 email_addresses.add(directory_to_owner[test_dir])
         return sorted(email_addresses)
 
     def write_test_expectations(self):
         self.print_('## Adding test expectations lines to LayoutTests/TestExpectations.')
         script_path = self.path_from_webkit_base('Tools', 'Scripts', 'update-w3c-test-expectations')
         self.run([self.host.executable, script_path])
         message = '\'Modifies TestExpectations and/or downloads new baselines for tests\''
         self.check_run(['git', 'commit', '-a', '-m', message])
         self.git_cl(['upload', '-m', message, '--rietveld'])