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

Side by Side Diff: remoting/remoting_webapp.gypi

Issue 927373005: [Chromoting] Enable jscompile for browser tests. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Review comments Created 5 years, 10 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 unified diff | Download patch
OLDNEW
1 # Copyright 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 # 4 #
5 # File included in remoting_webapp_* targets in remoting_client.gypi 5 # File included in remoting_webapp_* targets in remoting_client.gypi
6 6
7 { 7 {
8 'type': 'none', 8 'type': 'none',
9 'variables': { 9 'variables': {
10 'extra_files%': [], 10 'extra_files%': [],
11 'generated_html_files': [ 11 'generated_html_files': [
12 '<(SHARED_INTERMEDIATE_DIR)/main.html', 12 '<(SHARED_INTERMEDIATE_DIR)/main.html',
13 '<(SHARED_INTERMEDIATE_DIR)/wcs_sandbox.html', 13 '<(SHARED_INTERMEDIATE_DIR)/wcs_sandbox.html',
14 '<(SHARED_INTERMEDIATE_DIR)/background.html', 14 '<(SHARED_INTERMEDIATE_DIR)/background.html',
15 ], 15 ],
16 'dr_webapp_locales_listfile': '<(SHARED_INTERMEDIATE_DIR)/>(_target_name)_lo cales.txt', 16 'dr_webapp_locales_listfile': '<(SHARED_INTERMEDIATE_DIR)/>(_target_name)_lo cales.txt',
17 }, 17 },
18 'dependencies': [ 18 'dependencies': [
19 'remoting_resources', 19 'remoting_resources',
20 'remoting_webapp_html', 20 'remoting_webapp_html',
21 ], 21 ],
22 'conditions': [ 22 'conditions': [
23 ['run_jscompile != 0', { 23 ['run_jscompile != 0', {
24 'variables': { 24 'variables': {
25 'success_stamp': '<(PRODUCT_DIR)/<(_target_name)_jscompile.stamp', 25 'success_stamp': '<(PRODUCT_DIR)/<(_target_name)_jscompile.stamp',
26 'success_stamp_bt': '<(PRODUCT_DIR)/<(_target_name)_bt_jscompile.stamp',
26 }, 27 },
27 'actions': [ 28 'actions': [
28 { 29 {
29 'action_name': 'Verify remoting webapp', 30 'action_name': 'Verify remoting webapp',
30 'inputs': [ 31 'inputs': [
31 '<@(remoting_webapp_crd_js_files)', 32 '<@(remoting_webapp_crd_js_files)',
32 '<@(remoting_webapp_js_proto_files)', 33 '<@(remoting_webapp_js_proto_files)',
33 ], 34 ],
34 'outputs': [ 35 'outputs': [
35 '<(success_stamp)', 36 '<(success_stamp)',
36 ], 37 ],
37 'action': [ 38 'action': [
38 'python', '../third_party/closure_compiler/checker.py', 39 'python', '../third_party/closure_compiler/checker.py',
39 '--strict', 40 '--strict',
40 '--no-single-file', 41 '--no-single-file',
41 '--success-stamp', '<(success_stamp)', 42 '--success-stamp', '<(success_stamp)',
42 '<@(remoting_webapp_crd_js_files)', 43 '<@(remoting_webapp_crd_js_files)',
43 '<@(remoting_webapp_js_proto_files)', 44 '<@(remoting_webapp_js_proto_files)',
44 ], 45 ],
45 }, 46 },
47 {
48 'action_name': 'Verify remoting webapp with browsertests',
49 'inputs': [
50 '<@(remoting_webapp_crd_js_files)',
51 '<@(remoting_webapp_browsertest_js_all_files)',
52 ],
53 'outputs': [
54 '<(success_stamp_bt)',
55 ],
56 'action': [
57 'python', '../third_party/closure_compiler/checker.py',
58 '--strict',
59 '--no-single-file',
60 '--success-stamp', '<(success_stamp_bt)',
61 '<@(remoting_webapp_crd_js_files)',
62 '<@(remoting_webapp_browsertest_js_all_files)',
63 ],
64 },
46 ], # actions 65 ], # actions
47 }], 66 }],
48 ], 67 ],
49 'actions': [ 68 'actions': [
50 { 69 {
51 'action_name': 'Build Remoting locales listfile', 70 'action_name': 'Build Remoting locales listfile',
52 'inputs': [ 71 'inputs': [
53 '<(remoting_localize_path)', 72 '<(remoting_localize_path)',
54 ], 73 ],
55 'outputs': [ 74 'outputs': [
(...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after
91 '<(webapp_type)', 110 '<(webapp_type)',
92 '<@(generated_html_files)', 111 '<@(generated_html_files)',
93 '<@(remoting_webapp_crd_files)', 112 '<@(remoting_webapp_crd_files)',
94 '<@(extra_files)', 113 '<@(extra_files)',
95 '--locales_listfile', 114 '--locales_listfile',
96 '<(dr_webapp_locales_listfile)', 115 '<(dr_webapp_locales_listfile)',
97 ], 116 ],
98 }, 117 },
99 ], 118 ],
100 } 119 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698