Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 +0000 | [diff] [blame] | 1 | # 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 | |
| 30 | See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts |
| 31 | for more details about the presubmit API built into gcl. |
| 32 | """ |
| 33 | |
| 34 | import sys |
| 35 | |
| 36 | |
| 37 | def _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 | |
| 42 | def _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([ |
Tim van der Lippe | 63a4d3d | 2019-09-18 15:31:38 +0000 | [diff] [blame] | 81 | local_node.node_path(), |
Aaron Gable | c9d0005 | 2019-09-18 23:35:18 +0000 | [diff] [blame^] | 82 | local_node.eslint_path(), '--no-eslintrc', '--fix', '--env=es6', '--parser-options=ecmaVersion:9', |
John Emau | 90d0262 | 2019-07-03 08:34:26 +0000 | [diff] [blame] | 83 | '--rule={"curly": [2, "multi-or-nest", "consistent"], "semi": 2}' |
Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 +0000 | [diff] [blame] | 84 | ] + 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 Luo | 66e332c | 2018-04-09 18:00:14 +0000 | [diff] [blame] | 91 | output_api.PresubmitError("ERROR: Found formatting violations in third_party/blink/renderer/devtools.\n" |
Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 +0000 | [diff] [blame] | 92 | "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 Mitchell | c56ff2d | 2019-05-28 23:35:03 +0000 | [diff] [blame] | 98 | def _CheckDevtoolsLocalizableResources(input_api, output_api): # pylint: disable=invalid-name |
Mandy Chen | a6be46a | 2019-07-09 17:06:27 +0000 | [diff] [blame] | 99 | 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 Mitchell | c56ff2d | 2019-05-28 23:35:03 +0000 | [diff] [blame] | 102 | 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 Chen | a6be46a | 2019-07-09 17:06:27 +0000 | [diff] [blame] | 106 | return _checkWithNodeScript(input_api, output_api, script_path, args) |
Lorne Mitchell | c56ff2d | 2019-05-28 23:35:03 +0000 | [diff] [blame] | 107 | |
| 108 | |
| 109 | def _CheckDevtoolsLocalization(input_api, output_api): # pylint: disable=invalid-name |
Mandy Chen | e997da7 | 2019-08-22 23:50:19 +0000 | [diff] [blame] | 110 | devtools_root = input_api.PresubmitLocalPath() |
| 111 | devtools_front_end = input_api.os_path.join(devtools_root, "front_end") |
| 112 | affected_front_end_files = _getAffectedFiles(input_api, [devtools_front_end], ["D"], [".js", ".grdp"]) |
Lorne Mitchell | c56ff2d | 2019-05-28 23:35:03 +0000 | [diff] [blame] | 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 Chen | 465b4f7 | 2019-03-21 22:52:54 +0000 | [diff] [blame] | 117 | |
| 118 | |
Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 +0000 | [diff] [blame] | 119 | def _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 | |
| 134 | def _CompileDevtoolsFrontend(input_api, output_api): |
| 135 | compile_path = input_api.os_path.join(input_api.PresubmitLocalPath(), "scripts", "compile_frontend.py") |
Lorne Mitchell | c56ff2d | 2019-05-28 23:35:03 +0000 | [diff] [blame] | 136 | 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 Reformat | 4c46d09 | 2018-04-07 15:32:37 +0000 | [diff] [blame] | 139 | 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 Einbinder | f6f86b6 | 2019-06-10 23:19:12 +0000 | [diff] [blame] | 146 | def _CheckOptimizeSVGHashes(input_api, output_api): |
Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 +0000 | [diff] [blame] | 147 | 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 Einbinder | f6f86b6 | 2019-06-10 23:19:12 +0000 | [diff] [blame] | 161 | hashes_file_name = "optimize_svg.hashes" |
Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 +0000 | [diff] [blame] | 162 | 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 Einbinder | f6f86b6 | 2019-06-10 23:19:12 +0000 | [diff] [blame] | 168 | error_message = "The following SVG files should be optimized using optimize_svg_images script before uploading: \n - %s" % file_paths_str |
Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 +0000 | [diff] [blame] | 169 | return [output_api.PresubmitError(error_message)] |
| 170 | |
| 171 | |
| 172 | def _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 | |
| 185 | def CheckChangeOnUpload(input_api, output_api): |
| 186 | results = [] |
| 187 | results.extend(_CheckBuildGN(input_api, output_api)) |
| 188 | results.extend(_CheckFormat(input_api, output_api)) |
Lorne Mitchell | c56ff2d | 2019-05-28 23:35:03 +0000 | [diff] [blame] | 189 | results.extend(_CheckDevtoolsLocalizableResources(input_api, output_api)) |
Lorne Mitchell | 7aa2c6c | 2019-04-03 03:50:10 +0000 | [diff] [blame] | 190 | results.extend(_CheckDevtoolsLocalization(input_api, output_api)) |
Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 +0000 | [diff] [blame] | 191 | results.extend(_CheckDevtoolsStyle(input_api, output_api)) |
| 192 | results.extend(_CompileDevtoolsFrontend(input_api, output_api)) |
Joel Einbinder | f6f86b6 | 2019-06-10 23:19:12 +0000 | [diff] [blame] | 193 | results.extend(_CheckOptimizeSVGHashes(input_api, output_api)) |
Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 +0000 | [diff] [blame] | 194 | results.extend(_CheckCSSViolations(input_api, output_api)) |
| 195 | return results |
| 196 | |
| 197 | |
| 198 | def CheckChangeOnCommit(input_api, output_api): |
Mandy Chen | f0fbdbe | 2019-08-22 23:58:37 +0000 | [diff] [blame] | 199 | results = [] |
| 200 | results.extend(_CheckDevtoolsLocalizableResources(input_api, output_api)) |
| 201 | results.extend(_CheckDevtoolsLocalization(input_api, output_api)) |
| 202 | return results |
Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 +0000 | [diff] [blame] | 203 | |
| 204 | |
Mandy Chen | a6be46a | 2019-07-09 17:06:27 +0000 | [diff] [blame] | 205 | def _getAffectedFiles(input_api, parent_directories, excluded_actions, accepted_endings): # pylint: disable=invalid-name |
| 206 | '''Return absolute file paths of affected files (not due to an excluded action) |
| 207 | under a parent directory with an accepted file ending. |
| 208 | ''' |
| 209 | local_paths = [ |
| 210 | f.AbsoluteLocalPath() for f in input_api.AffectedFiles() if all(f.Action() != action for action in excluded_actions) |
| 211 | ] |
| 212 | affected_files = [ |
| 213 | file_name for file_name in local_paths |
| 214 | if any(parent_directory in file_name for parent_directory in parent_directories) and any( |
| 215 | file_name.endswith(accepted_ending) for accepted_ending in accepted_endings) |
| 216 | ] |
| 217 | return affected_files |
| 218 | |
| 219 | |
Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 +0000 | [diff] [blame] | 220 | def _getAffectedFrontEndFiles(input_api): |
Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 +0000 | [diff] [blame] | 221 | devtools_root = input_api.PresubmitLocalPath() |
| 222 | devtools_front_end = input_api.os_path.join(devtools_root, "front_end") |
Mandy Chen | a6be46a | 2019-07-09 17:06:27 +0000 | [diff] [blame] | 223 | affected_front_end_files = _getAffectedFiles(input_api, [devtools_front_end], ["D"], [".js"]) |
Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 +0000 | [diff] [blame] | 224 | return [input_api.os_path.relpath(file_name, devtools_root) for file_name in affected_front_end_files] |
| 225 | |
| 226 | |
| 227 | def _getAffectedJSFiles(input_api): |
Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 +0000 | [diff] [blame] | 228 | devtools_root = input_api.PresubmitLocalPath() |
| 229 | devtools_front_end = input_api.os_path.join(devtools_root, "front_end") |
| 230 | devtools_scripts = input_api.os_path.join(devtools_root, "scripts") |
Mandy Chen | a6be46a | 2019-07-09 17:06:27 +0000 | [diff] [blame] | 231 | affected_js_files = _getAffectedFiles(input_api, [devtools_front_end, devtools_scripts], ["D"], [".js"]) |
Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 +0000 | [diff] [blame] | 232 | return [input_api.os_path.relpath(file_name, devtools_root) for file_name in affected_js_files] |
| 233 | |
| 234 | |
Lorne Mitchell | c56ff2d | 2019-05-28 23:35:03 +0000 | [diff] [blame] | 235 | def _checkWithNodeScript(input_api, output_api, script_path, script_arguments=None): # pylint: disable=invalid-name |
Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 +0000 | [diff] [blame] | 236 | original_sys_path = sys.path |
| 237 | try: |
| 238 | sys.path = sys.path + [input_api.os_path.join(input_api.PresubmitLocalPath(), "scripts")] |
| 239 | import local_node |
| 240 | finally: |
| 241 | sys.path = original_sys_path |
| 242 | |
| 243 | node_path = local_node.node_path() |
| 244 | |
Lorne Mitchell | c56ff2d | 2019-05-28 23:35:03 +0000 | [diff] [blame] | 245 | if script_arguments is None: |
| 246 | script_arguments = [] |
Mandy Chen | 465b4f7 | 2019-03-21 22:52:54 +0000 | [diff] [blame] | 247 | |
Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 +0000 | [diff] [blame] | 248 | process = input_api.subprocess.Popen( |
Lorne Mitchell | c56ff2d | 2019-05-28 23:35:03 +0000 | [diff] [blame] | 249 | [node_path, script_path] + script_arguments, stdout=input_api.subprocess.PIPE, stderr=input_api.subprocess.STDOUT) |
Blink Reformat | 4c46d09 | 2018-04-07 15:32:37 +0000 | [diff] [blame] | 250 | out, _ = process.communicate() |
| 251 | |
| 252 | if process.returncode != 0: |
| 253 | return [output_api.PresubmitError(out)] |
| 254 | return [output_api.PresubmitNotifyResult(out)] |