diff options
Diffstat (limited to 'chromium/chromeos/components/media_app_ui')
| -rw-r--r-- | chromium/chromeos/components/media_app_ui/resources/js/BUILD.gn | 16 |
1 files changed, 16 insertions, 0 deletions
diff --git a/chromium/chromeos/components/media_app_ui/resources/js/BUILD.gn b/chromium/chromeos/components/media_app_ui/resources/js/BUILD.gn index 2d9eff502cb..643bc13ac0b 100644 --- a/chromium/chromeos/components/media_app_ui/resources/js/BUILD.gn +++ b/chromium/chromeos/components/media_app_ui/resources/js/BUILD.gn @@ -4,9 +4,18 @@ import("//third_party/closure_compiler/compile_js.gni") +# Lint checks are not well documented, but pick up some useful stuff. Currently +# require it to be requested in developer builds only. +enable_lint_checks = false + # Note we compile with reportUnknownTypes while it works, but if dependencies # get more complex, we should remove it and only enable in developer builds. media_closure_flags = default_closure_args + [ + "conformance_configs " + rebase_path( + "../../../web_applications/closure_conformance_checks.txt", + root_build_dir), + "jscomp_error=conformanceViolations", + "jscomp_error=strictCheckTypes", "jscomp_error=reportUnknownTypes", "language_in=ECMASCRIPT_2018", @@ -17,6 +26,13 @@ media_closure_flags = default_closure_args + [ "hide_warnings_for=chromeos/components/media_app_ui/media_app_ui.mojom-lite-for-compile.js", ] +if (enable_lint_checks) { + media_closure_flags += [ + "jscomp_error=lintChecks", + "hide_warnings_for=mojo/public/interfaces/bindings", + ] +} + group("closure_compile") { deps = [ ":closure_compile_app", |