Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(138)

Unified Diff: remoting/remoting_webapp_compile.gypi

Issue 1037613002: Rework how remoting JS compilation GYP works. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: remoting/remoting_webapp_compile.gypi
diff --git a/remoting/remoting_webapp_compile.gypi b/remoting/remoting_webapp_compile.gypi
new file mode 100644
index 0000000000000000000000000000000000000000..094ff9290e57a8bbe94b5aa0fae10e4cb16253ca
--- /dev/null
+++ b/remoting/remoting_webapp_compile.gypi
@@ -0,0 +1,80 @@
+# Copyright 2015 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+#
+# File in charge of Closure compiling remoting's webapp.
+
+{
+ 'variables': {
+ 'success_stamp': '<(PRODUCT_DIR)/<(_target_name)_jscompile.stamp',
+ 'success_stamp_bt': '<(PRODUCT_DIR)/<(_target_name)_bt_jscompile.stamp',
+ 'success_stamp_ut': '<(PRODUCT_DIR)/<(_target_name)_ut_jscompile.stamp',
+ },
+ 'actions': [
+ {
+ 'action_name': 'Verify remoting webapp',
+ 'inputs': [
+ 'remoting_webapp_compile.gypi',
+ 'remoting_webapp_files.gypi',
+ '<@(remoting_webapp_crd_js_files)',
+ '<@(remoting_webapp_js_proto_files)',
+ ],
+ 'outputs': [
+ '<(success_stamp)',
+ ],
+ 'action': [
+ 'python', '<(DEPTH)/third_party/closure_compiler/checker.py',
+ '--strict',
+ '--no-single-file',
+ '--success-stamp', '<(success_stamp)',
+ '<@(remoting_webapp_crd_js_files)',
+ '<@(remoting_webapp_js_proto_files)',
+ ],
+ },
+ {
+ 'action_name': 'Verify remoting webapp with browsertests',
+ 'inputs': [
+ 'remoting_webapp_compile.gypi',
+ 'remoting_webapp_files.gypi',
+ '<@(remoting_webapp_crd_js_files)',
+ '<@(remoting_webapp_browsertest_all_js_files)',
+ '<@(remoting_webapp_browsertest_js_proto_files)',
+ ],
+ 'outputs': [
+ '<(success_stamp_bt)',
+ ],
+ 'action': [
+ 'python', '<(DEPTH)/third_party/closure_compiler/checker.py',
+ '--strict',
+ '--no-single-file',
+ '--success-stamp', '<(success_stamp_bt)',
+ '<@(remoting_webapp_crd_js_files)',
+ '<@(remoting_webapp_browsertest_all_js_files)',
+ '<@(remoting_webapp_browsertest_js_proto_files)',
+ ],
+ },
+ {
+ 'action_name': 'Verify remoting webapp unittests',
+ 'inputs': [
+ 'remoting_webapp_compile.gypi',
+ 'remoting_webapp_files.gypi',
+ '<@(remoting_webapp_crd_js_files)',
+ '<@(remoting_webapp_unittests_all_js_files)',
+ '<@(remoting_webapp_unittests_js_proto_files)',
+ ],
+ 'outputs': [
+ '<(success_stamp_ut)',
+ ],
+ 'action': [
+ 'python', '<(DEPTH)/third_party/closure_compiler/checker.py',
+ '--strict',
+ '--no-single-file',
+ '--success-stamp', '<(success_stamp_ut)',
+ '<@(remoting_webapp_crd_js_files)',
+ '<@(remoting_webapp_unittests_all_js_files)',
+ '<@(remoting_webapp_unittests_js_proto_files)',
+ ],
+ },
+ ],
+ 'includes': ['remoting_webapp_files.gypi'],
+}

Powered by Google App Engine
This is Rietveld 408576698