| OLD | NEW |
| 1 # Copyright 2016 The Chromium Authors. All rights reserved. | 1 # Copyright 2016 The Chromium Authors. All rights reserved. |
| 2 # Use of this source code is governed by a BSD-style license that can be | 2 # Use of this source code is governed by a BSD-style license that can be |
| 3 # found in the LICENSE file. | 3 # found in the LICENSE file. |
| 4 | 4 |
| 5 import re | 5 import re |
| 6 | 6 |
| 7 | 7 |
| 8 def PostUploadHook(cl, change, output_api): | 8 def PostUploadHook(cl, change, output_api): |
| 9 rietveld_obj = cl.RpcServer() | 9 rietveld_obj = cl.RpcServer() |
| 10 description = rietveld_obj.get_description(cl.issue) | 10 description = rietveld_obj.get_description(cl.issue) |
| 11 | 11 |
| 12 existing_bots = (change.CQ_INCLUDE_TRYBOTS or '').split(';') | 12 existing_bots = (change.CQ_INCLUDE_TRYBOTS or '').split(';') |
| 13 clean_bots = set(filter(None, map(lambda s: s.strip(), existing_bots))) | 13 clean_bots = set(filter(None, map(lambda s: s.strip(), existing_bots))) |
| 14 new_bots = clean_bots | set( | 14 new_bots = clean_bots | set( |
| 15 ['master.tryserver.chromium.linux:closure_compilation']) | 15 ['master.tryserver.chromium.linux:closure_compilation']) |
| 16 new_tag = 'CQ_INCLUDE_TRYBOTS=%s' % ';'.join(new_bots) | 16 new_tag = 'CQ_INCLUDE_TRYBOTS=%s' % ';'.join(new_bots) |
| 17 | 17 |
| 18 if clean_bots: | 18 if clean_bots: |
| 19 tag_reg = '^CQ_INCLUDE_TRYBOTS=.*$' | 19 tag_reg = '^CQ_INCLUDE_TRYBOTS=.*$' |
| 20 new_description = re.sub(tag_reg, new_tag, description, flags=re.M | re.I) | 20 new_description = re.sub(tag_reg, new_tag, description, flags=re.M | re.I) |
| 21 else: | 21 else: |
| 22 new_description = description + '\n' + new_tag | 22 new_description = description + '\n' + new_tag |
| 23 | 23 |
| 24 if new_description == description: | 24 if new_description == description: |
| 25 return [] | 25 return [] |
| 26 | 26 |
| 27 rietveld_obj.update_description(cl.issue, new_description) | 27 rietveld_obj.update_description(cl.issue, new_description) |
| 28 return [output_api.PresubmitNotifyResult( | 28 return [output_api.PresubmitNotifyResult( |
| 29 'Automatically added optional Closure bots to run on CQ.')] | 29 'Automatically added optional Closure bots to run on CQ.')] |
| 30 |
| 31 |
| 32 def CheckChangeOnUpload(input_api, output_api): |
| 33 return _CheckForTranslations(input_api, output_api) |
| 34 |
| 35 |
| 36 def CheckChangeOnCommit(input_api, output_api): |
| 37 return _CheckForTranslations(input_api, output_api) |
| 38 |
| 39 |
| 40 def _CheckForTranslations(input_api, output_api): |
| 41 shared_keywords = ['i18n('] |
| 42 html_keywords = shared_keywords + ['$118n{'] |
| 43 js_keywords = shared_keywords + ['I18nBehavior', 'loadTimeData.'] |
| 44 |
| 45 errors = [] |
| 46 |
| 47 for f in input_api.AffectedFiles(): |
| 48 local_path = f.LocalPath() |
| 49 |
| 50 keywords = None |
| 51 if local_path.endswith('.js'): |
| 52 keywords = js_keywords |
| 53 elif local_path.endswith('.html'): |
| 54 keywords = html_keywords |
| 55 |
| 56 if not keywords: |
| 57 continue |
| 58 |
| 59 for lnum, line in f.ChangedContents(): |
| 60 if any(line for keyword in keywords if keyword in line): |
| 61 errors.append("%s:%d\n%s" % (f.LocalPath(), lnum, line)) |
| 62 |
| 63 if not errors: |
| 64 return [] |
| 65 |
| 66 return [output_api.PresubmitError("\n".join(errors) + """ |
| 67 |
| 68 Don't embed translations directly in shared UI code. Instead, inject your |
| 69 translation from the place using the shared code. For an example: see |
| 70 <cr-dialog>#closeText (http://bit.ly/2eLEsqh).""")] |
| OLD | NEW |