blob: 6b1f9d3d8f40398df96f3708276e360df75c94a9 [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(),
John Emau90d02622019-07-03 08:34:26 +000082 '--no-eslintrc', '--fix', '--env=es6', '--parser-options=ecmaVersion:9',
83 '--rule={"curly": [2, "multi-or-nest", "consistent"], "semi": 2}'
Blink Reformat4c46d092018-04-07 15:32:37 +000084 ] + affected_files)
85 eslint_process.communicate()
86
87 # Need to run clang-format again to align the braces
88 popen(format_args).communicate()
89
90 return [
Erik Luo66e332c2018-04-09 18:00:14 +000091 output_api.PresubmitError("ERROR: Found formatting violations in third_party/blink/renderer/devtools.\n"
Blink Reformat4c46d092018-04-07 15:32:37 +000092 "Ran clang-format on diff\n"
93 "Use git status to check the formatting changes"),
94 output_api.PresubmitError(format_out),
95 ]
96
97
Lorne Mitchellc56ff2d2019-05-28 23:35:03 +000098def _CheckDevtoolsLocalizableResources(input_api, output_api): # pylint: disable=invalid-name
Mandy Chena6be46a2019-07-09 17:06:27 +000099 devtools_root = input_api.PresubmitLocalPath()
100 devtools_front_end = input_api.os_path.join(devtools_root, "front_end")
101 affected_front_end_files = _getAffectedFiles(input_api, [devtools_front_end], [], [".js", "module.json", ".grd", ".grdp"])
Lorne Mitchellc56ff2d2019-05-28 23:35:03 +0000102 if len(affected_front_end_files) == 0:
103 return []
104 script_path = input_api.os_path.join(input_api.PresubmitLocalPath(), "scripts", "check_localizable_resources.js")
105 args = ['--autofix']
Mandy Chena6be46a2019-07-09 17:06:27 +0000106 return _checkWithNodeScript(input_api, output_api, script_path, args)
Lorne Mitchellc56ff2d2019-05-28 23:35:03 +0000107
108
109def _CheckDevtoolsLocalization(input_api, output_api): # pylint: disable=invalid-name
110 affected_front_end_files = [
111 input_api.os_path.join(input_api.PresubmitLocalPath(), file_path) for file_path in _getAffectedFrontEndFiles(input_api)
112 ]
113 if len(affected_front_end_files) == 0:
114 return []
115 script_path = input_api.os_path.join(input_api.PresubmitLocalPath(), "scripts", "check_localizability.js")
116 return _checkWithNodeScript(input_api, output_api, script_path, affected_front_end_files)
Mandy Chen465b4f72019-03-21 22:52:54 +0000117
118
Blink Reformat4c46d092018-04-07 15:32:37 +0000119def _CheckDevtoolsStyle(input_api, output_api):
120 affected_front_end_files = _getAffectedFrontEndFiles(input_api)
121 if len(affected_front_end_files) > 0:
122 lint_path = input_api.os_path.join(input_api.PresubmitLocalPath(), "scripts", "lint_javascript.py")
123 process = input_api.subprocess.Popen(
124 [input_api.python_executable, lint_path] + affected_front_end_files,
125 stdout=input_api.subprocess.PIPE,
126 stderr=input_api.subprocess.STDOUT)
127 out, _ = process.communicate()
128 if process.returncode != 0:
129 return [output_api.PresubmitError(out)]
130 return [output_api.PresubmitNotifyResult(out)]
131 return []
132
133
134def _CompileDevtoolsFrontend(input_api, output_api):
135 compile_path = input_api.os_path.join(input_api.PresubmitLocalPath(), "scripts", "compile_frontend.py")
Lorne Mitchellc56ff2d2019-05-28 23:35:03 +0000136 out, _ = input_api.subprocess.Popen([input_api.python_executable, compile_path],
137 stdout=input_api.subprocess.PIPE,
138 stderr=input_api.subprocess.STDOUT).communicate()
Blink Reformat4c46d092018-04-07 15:32:37 +0000139 if "ERROR" in out or "WARNING" in out:
140 return [output_api.PresubmitError(out)]
141 if "NOTE" in out:
142 return [output_api.PresubmitPromptWarning(out)]
143 return []
144
145
Joel Einbinderf6f86b62019-06-10 23:19:12 +0000146def _CheckOptimizeSVGHashes(input_api, output_api):
Blink Reformat4c46d092018-04-07 15:32:37 +0000147 if not input_api.platform.startswith('linux'):
148 return []
149
150 original_sys_path = sys.path
151 try:
152 sys.path = sys.path + [input_api.os_path.join(input_api.PresubmitLocalPath(), 'scripts', 'build')]
153 import devtools_file_hashes
154 finally:
155 sys.path = original_sys_path
156
157 absolute_local_paths = [af.AbsoluteLocalPath() for af in input_api.AffectedFiles(include_deletes=False)]
158 images_src_path = input_api.os_path.join("devtools", "front_end", "Images", "src")
159 image_source_file_paths = [path for path in absolute_local_paths if images_src_path in path and path.endswith(".svg")]
160 image_sources_path = input_api.os_path.join(input_api.PresubmitLocalPath(), "front_end", "Images", "src")
Joel Einbinderf6f86b62019-06-10 23:19:12 +0000161 hashes_file_name = "optimize_svg.hashes"
Blink Reformat4c46d092018-04-07 15:32:37 +0000162 hashes_file_path = input_api.os_path.join(image_sources_path, hashes_file_name)
163 invalid_hash_file_paths = devtools_file_hashes.files_with_invalid_hashes(hashes_file_path, image_source_file_paths)
164 if len(invalid_hash_file_paths) == 0:
165 return []
166 invalid_hash_file_names = [input_api.os_path.basename(file_path) for file_path in invalid_hash_file_paths]
167 file_paths_str = ", ".join(invalid_hash_file_names)
Joel Einbinderf6f86b62019-06-10 23:19:12 +0000168 error_message = "The following SVG files should be optimized using optimize_svg_images script before uploading: \n - %s" % file_paths_str
Blink Reformat4c46d092018-04-07 15:32:37 +0000169 return [output_api.PresubmitError(error_message)]
170
171
172def _CheckCSSViolations(input_api, output_api):
173 results = []
174 for f in input_api.AffectedFiles(include_deletes=False):
175 if not f.LocalPath().endswith(".css"):
176 continue
177 for line_number, line in f.ChangedContents():
178 if "/deep/" in line:
179 results.append(output_api.PresubmitError(("%s:%d uses /deep/ selector") % (f.LocalPath(), line_number)))
180 if "::shadow" in line:
181 results.append(output_api.PresubmitError(("%s:%d uses ::shadow selector") % (f.LocalPath(), line_number)))
182 return results
183
184
185def CheckChangeOnUpload(input_api, output_api):
186 results = []
187 results.extend(_CheckBuildGN(input_api, output_api))
188 results.extend(_CheckFormat(input_api, output_api))
Lorne Mitchellc56ff2d2019-05-28 23:35:03 +0000189 results.extend(_CheckDevtoolsLocalizableResources(input_api, output_api))
Lorne Mitchell7aa2c6c2019-04-03 03:50:10 +0000190 results.extend(_CheckDevtoolsLocalization(input_api, output_api))
Blink Reformat4c46d092018-04-07 15:32:37 +0000191 results.extend(_CheckDevtoolsStyle(input_api, output_api))
192 results.extend(_CompileDevtoolsFrontend(input_api, output_api))
Joel Einbinderf6f86b62019-06-10 23:19:12 +0000193 results.extend(_CheckOptimizeSVGHashes(input_api, output_api))
Blink Reformat4c46d092018-04-07 15:32:37 +0000194 results.extend(_CheckCSSViolations(input_api, output_api))
195 return results
196
197
198def CheckChangeOnCommit(input_api, output_api):
199 return []
200
201
Mandy Chena6be46a2019-07-09 17:06:27 +0000202def _getAffectedFiles(input_api, parent_directories, excluded_actions, accepted_endings): # pylint: disable=invalid-name
203 '''Return absolute file paths of affected files (not due to an excluded action)
204 under a parent directory with an accepted file ending.
205 '''
206 local_paths = [
207 f.AbsoluteLocalPath() for f in input_api.AffectedFiles() if all(f.Action() != action for action in excluded_actions)
208 ]
209 affected_files = [
210 file_name for file_name in local_paths
211 if any(parent_directory in file_name for parent_directory in parent_directories) and any(
212 file_name.endswith(accepted_ending) for accepted_ending in accepted_endings)
213 ]
214 return affected_files
215
216
Blink Reformat4c46d092018-04-07 15:32:37 +0000217def _getAffectedFrontEndFiles(input_api):
Blink Reformat4c46d092018-04-07 15:32:37 +0000218 devtools_root = input_api.PresubmitLocalPath()
219 devtools_front_end = input_api.os_path.join(devtools_root, "front_end")
Mandy Chena6be46a2019-07-09 17:06:27 +0000220 affected_front_end_files = _getAffectedFiles(input_api, [devtools_front_end], ["D"], [".js"])
Blink Reformat4c46d092018-04-07 15:32:37 +0000221 return [input_api.os_path.relpath(file_name, devtools_root) for file_name in affected_front_end_files]
222
223
224def _getAffectedJSFiles(input_api):
Blink Reformat4c46d092018-04-07 15:32:37 +0000225 devtools_root = input_api.PresubmitLocalPath()
226 devtools_front_end = input_api.os_path.join(devtools_root, "front_end")
227 devtools_scripts = input_api.os_path.join(devtools_root, "scripts")
Mandy Chena6be46a2019-07-09 17:06:27 +0000228 affected_js_files = _getAffectedFiles(input_api, [devtools_front_end, devtools_scripts], ["D"], [".js"])
Blink Reformat4c46d092018-04-07 15:32:37 +0000229 return [input_api.os_path.relpath(file_name, devtools_root) for file_name in affected_js_files]
230
231
Lorne Mitchellc56ff2d2019-05-28 23:35:03 +0000232def _checkWithNodeScript(input_api, output_api, script_path, script_arguments=None): # pylint: disable=invalid-name
Blink Reformat4c46d092018-04-07 15:32:37 +0000233 original_sys_path = sys.path
234 try:
235 sys.path = sys.path + [input_api.os_path.join(input_api.PresubmitLocalPath(), "scripts")]
236 import local_node
237 finally:
238 sys.path = original_sys_path
239
240 node_path = local_node.node_path()
241
Lorne Mitchellc56ff2d2019-05-28 23:35:03 +0000242 if script_arguments is None:
243 script_arguments = []
Mandy Chen465b4f72019-03-21 22:52:54 +0000244
Blink Reformat4c46d092018-04-07 15:32:37 +0000245 process = input_api.subprocess.Popen(
Lorne Mitchellc56ff2d2019-05-28 23:35:03 +0000246 [node_path, script_path] + script_arguments, stdout=input_api.subprocess.PIPE, stderr=input_api.subprocess.STDOUT)
Blink Reformat4c46d092018-04-07 15:32:37 +0000247 out, _ = process.communicate()
248
249 if process.returncode != 0:
250 return [output_api.PresubmitError(out)]
251 return [output_api.PresubmitNotifyResult(out)]