blob: f9b734a693f30666afc66989c6431a04ebe8de94 [file] [log] [blame]
Blink Reformat4c46d092018-04-07 15:32:37 +00001# Copyright (C) 2014 Google Inc. All rights reserved.
2#
3# Redistribution and use in source and binary forms, with or without
4# modification, are permitted provided that the following conditions are
5# met:
6#
7# * Redistributions of source code must retain the above copyright
8# notice, this list of conditions and the following disclaimer.
9# * Redistributions in binary form must reproduce the above
10# copyright notice, this list of conditions and the following disclaimer
11# in the documentation and/or other materials provided with the
12# distribution.
13# * Neither the name of Google Inc. nor the names of its
14# contributors may be used to endorse or promote products derived from
15# this software without specific prior written permission.
16#
17# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
18# "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
19# LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
20# A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
21# OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
22# SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
23# LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
24# DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
25# THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
26# (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
27# OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
Yang Guo75beda92019-10-28 08:29:25 +010028"""
29DevTools presubmit script
Blink Reformat4c46d092018-04-07 15:32:37 +000030
31See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts
32for more details about the presubmit API built into gcl.
33"""
34
35import sys
36
Yang Guoa7845d52019-10-31 11:30:23 +010037EXCLUSIVE_CHANGE_DIRECTORIES = [
38 [ 'third_party', 'v8' ],
39 [ 'node_modules' ],
40 [ 'OWNERS' ],
41]
42
Liviu Raufd2e3212019-12-18 16:38:20 +010043AUTOROLL_ACCOUNT = "devtools-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com"
Mathias Bynensa0a6e292019-12-17 13:24:08 +010044
Tim van der Lippe4d004ec2020-03-03 18:32:01 +000045
46def _ExecuteSubProcess(input_api, output_api, script_path, args, results):
47 process = input_api.subprocess.Popen(
48 [input_api.python_executable, script_path] + args, stdout=input_api.subprocess.PIPE, stderr=input_api.subprocess.STDOUT)
49 out, _ = process.communicate()
50 if process.returncode != 0:
51 results.append(output_api.PresubmitError(out))
52 else:
53 results.append(output_api.PresubmitNotifyResult(out))
54 return results
55
56
Yang Guoa7845d52019-10-31 11:30:23 +010057def _CheckChangesAreExclusiveToDirectory(input_api, output_api):
Tim van der Lippebc42a632019-11-28 14:22:55 +000058 if input_api.change.DISABLE_THIRD_PARTY_CHECK != None:
59 return []
Brandon Goddarde7028672020-01-30 09:31:04 -080060 results = [output_api.PresubmitNotifyResult('Directory Exclusivity Check:')]
Yang Guoa7845d52019-10-31 11:30:23 +010061 def IsParentDir(file, dir):
62 while file != '':
63 if file == dir:
64 return True
65 file = input_api.os_path.dirname(file)
Yang Guoa7845d52019-10-31 11:30:23 +010066 return False
67
68 def FileIsInDir(file, dirs):
69 for dir in dirs:
70 if IsParentDir(file, dir):
71 return True
72
73 affected_files = input_api.LocalPaths()
Yang Guoa7845d52019-10-31 11:30:23 +010074 num_affected = len(affected_files)
75 for dirs in EXCLUSIVE_CHANGE_DIRECTORIES:
Paul Lewis14effba2019-12-02 14:56:40 +000076 dir_list = ', '.join(dirs)
Yang Guoa7845d52019-10-31 11:30:23 +010077 affected_in_dir = filter(lambda f: FileIsInDir(f, dirs), affected_files)
78 num_in_dir = len(affected_in_dir)
79 if num_in_dir == 0:
80 continue
Tim van der Lippeebb94a92019-11-19 17:07:53 +000081 # Addition of new third_party folders must have a new entry in `.gitignore`
82 if '.gitignore' in affected_files:
83 num_in_dir = num_in_dir + 1
Yang Guoa7845d52019-10-31 11:30:23 +010084 if num_in_dir < num_affected:
Brandon Goddarde7028672020-01-30 09:31:04 -080085 results.append(output_api
Paul Lewis14effba2019-12-02 14:56:40 +000086 .PresubmitError(('CLs that affect files in "%s" should be limited to these files/directories.' % dir_list) +
Brandon Goddarde7028672020-01-30 09:31:04 -080087 ' You can disable this check by adding DISABLE_THIRD_PARTY_CHECK=<reason> to your commit message'))
88 break
89
90 return results
Yang Guoa7845d52019-10-31 11:30:23 +010091
Blink Reformat4c46d092018-04-07 15:32:37 +000092
93def _CheckBuildGN(input_api, output_api):
Brandon Goddarde7028672020-01-30 09:31:04 -080094 results = [output_api.PresubmitNotifyResult('Running BUILD.GN check:')]
Yang Guo75beda92019-10-28 08:29:25 +010095 script_path = input_api.os_path.join(input_api.PresubmitLocalPath(), 'scripts', 'check_gn.js')
Brandon Goddarde7028672020-01-30 09:31:04 -080096 results.extend(_checkWithNodeScript(input_api, output_api, script_path))
97 return results
Blink Reformat4c46d092018-04-07 15:32:37 +000098
99
Tim van der Lippee4bdd742019-12-17 16:40:16 +0100100def _CheckJSON(input_api, output_api):
Brandon Goddarde7028672020-01-30 09:31:04 -0800101 results = [output_api.PresubmitNotifyResult('Running JSON Validator:')]
Tim van der Lippee4bdd742019-12-17 16:40:16 +0100102 script_path = input_api.os_path.join(input_api.PresubmitLocalPath(), 'scripts', 'json_validator', 'validate_module_json.js')
Brandon Goddarde7028672020-01-30 09:31:04 -0800103 results.extend(_checkWithNodeScript(input_api, output_api, script_path))
104 return results
Tim van der Lippee4bdd742019-12-17 16:40:16 +0100105
106
Brandon Goddard53faba12020-01-24 16:55:04 +0000107def _CheckLicenses(input_api, output_api):
Brandon Goddarde7028672020-01-30 09:31:04 -0800108 results = [output_api.PresubmitNotifyResult('Running License Header Check:')]
Brandon Goddard53faba12020-01-24 16:55:04 +0000109 script_path = input_api.os_path.join(input_api.PresubmitLocalPath(), 'scripts', 'test', 'run_license_header_check.js')
Brandon Goddarde7028672020-01-30 09:31:04 -0800110 results.extend(_checkWithNodeScript(input_api, output_api, script_path))
111 return results
Brandon Goddard53faba12020-01-24 16:55:04 +0000112
113
Blink Reformat4c46d092018-04-07 15:32:37 +0000114def _CheckFormat(input_api, output_api):
Brandon Goddarde7028672020-01-30 09:31:04 -0800115 results = [output_api.PresubmitNotifyResult('Running Format Checks:')]
Blink Reformat4c46d092018-04-07 15:32:37 +0000116 def popen(args):
117 return input_api.subprocess.Popen(args=args, stdout=input_api.subprocess.PIPE, stderr=input_api.subprocess.STDOUT)
118
Tim van der Lipped8c23de2020-03-06 11:01:05 +0000119 format_args = ['git', 'cl', 'format', '--js']
Blink Reformat4c46d092018-04-07 15:32:37 +0000120 format_process = popen(format_args)
121 format_out, _ = format_process.communicate()
122 if format_process.returncode != 0:
Brandon Goddarde7028672020-01-30 09:31:04 -0800123 results.append(output_api.PresubmitError(format_out))
Blink Reformat4c46d092018-04-07 15:32:37 +0000124
Brandon Goddarde7028672020-01-30 09:31:04 -0800125 return results
Blink Reformat4c46d092018-04-07 15:32:37 +0000126
127
e52a82bdfb5106bd658c2c5ea465e200594be1e2019-10-29 16:02:46 -0700128def _CheckDevtoolsLocalization(input_api, output_api, check_all_files=False): # pylint: disable=invalid-name
Brandon Goddarde7028672020-01-30 09:31:04 -0800129 results = [output_api.PresubmitNotifyResult('Running Localization Checks:')]
Mandy Chene997da72019-08-22 23:50:19 +0000130 devtools_root = input_api.PresubmitLocalPath()
vidorteg2b675b02019-11-25 09:51:28 -0800131 script_path = input_api.os_path.join(devtools_root, 'scripts', 'test', 'run_localization_check.py')
Paul Lewis954a5a92019-11-20 15:33:49 +0000132 if check_all_files == True:
vidorteg2b675b02019-11-25 09:51:28 -0800133 # Scan all files and fix any errors
vidorteg75c025e2019-11-25 09:52:43 -0800134 args = ['--autofix', '--all']
Paul Lewis954a5a92019-11-20 15:33:49 +0000135 else:
vidorteg2b675b02019-11-25 09:51:28 -0800136 devtools_front_end = input_api.os_path.join(devtools_root, 'front_end')
137 affected_front_end_files = _getAffectedFiles(input_api, [devtools_front_end], ['D'],
138 ['.js', '.grdp', '.grd', 'module.json'])
139
140 if len(affected_front_end_files) == 0:
Brandon Goddarde7028672020-01-30 09:31:04 -0800141 return results
Christy Chen1ab87e02020-01-30 16:32:16 -0800142
143 with input_api.CreateTemporaryFile() as file_list:
144 for affected_file in affected_front_end_files:
145 file_list.write(affected_file + '\n')
146 file_list.close()
147
vidorteg2b675b02019-11-25 09:51:28 -0800148 # Scan only added or modified files with specific extensions.
Christy Chen1ab87e02020-01-30 16:32:16 -0800149 args = ['--autofix', '--file-list', file_list.name]
Tim van der Lippe4d004ec2020-03-03 18:32:01 +0000150
151 return _ExecuteSubProcess(input_api, output_api, script_path, args, results)
Mandy Chen465b4f72019-03-21 22:52:54 +0000152
153
Blink Reformat4c46d092018-04-07 15:32:37 +0000154def _CheckDevtoolsStyle(input_api, output_api):
Brandon Goddarde7028672020-01-30 09:31:04 -0800155 results = [output_api.PresubmitNotifyResult('Running Devtools Style Check:')]
Yang Guo75beda92019-10-28 08:29:25 +0100156 lint_path = input_api.os_path.join(input_api.PresubmitLocalPath(), 'scripts', 'test', 'run_lint_check.py')
Tim van der Lippe4d004ec2020-03-03 18:32:01 +0000157
158 return _ExecuteSubProcess(input_api, output_api, lint_path, [], results)
Blink Reformat4c46d092018-04-07 15:32:37 +0000159
160
Joel Einbinderf6f86b62019-06-10 23:19:12 +0000161def _CheckOptimizeSVGHashes(input_api, output_api):
Brandon Goddarde7028672020-01-30 09:31:04 -0800162 results = [output_api.PresubmitNotifyResult('Running SVG Optimization Check:')]
Blink Reformat4c46d092018-04-07 15:32:37 +0000163 if not input_api.platform.startswith('linux'):
Brandon Goddarde7028672020-01-30 09:31:04 -0800164 return results
Blink Reformat4c46d092018-04-07 15:32:37 +0000165
166 original_sys_path = sys.path
167 try:
168 sys.path = sys.path + [input_api.os_path.join(input_api.PresubmitLocalPath(), 'scripts', 'build')]
169 import devtools_file_hashes
170 finally:
171 sys.path = original_sys_path
172
173 absolute_local_paths = [af.AbsoluteLocalPath() for af in input_api.AffectedFiles(include_deletes=False)]
Yang Guo75beda92019-10-28 08:29:25 +0100174 images_src_path = input_api.os_path.join('devtools', 'front_end', 'Images', 'src')
175 image_source_file_paths = [path for path in absolute_local_paths if images_src_path in path and path.endswith('.svg')]
176 image_sources_path = input_api.os_path.join(input_api.PresubmitLocalPath(), 'front_end', 'Images', 'src')
177 hashes_file_name = 'optimize_svg.hashes'
Blink Reformat4c46d092018-04-07 15:32:37 +0000178 hashes_file_path = input_api.os_path.join(image_sources_path, hashes_file_name)
179 invalid_hash_file_paths = devtools_file_hashes.files_with_invalid_hashes(hashes_file_path, image_source_file_paths)
180 if len(invalid_hash_file_paths) == 0:
Brandon Goddarde7028672020-01-30 09:31:04 -0800181 return results
Blink Reformat4c46d092018-04-07 15:32:37 +0000182 invalid_hash_file_names = [input_api.os_path.basename(file_path) for file_path in invalid_hash_file_paths]
Yang Guo75beda92019-10-28 08:29:25 +0100183 file_paths_str = ', '.join(invalid_hash_file_names)
184 error_message = 'The following SVG files should be optimized using optimize_svg_images script before uploading: \n - %s' % file_paths_str
Brandon Goddarde7028672020-01-30 09:31:04 -0800185 results.append(output_api.PresubmitError(error_message))
186 return results
Blink Reformat4c46d092018-04-07 15:32:37 +0000187
188
189def _CheckCSSViolations(input_api, output_api):
Brandon Goddarde7028672020-01-30 09:31:04 -0800190 results = [output_api.PresubmitNotifyResult('Running CSS Violation Check:')]
Blink Reformat4c46d092018-04-07 15:32:37 +0000191 for f in input_api.AffectedFiles(include_deletes=False):
Yang Guo75beda92019-10-28 08:29:25 +0100192 if not f.LocalPath().endswith('.css'):
Blink Reformat4c46d092018-04-07 15:32:37 +0000193 continue
194 for line_number, line in f.ChangedContents():
Yang Guo75beda92019-10-28 08:29:25 +0100195 if '/deep/' in line:
196 results.append(output_api.PresubmitError(('%s:%d uses /deep/ selector') % (f.LocalPath(), line_number)))
197 if '::shadow' in line:
198 results.append(output_api.PresubmitError(('%s:%d uses ::shadow selector') % (f.LocalPath(), line_number)))
Blink Reformat4c46d092018-04-07 15:32:37 +0000199 return results
200
Mathias Bynens032591d2019-10-21 11:51:31 +0200201
Tim van der Lippe4d004ec2020-03-03 18:32:01 +0000202def _CheckGeneratedFiles(input_api, output_api):
Tim van der Lippeb3b90762020-03-04 15:21:52 +0000203 v8_directory_path = input_api.os_path.join(input_api.PresubmitLocalPath(), 'v8')
204 blink_directory_path = input_api.os_path.join(input_api.PresubmitLocalPath(), 'third_party', 'blink')
205 protocol_location = input_api.os_path.join(blink_directory_path, 'public', 'devtools_protocol')
206 scripts_build_path = input_api.os_path.join(input_api.PresubmitLocalPath(), 'scripts', 'build')
207
208 generated_aria_path = input_api.os_path.join(scripts_build_path, 'generate_aria.py')
209 generated_supported_css_path = input_api.os_path.join(scripts_build_path, 'generate_supported_css.py')
210 generated_protocol_path = input_api.os_path.join(scripts_build_path, 'code_generator_frontend.py')
211 concatenate_protocols_path = input_api.os_path.join(input_api.PresubmitLocalPath(), 'third_party', 'inspector_protocol',
212 'concatenate_protocols.py')
213
214 affected_files = _getAffectedFiles(input_api, [
215 v8_directory_path,
216 blink_directory_path,
217 input_api.os_path.join(input_api.PresubmitLocalPath(), 'third_party', 'pyjson5'),
218 generated_aria_path,
219 generated_supported_css_path,
220 concatenate_protocols_path,
221 generated_protocol_path,
222 ], [], ['.pdl', '.json5', '.py'])
223
224 if len(affected_files) == 0:
225 return []
226
Tim van der Lippe4d004ec2020-03-03 18:32:01 +0000227 results = [output_api.PresubmitNotifyResult('Running Generated Files Check:')]
Tim van der Lippeb0d65f12020-03-05 12:15:24 +0000228 generate_protocol_resources_path = input_api.os_path.join(input_api.PresubmitLocalPath(), 'scripts', 'deps',
229 'generate_protocol_resources.py')
Tim van der Lippe4d004ec2020-03-03 18:32:01 +0000230
Tim van der Lippeb0d65f12020-03-05 12:15:24 +0000231 return _ExecuteSubProcess(input_api, output_api, generate_protocol_resources_path, [], results)
Tim van der Lippe4d004ec2020-03-03 18:32:01 +0000232
233
Tim van der Lippe5279f842020-01-14 16:26:38 +0000234def _CheckNoUncheckedFiles(input_api, output_api):
235 results = []
236 process = input_api.subprocess.Popen(['git', 'diff', '--exit-code'],
237 stdout=input_api.subprocess.PIPE,
238 stderr=input_api.subprocess.STDOUT)
239 out, _ = process.communicate()
240 if process.returncode != 0:
241 return [output_api.PresubmitError('You have changed files that need to be committed.')]
242 return []
243
Tim van der Lippe8fdda112020-01-27 11:27:06 +0000244def _CheckForTooLargeFiles(input_api, output_api):
Christy Chen1ab87e02020-01-30 16:32:16 -0800245 """Avoid large files, especially binary files, in the repository since
Tim van der Lippe8fdda112020-01-27 11:27:06 +0000246 git doesn't scale well for those. They will be in everyone's repo
247 clones forever, forever making Chromium slower to clone and work
248 with."""
Christy Chen1ab87e02020-01-30 16:32:16 -0800249 # Uploading files to cloud storage is not trivial so we don't want
250 # to set the limit too low, but the upper limit for "normal" large
251 # files seems to be 1-2 MB, with a handful around 5-8 MB, so
252 # anything over 20 MB is exceptional.
253 TOO_LARGE_FILE_SIZE_LIMIT = 20 * 1024 * 1024 # 10 MB
254 too_large_files = []
255 for f in input_api.AffectedFiles():
256 # Check both added and modified files (but not deleted files).
257 if f.Action() in ('A', 'M'):
258 size = input_api.os_path.getsize(f.AbsoluteLocalPath())
259 if size > TOO_LARGE_FILE_SIZE_LIMIT:
260 too_large_files.append("%s: %d bytes" % (f.LocalPath(), size))
261 if too_large_files:
262 message = (
263 'Do not commit large files to git since git scales badly for those.\n' +
264 'Instead put the large files in cloud storage and use DEPS to\n' +
265 'fetch them.\n' + '\n'.join(too_large_files)
266 )
267 return [output_api.PresubmitError(
268 'Too large files found in commit', long_text=message + '\n')]
269 else:
270 return []
Tim van der Lippe8fdda112020-01-27 11:27:06 +0000271
Tim van der Lippe5279f842020-01-14 16:26:38 +0000272
Yang Guo4fd355c2019-09-19 10:59:03 +0200273def _CommonChecks(input_api, output_api):
Mathias Bynens032591d2019-10-21 11:51:31 +0200274 """Checks common to both upload and commit."""
275 results = []
Liviu Raufd2e3212019-12-18 16:38:20 +0100276 results.extend(input_api.canned_checks.CheckAuthorizedAuthor(input_api, output_api,
277 bot_whitelist=[AUTOROLL_ACCOUNT]
278 ))
Mathias Bynens032591d2019-10-21 11:51:31 +0200279 results.extend(input_api.canned_checks.CheckOwnersFormat(input_api, output_api))
280 results.extend(input_api.canned_checks.CheckOwners(input_api, output_api))
281 results.extend(input_api.canned_checks.CheckChangeHasNoCrAndHasOnlyOneEol(input_api, output_api))
282 results.extend(input_api.canned_checks.CheckChangeHasNoStrayWhitespace(input_api, output_api))
283 results.extend(input_api.canned_checks.CheckGenderNeutral(input_api, output_api))
Blink Reformat4c46d092018-04-07 15:32:37 +0000284 results.extend(_CheckBuildGN(input_api, output_api))
Tim van der Lippe4d004ec2020-03-03 18:32:01 +0000285 results.extend(_CheckGeneratedFiles(input_api, output_api))
Tim van der Lippee4bdd742019-12-17 16:40:16 +0100286 results.extend(_CheckJSON(input_api, output_api))
Brandon Goddard53faba12020-01-24 16:55:04 +0000287 results.extend(_CheckLicenses(input_api, output_api))
Blink Reformat4c46d092018-04-07 15:32:37 +0000288 results.extend(_CheckDevtoolsStyle(input_api, output_api))
Tim van der Lippe5497d482020-01-14 15:27:30 +0000289 results.extend(_CheckFormat(input_api, output_api))
Joel Einbinderf6f86b62019-06-10 23:19:12 +0000290 results.extend(_CheckOptimizeSVGHashes(input_api, output_api))
Blink Reformat4c46d092018-04-07 15:32:37 +0000291 results.extend(_CheckCSSViolations(input_api, output_api))
Yang Guoa7845d52019-10-31 11:30:23 +0100292 results.extend(_CheckChangesAreExclusiveToDirectory(input_api, output_api))
Tim van der Lippe5279f842020-01-14 16:26:38 +0000293 results.extend(_CheckNoUncheckedFiles(input_api, output_api))
Tim van der Lippe8fdda112020-01-27 11:27:06 +0000294 results.extend(_CheckForTooLargeFiles(input_api, output_api))
Blink Reformat4c46d092018-04-07 15:32:37 +0000295 return results
296
297
Liviu Raud614e092020-01-08 10:56:33 +0100298def CheckChangeOnUpload(input_api, output_api):
299 results = []
300 results.extend(_CommonChecks(input_api, output_api))
301 results.extend(_CheckDevtoolsLocalization(input_api, output_api))
302 return results
303
304
Blink Reformat4c46d092018-04-07 15:32:37 +0000305def CheckChangeOnCommit(input_api, output_api):
Mandy Chenf0fbdbe2019-08-22 23:58:37 +0000306 results = []
Yang Guo4fd355c2019-09-19 10:59:03 +0200307 results.extend(_CommonChecks(input_api, output_api))
e52a82bdfb5106bd658c2c5ea465e200594be1e2019-10-29 16:02:46 -0700308 results.extend(_CheckDevtoolsLocalization(input_api, output_api, True))
Mathias Bynens032591d2019-10-21 11:51:31 +0200309 results.extend(input_api.canned_checks.CheckChangeHasDescription(input_api, output_api))
Mandy Chenf0fbdbe2019-08-22 23:58:37 +0000310 return results
Blink Reformat4c46d092018-04-07 15:32:37 +0000311
312
Mandy Chena6be46a2019-07-09 17:06:27 +0000313def _getAffectedFiles(input_api, parent_directories, excluded_actions, accepted_endings): # pylint: disable=invalid-name
Yang Guo75beda92019-10-28 08:29:25 +0100314 """Return absolute file paths of affected files (not due to an excluded action)
Mandy Chena6be46a2019-07-09 17:06:27 +0000315 under a parent directory with an accepted file ending.
Yang Guo75beda92019-10-28 08:29:25 +0100316 """
Mandy Chena6be46a2019-07-09 17:06:27 +0000317 local_paths = [
318 f.AbsoluteLocalPath() for f in input_api.AffectedFiles() if all(f.Action() != action for action in excluded_actions)
319 ]
320 affected_files = [
321 file_name for file_name in local_paths
322 if any(parent_directory in file_name for parent_directory in parent_directories) and any(
323 file_name.endswith(accepted_ending) for accepted_ending in accepted_endings)
324 ]
325 return affected_files
326
327
Blink Reformat4c46d092018-04-07 15:32:37 +0000328def _getAffectedFrontEndFiles(input_api):
Blink Reformat4c46d092018-04-07 15:32:37 +0000329 devtools_root = input_api.PresubmitLocalPath()
Yang Guo75beda92019-10-28 08:29:25 +0100330 devtools_front_end = input_api.os_path.join(devtools_root, 'front_end')
331 affected_front_end_files = _getAffectedFiles(input_api, [devtools_front_end], ['D'], ['.js'])
Blink Reformat4c46d092018-04-07 15:32:37 +0000332 return [input_api.os_path.relpath(file_name, devtools_root) for file_name in affected_front_end_files]
333
334
335def _getAffectedJSFiles(input_api):
Blink Reformat4c46d092018-04-07 15:32:37 +0000336 devtools_root = input_api.PresubmitLocalPath()
Yang Guo75beda92019-10-28 08:29:25 +0100337 devtools_front_end = input_api.os_path.join(devtools_root, 'front_end')
338 devtools_scripts = input_api.os_path.join(devtools_root, 'scripts')
339 affected_js_files = _getAffectedFiles(input_api, [devtools_front_end, devtools_scripts], ['D'], ['.js'])
Blink Reformat4c46d092018-04-07 15:32:37 +0000340 return [input_api.os_path.relpath(file_name, devtools_root) for file_name in affected_js_files]
341
342
Lorne Mitchellc56ff2d2019-05-28 23:35:03 +0000343def _checkWithNodeScript(input_api, output_api, script_path, script_arguments=None): # pylint: disable=invalid-name
Blink Reformat4c46d092018-04-07 15:32:37 +0000344 original_sys_path = sys.path
345 try:
Yang Guo75beda92019-10-28 08:29:25 +0100346 sys.path = sys.path + [input_api.os_path.join(input_api.PresubmitLocalPath(), 'scripts')]
Yang Guod8176982019-10-04 20:30:35 +0000347 import devtools_paths
Blink Reformat4c46d092018-04-07 15:32:37 +0000348 finally:
349 sys.path = original_sys_path
350
Yang Guod8176982019-10-04 20:30:35 +0000351 node_path = devtools_paths.node_path()
Blink Reformat4c46d092018-04-07 15:32:37 +0000352
Lorne Mitchellc56ff2d2019-05-28 23:35:03 +0000353 if script_arguments is None:
354 script_arguments = []
Mandy Chen465b4f72019-03-21 22:52:54 +0000355
Blink Reformat4c46d092018-04-07 15:32:37 +0000356 process = input_api.subprocess.Popen(
Lorne Mitchellc56ff2d2019-05-28 23:35:03 +0000357 [node_path, script_path] + script_arguments, stdout=input_api.subprocess.PIPE, stderr=input_api.subprocess.STDOUT)
Blink Reformat4c46d092018-04-07 15:32:37 +0000358 out, _ = process.communicate()
359
360 if process.returncode != 0:
361 return [output_api.PresubmitError(out)]
362 return [output_api.PresubmitNotifyResult(out)]