| OLD | NEW |
| 1 # Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 # Copyright (c) 2012 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 """Presubmit script for Chromium JS resources. | 5 """Presubmit script for Chromium JS resources. |
| 6 | 6 |
| 7 See chrome/browser/PRESUBMIT.py | 7 See chrome/browser/PRESUBMIT.py |
| 8 """ | 8 """ |
| 9 | 9 |
| 10 import regex_check |
| 11 |
| 12 |
| 10 class JSChecker(object): | 13 class JSChecker(object): |
| 11 def __init__(self, input_api, output_api, file_filter=None): | 14 def __init__(self, input_api, output_api, file_filter=None): |
| 12 self.input_api = input_api | 15 self.input_api = input_api |
| 13 self.output_api = output_api | 16 self.output_api = output_api |
| 14 self.file_filter = file_filter | 17 self.file_filter = file_filter |
| 15 | 18 |
| 16 def RegexCheck(self, line_number, line, regex, message): | 19 def RegexCheck(self, line_number, line, regex, message): |
| 17 """Searches for |regex| in |line| to check for a particular style | 20 return regex_check.RegexCheck( |
| 18 violation, returning a message like the one below if the regex matches. | 21 self.input_api.re, line_number, line, regex, message) |
| 19 The |regex| must have exactly one capturing group so that the relevant | |
| 20 part of |line| can be highlighted. If more groups are needed, use | |
| 21 "(?:...)" to make a non-capturing group. Sample message: | |
| 22 | |
| 23 line 6: Use var instead of const. | |
| 24 const foo = bar(); | |
| 25 ^^^^^ | |
| 26 """ | |
| 27 match = self.input_api.re.search(regex, line) | |
| 28 if match: | |
| 29 assert len(match.groups()) == 1 | |
| 30 start = match.start(1) | |
| 31 length = match.end(1) - start | |
| 32 return ' line %d: %s\n%s\n%s' % ( | |
| 33 line_number, | |
| 34 message, | |
| 35 line, | |
| 36 self.error_highlight(start, length)) | |
| 37 return '' | |
| 38 | 22 |
| 39 def ChromeSendCheck(self, i, line): | 23 def ChromeSendCheck(self, i, line): |
| 40 """Checks for a particular misuse of 'chrome.send'.""" | 24 """Checks for a particular misuse of 'chrome.send'.""" |
| 41 return self.RegexCheck(i, line, r"chrome\.send\('[^']+'\s*(, \[\])\)", | 25 return self.RegexCheck(i, line, r"chrome\.send\('[^']+'\s*(, \[\])\)", |
| 42 'Passing an empty array to chrome.send is unnecessary') | 26 'Passing an empty array to chrome.send is unnecessary') |
| 43 | 27 |
| 44 def ConstCheck(self, i, line): | 28 def ConstCheck(self, i, line): |
| 45 """Check for use of the 'const' keyword.""" | 29 """Check for use of the 'const' keyword.""" |
| 46 if self.input_api.re.search(r'\*\s+@const', line): | 30 if self.input_api.re.search(r'\*\s+@const', line): |
| 47 # Probably a JsDoc line | 31 # Probably a JsDoc line |
| (...skipping 198 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 246 | 230 |
| 247 if results: | 231 if results: |
| 248 results.append(self.output_api.PresubmitNotifyResult( | 232 results.append(self.output_api.PresubmitNotifyResult( |
| 249 'See the JavaScript style guide at ' | 233 'See the JavaScript style guide at ' |
| 250 'http://www.chromium.org/developers/web-development-style-guide' | 234 'http://www.chromium.org/developers/web-development-style-guide' |
| 251 '#TOC-JavaScript and if you have any feedback about the JavaScript ' | 235 '#TOC-JavaScript and if you have any feedback about the JavaScript ' |
| 252 'PRESUBMIT check, contact tbreisacher@chromium.org or ' | 236 'PRESUBMIT check, contact tbreisacher@chromium.org or ' |
| 253 'dbeam@chromium.org')) | 237 'dbeam@chromium.org')) |
| 254 | 238 |
| 255 return results | 239 return results |
| OLD | NEW |