blob: 68de82a5442a33ff63ad576552993253eaf89d6b [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.
28"""DevTools JSDoc validator presubmit script
29
30See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts
31for more details about the presubmit API built into gcl.
32"""
33
34import sys
35
36
37def _CheckBuildGN(input_api, output_api):
38 script_path = input_api.os_path.join(input_api.PresubmitLocalPath(), "scripts", "check_gn.js")
39 return _checkWithNodeScript(input_api, output_api, script_path)
40
41
42def _CheckFormat(input_api, output_api):
43
44 def popen(args):
45 return input_api.subprocess.Popen(args=args, stdout=input_api.subprocess.PIPE, stderr=input_api.subprocess.STDOUT)
46
47 affected_files = _getAffectedJSFiles(input_api)
48 if len(affected_files) == 0:
49 return []
50 original_sys_path = sys.path
51 try:
52 sys.path = sys.path + [input_api.os_path.join(input_api.PresubmitLocalPath(), "scripts")]
53 import local_node
54 finally:
55 sys.path = original_sys_path
56
57 ignore_files = []
58 eslint_ignore_path = input_api.os_path.join(input_api.PresubmitLocalPath(), '.eslintignore')
59 with open(eslint_ignore_path, 'r') as ignore_manifest:
60 for line in ignore_manifest:
61 ignore_files.append(line.strip())
62 formattable_files = [affected_file for affected_file in affected_files
63 if all(ignore_file not in affected_file for ignore_file in ignore_files)]
64 if len(formattable_files) == 0:
65 return []
66
67 check_formatting_process = popen(['git', 'cl', 'format', '--js', '--dry-run'] + formattable_files)
68 check_formatting_process.communicate()
69 if check_formatting_process.returncode == 0:
70 return []
71
72 format_args = ['git', 'cl', 'format', '--js'] + formattable_files
73 format_process = popen(format_args)
74 format_out, _ = format_process.communicate()
75 if format_process.returncode != 0:
76 return [output_api.PresubmitError(format_out)]
77
78 # Use eslint to autofix the braces.
79 # Also fix semicolon to avoid confusing clang-format.
80 eslint_process = popen([
81 local_node.node_path(), local_node.eslint_path(),
82 '--no-eslintrc', '--fix', '--env=es6', '--rule={"curly": [2, "multi-or-nest", "consistent"], "semi": 2}'
83 ] + affected_files)
84 eslint_process.communicate()
85
86 # Need to run clang-format again to align the braces
87 popen(format_args).communicate()
88
89 return [
Erik Luo66e332c2018-04-09 18:00:14 +000090 output_api.PresubmitError("ERROR: Found formatting violations in third_party/blink/renderer/devtools.\n"
Blink Reformat4c46d092018-04-07 15:32:37 +000091 "Ran clang-format on diff\n"
92 "Use git status to check the formatting changes"),
93 output_api.PresubmitError(format_out),
94 ]
95
96
Mandy Chen465b4f72019-03-21 22:52:54 +000097def _CheckDevtoolsLocalization(input_api, output_api): # pylint: disable=invalid-name
98 affected_front_end_files = _getAffectedFrontEndFiles(input_api)
99 if len(affected_front_end_files) == 0:
100 return []
101 else:
102 affected_front_end_files = [
103 input_api.os_path.join(input_api.PresubmitLocalPath(), file_path) for file_path in affected_front_end_files
104 ]
105 script_path = input_api.os_path.join(input_api.PresubmitLocalPath(), "scripts", "check_localizability.js")
106 return _checkWithNodeScript(input_api, output_api, script_path, affected_front_end_files)
107
108
Blink Reformat4c46d092018-04-07 15:32:37 +0000109def _CheckDevtoolsStyle(input_api, output_api):
110 affected_front_end_files = _getAffectedFrontEndFiles(input_api)
111 if len(affected_front_end_files) > 0:
112 lint_path = input_api.os_path.join(input_api.PresubmitLocalPath(), "scripts", "lint_javascript.py")
113 process = input_api.subprocess.Popen(
114 [input_api.python_executable, lint_path] + affected_front_end_files,
115 stdout=input_api.subprocess.PIPE,
116 stderr=input_api.subprocess.STDOUT)
117 out, _ = process.communicate()
118 if process.returncode != 0:
119 return [output_api.PresubmitError(out)]
120 return [output_api.PresubmitNotifyResult(out)]
121 return []
122
123
124def _CompileDevtoolsFrontend(input_api, output_api):
125 compile_path = input_api.os_path.join(input_api.PresubmitLocalPath(), "scripts", "compile_frontend.py")
126 out, _ = input_api.subprocess.Popen(
127 [input_api.python_executable, compile_path], stdout=input_api.subprocess.PIPE,
128 stderr=input_api.subprocess.STDOUT).communicate()
129 if "ERROR" in out or "WARNING" in out:
130 return [output_api.PresubmitError(out)]
131 if "NOTE" in out:
132 return [output_api.PresubmitPromptWarning(out)]
133 return []
134
135
136def _CheckConvertSVGToPNGHashes(input_api, output_api):
137 if not input_api.platform.startswith('linux'):
138 return []
139
140 original_sys_path = sys.path
141 try:
142 sys.path = sys.path + [input_api.os_path.join(input_api.PresubmitLocalPath(), 'scripts', 'build')]
143 import devtools_file_hashes
144 finally:
145 sys.path = original_sys_path
146
147 absolute_local_paths = [af.AbsoluteLocalPath() for af in input_api.AffectedFiles(include_deletes=False)]
148 images_src_path = input_api.os_path.join("devtools", "front_end", "Images", "src")
149 image_source_file_paths = [path for path in absolute_local_paths if images_src_path in path and path.endswith(".svg")]
150 image_sources_path = input_api.os_path.join(input_api.PresubmitLocalPath(), "front_end", "Images", "src")
151 hashes_file_name = "svg2png.hashes"
152 hashes_file_path = input_api.os_path.join(image_sources_path, hashes_file_name)
153 invalid_hash_file_paths = devtools_file_hashes.files_with_invalid_hashes(hashes_file_path, image_source_file_paths)
154 if len(invalid_hash_file_paths) == 0:
155 return []
156 invalid_hash_file_names = [input_api.os_path.basename(file_path) for file_path in invalid_hash_file_paths]
157 file_paths_str = ", ".join(invalid_hash_file_names)
158 error_message = "The following SVG files should be converted to PNG using convert_svg_images_png.py script before uploading: \n - %s" % file_paths_str
159 return [output_api.PresubmitError(error_message)]
160
161
162def _CheckOptimizePNGHashes(input_api, output_api):
163 if not input_api.platform.startswith('linux'):
164 return []
165
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)]
174 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_png.hashes"
178 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:
181 return []
182 invalid_hash_file_names = [input_api.os_path.basename(file_path) for file_path in invalid_hash_file_paths]
183 file_paths_str = ", ".join(invalid_hash_file_names)
184 error_message = "The following PNG files should be optimized using optimize_png_images.py script before uploading: \n - %s" % file_paths_str
185 return [output_api.PresubmitError(error_message)]
186
187
188def _CheckCSSViolations(input_api, output_api):
189 results = []
190 for f in input_api.AffectedFiles(include_deletes=False):
191 if not f.LocalPath().endswith(".css"):
192 continue
193 for line_number, line in f.ChangedContents():
194 if "/deep/" in line:
195 results.append(output_api.PresubmitError(("%s:%d uses /deep/ selector") % (f.LocalPath(), line_number)))
196 if "::shadow" in line:
197 results.append(output_api.PresubmitError(("%s:%d uses ::shadow selector") % (f.LocalPath(), line_number)))
198 return results
199
200
201def CheckChangeOnUpload(input_api, output_api):
202 results = []
203 results.extend(_CheckBuildGN(input_api, output_api))
204 results.extend(_CheckFormat(input_api, output_api))
Lorne Mitchell7aa2c6c2019-04-03 03:50:10 +0000205 results.extend(_CheckDevtoolsLocalization(input_api, output_api))
Blink Reformat4c46d092018-04-07 15:32:37 +0000206 results.extend(_CheckDevtoolsStyle(input_api, output_api))
207 results.extend(_CompileDevtoolsFrontend(input_api, output_api))
208 results.extend(_CheckConvertSVGToPNGHashes(input_api, output_api))
209 results.extend(_CheckOptimizePNGHashes(input_api, output_api))
210 results.extend(_CheckCSSViolations(input_api, output_api))
211 return results
212
213
214def CheckChangeOnCommit(input_api, output_api):
215 return []
216
217
218def _getAffectedFrontEndFiles(input_api):
219 local_paths = [f.AbsoluteLocalPath() for f in input_api.AffectedFiles() if f.Action() != "D"]
220 devtools_root = input_api.PresubmitLocalPath()
221 devtools_front_end = input_api.os_path.join(devtools_root, "front_end")
222 affected_front_end_files = [
223 file_name for file_name in local_paths if devtools_front_end in file_name and file_name.endswith(".js")
224 ]
225 return [input_api.os_path.relpath(file_name, devtools_root) for file_name in affected_front_end_files]
226
227
228def _getAffectedJSFiles(input_api):
229 local_paths = [f.AbsoluteLocalPath() for f in input_api.AffectedFiles() if f.Action() != "D"]
230 devtools_root = input_api.PresubmitLocalPath()
231 devtools_front_end = input_api.os_path.join(devtools_root, "front_end")
232 devtools_scripts = input_api.os_path.join(devtools_root, "scripts")
233 affected_js_files = [
234 file_name for file_name in local_paths
235 if (devtools_front_end in file_name or devtools_scripts in file_name) and file_name.endswith(".js")
236 ]
237 return [input_api.os_path.relpath(file_name, devtools_root) for file_name in affected_js_files]
238
239
Mandy Chen465b4f72019-03-21 22:52:54 +0000240def _checkWithNodeScript(input_api, output_api, script_path, files=None): # pylint: disable=invalid-name
Blink Reformat4c46d092018-04-07 15:32:37 +0000241 original_sys_path = sys.path
242 try:
243 sys.path = sys.path + [input_api.os_path.join(input_api.PresubmitLocalPath(), "scripts")]
244 import local_node
245 finally:
246 sys.path = original_sys_path
247
248 node_path = local_node.node_path()
249
Mandy Chen465b4f72019-03-21 22:52:54 +0000250 if files is None:
251 files = []
252
Blink Reformat4c46d092018-04-07 15:32:37 +0000253 process = input_api.subprocess.Popen(
Mandy Chen465b4f72019-03-21 22:52:54 +0000254 [node_path, script_path] + files, stdout=input_api.subprocess.PIPE, stderr=input_api.subprocess.STDOUT)
Blink Reformat4c46d092018-04-07 15:32:37 +0000255 out, _ = process.communicate()
256
257 if process.returncode != 0:
258 return [output_api.PresubmitError(out)]
259 return [output_api.PresubmitNotifyResult(out)]