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

Side by Side Diff: remoting/remoting_key_tester.gypi

Issue 1152583011: Refactor compile_js.gypi to support script_args and closure_args (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added remoting/compile_js.gypi and decoupled target_name from source_files Created 5 years, 6 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 2015 The Chromium Authors. All rights reserved. 1 # Copyright 2015 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 { 5 {
6 'includes': [ 6 'includes': [
7 '../build/common_untrusted.gypi', 7 '../build/common_untrusted.gypi',
8 ], 8 ],
9 9
10 'variables': { 10 'variables': {
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
42 ], 42 ],
43 }, # end of target 'remoting_key_tester' 43 }, # end of target 'remoting_key_tester'
44 44
45 { 45 {
46 'target_name': 'remoting_key_tester_jscompile', 46 'target_name': 'remoting_key_tester_jscompile',
47 'type': 'none', 47 'type': 'none',
48 'conditions': [ 48 'conditions': [
49 ['run_jscompile != 0', { 49 ['run_jscompile != 0', {
50 'variables': { 50 'variables': {
51 'success_stamp': '<(PRODUCT_DIR)/<(_target_name).stamp', 51 'success_stamp': '<(PRODUCT_DIR)/<(_target_name).stamp',
52 'source_files': [
53 '<@(remoting_key_tester_js_files)',
54 'webapp/js_proto/chrome_proto.js',
55 'webapp/js_proto/chrome_event_proto.js',
56 ],
57 'script_args' : [
Dan Beam 2015/06/01 22:57:39 'script_args': [
Theresa 2015/06/02 22:02:12 Done.
58 '--success-stamp', '<(success_stamp)',
59 '--out_file', '<(success_stamp)',
60 ]
52 }, 61 },
53 'actions': [ 62 'includes': [
54 { 63 'compile_js.gypi'
55 'action_name': 'jscompile remoting_key_tester', 64 ],
Dan Beam 2015/06/01 22:57:39 'includes': ['compile_js.gypi'],
Theresa 2015/06/02 22:02:12 Done.
56 'inputs': [
57 '<@(remoting_key_tester_js_files)',
58 'webapp/js_proto/chrome_proto.js',
59 'webapp/js_proto/chrome_event_proto.js',
60 ],
61 'outputs': [
62 '<(success_stamp)',
63 ],
64 'action': [
65 'python', '../third_party/closure_compiler/compile.py',
66 '--strict',
67 '--no-single-file',
68 '--success-stamp', '<(success_stamp)',
69 '<@(remoting_key_tester_js_files)',
70 'webapp/js_proto/chrome_proto.js',
71 'webapp/js_proto/chrome_event_proto.js',
72 ],
73 },
74 ], # actions
75 }], 65 }],
76 ], 66 ],
77 }, # end of target 'remoting_key_tester_jscompile' 67 }, # end of target 'remoting_key_tester_jscompile'
78 68
79 { 69 {
80 'target_name': 'remoting_key_tester_pexe', 70 'target_name': 'remoting_key_tester_pexe',
81 'type': 'none', 71 'type': 'none',
82 'sources': [ 72 'sources': [
83 'tools/javascript_key_tester/pnacl/remoting_key_tester.cc', 73 'tools/javascript_key_tester/pnacl/remoting_key_tester.cc',
84 ], 74 ],
85 'variables': { 75 'variables': {
86 'nexe_target': 'remoting_key_tester', 76 'nexe_target': 'remoting_key_tester',
87 'build_glibc': 0, 77 'build_glibc': 0,
88 'build_newlib': 0, 78 'build_newlib': 0,
89 'build_pnacl_newlib': 1, 79 'build_pnacl_newlib': 1,
90 'extra_deps_pnacl_newlib': [ 80 'extra_deps_pnacl_newlib': [
91 '>(tc_lib_dir_pnacl_newlib)/libppapi.a', 81 '>(tc_lib_dir_pnacl_newlib)/libppapi.a',
92 '>(tc_lib_dir_pnacl_newlib)/libppapi_cpp.a', 82 '>(tc_lib_dir_pnacl_newlib)/libppapi_cpp.a',
93 ], 83 ],
94 }, 84 },
95 'link_flags': [ 85 'link_flags': [
96 '-lppapi_stub', 86 '-lppapi_stub',
97 '-lppapi_cpp', 87 '-lppapi_cpp',
98 ], 88 ],
99 }, # end of target 'remoting_key_tester_pexe' 89 }, # end of target 'remoting_key_tester_pexe'
100 ], 90 ],
101 }] 91 }]
102 ], 92 ],
103 } 93 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698