Chromium Code Reviews| Index: third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater.py |
| diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater.py b/third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater.py |
| index f2f029369490421abc5a9ef268f49ef3cc32d6e2..dbf2378703e017f679d10cab8e38e8552d30ca82 100644 |
| --- a/third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater.py |
| +++ b/third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater.py |
| @@ -342,11 +342,10 @@ class DepsUpdater(object): |
| 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 .\'' |
|
qyearsley
2016/08/10 17:08:52
Nit: This change could be un-done for this CL.
|
| + message = '\'Modifies Test Expectations and/or downloads new baselines for tests\'' |
| self.check_run(['git', 'commit', '-a', '-m', message]) |
| self.check_run(['git', 'cl', 'upload', '-m', message, |
| '--auth-refresh-token-json', self.auth_refresh_token_json]) |
| - |
|
qyearsley
2016/08/10 17:08:53
Nit: This newline should be kept.
|
| def has_failing_results(self): |
| while True: |
| time.sleep(POLL_DELAY_SECONDS) |
| @@ -360,7 +359,10 @@ class DepsUpdater(object): |
| return False |
| def generate_upload_command(self, email_list): |
| - command = ['git', 'cl', 'upload', '-f', '-m', 'W3C auto test importer'] |
| + message = """W3C auto test importer |
| + |
| +TBR=qyearsley@chromium.org""" |
|
qyearsley
2016/08/10 17:08:53
Could add another newline after this, i.e.
|
| + command = ['git', 'cl', 'upload', '-f', '-m', message] |
| command += ['--cc=' + email for email in email_list] |
| if self.auth_refresh_token_json: |
| command += ['--auth-refresh-token-json', self.auth_refresh_token_json] |