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

Side by Side Diff: chrome/android/chrome_apk.gyp

Issue 1161833003: Make chrome_public_apk, not chrome_staging_java depend on chrome_template_resources (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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
« no previous file with comments | « chrome/android/BUILD.gn ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 'includes': [ 5 'includes': [
6 '../../chrome/chrome_android_paks.gypi', # Included for the list of pak reso urces. 6 '../../chrome/chrome_android_paks.gypi', # Included for the list of pak reso urces.
7 '../../build/util/version.gypi' 7 '../../build/util/version.gypi'
8 ], 8 ],
9 'variables': { 9 'variables': {
10 'chromium_code': 1, 10 'chromium_code': 1,
11 'package_name': 'chrome_public_apk', 11 'package_name': 'chrome_public_apk',
12 'manifest_package%': 'org.chromium.chrome', 12 'manifest_package': 'org.chromium.chrome',
13 'chrome_public_apk_manifest': '<(SHARED_INTERMEDIATE_DIR)/chrome_public_apk_ manifest/AndroidManifest.xml', 13 'chrome_public_apk_manifest': '<(SHARED_INTERMEDIATE_DIR)/chrome_public_apk_ manifest/AndroidManifest.xml',
14 'chrome_public_test_apk_manifest': '<(SHARED_INTERMEDIATE_DIR)/chrome_public _test_apk_manifest/AndroidManifest.xml', 14 'chrome_public_test_apk_manifest': '<(SHARED_INTERMEDIATE_DIR)/chrome_public _test_apk_manifest/AndroidManifest.xml',
15 'never_lint': 1, 15 'never_lint': 1,
16 'chrome_java_dir': 'java_staging', 16 'chrome_java_dir': 'java_staging',
17 'chrome_java_tests_dir': 'javatests', 17 'chrome_java_tests_dir': 'javatests',
18 'chrome_java_test_support_dir': '../test/android/javatests_staging', 18 'chrome_java_test_support_dir': '../test/android/javatests_staging',
19 'chrome_native_sources_dir': '../browser/android/', 19 'chrome_native_sources_dir': '../browser/android/',
20 # This list is shared with GN. 20 # This list is shared with GN.
21 'chrome_staging_jni_files': [ 21 'chrome_staging_jni_files': [
22 '<(chrome_java_dir)/src/org/chromium/chrome/browser/bookmark/EditBookmarkH elper.java', 22 '<(chrome_java_dir)/src/org/chromium/chrome/browser/bookmark/EditBookmarkH elper.java',
(...skipping 110 matching lines...) Expand 10 before | Expand all | Expand 10 after
133 'variables': { 133 'variables': {
134 'java_in_dir': '<(chrome_java_dir)', 134 'java_in_dir': '<(chrome_java_dir)',
135 'R_package': 'com.google.android.apps.chrome', 135 'R_package': 'com.google.android.apps.chrome',
136 'R_package_relpath': 'com/google/android/apps/chrome', 136 'R_package_relpath': 'com/google/android/apps/chrome',
137 'has_java_resources': 1, 137 'has_java_resources': 1,
138 'res_channel_dir': '<(chrome_java_dir)/res_default', 138 'res_channel_dir': '<(chrome_java_dir)/res_default',
139 'res_extra_dirs': ['<(res_channel_dir)'], 139 'res_extra_dirs': ['<(res_channel_dir)'],
140 'res_extra_files': ['<!@(find <(res_channel_dir) -type f)'], 140 'res_extra_files': ['<!@(find <(res_channel_dir) -type f)'],
141 }, 141 },
142 'dependencies': [ 142 'dependencies': [
143 'chrome_template_resources',
144 'custom_tabs_service_aidl', 143 'custom_tabs_service_aidl',
145 '<(DEPTH)/base/base.gyp:base_java', 144 '<(DEPTH)/base/base.gyp:base_java',
146 '<(DEPTH)/chrome/chrome.gyp:chrome_java', 145 '<(DEPTH)/chrome/chrome.gyp:chrome_java',
147 '<(DEPTH)/chrome/chrome.gyp:document_tab_model_info_proto_java', 146 '<(DEPTH)/chrome/chrome.gyp:document_tab_model_info_proto_java',
148 '<(DEPTH)/components/components.gyp:app_restrictions_resources', 147 '<(DEPTH)/components/components.gyp:app_restrictions_resources',
149 '<(DEPTH)/components/components.gyp:navigation_interception_java', 148 '<(DEPTH)/components/components.gyp:navigation_interception_java',
150 '<(DEPTH)/components/components.gyp:service_tab_launcher', 149 '<(DEPTH)/components/components.gyp:service_tab_launcher',
151 '<(DEPTH)/components/components.gyp:web_contents_delegate_android_java', 150 '<(DEPTH)/components/components.gyp:web_contents_delegate_android_java',
152 '<(DEPTH)/content/content.gyp:content_java', 151 '<(DEPTH)/content/content.gyp:content_java',
153 '<(DEPTH)/media/media.gyp:media_java', 152 '<(DEPTH)/media/media.gyp:media_java',
(...skipping 30 matching lines...) Expand all
184 '<(DEPTH)/chrome/chrome.gyp:chrome_java', 183 '<(DEPTH)/chrome/chrome.gyp:chrome_java',
185 '<(DEPTH)/chrome/chrome.gyp:chrome_java_test_support', 184 '<(DEPTH)/chrome/chrome.gyp:chrome_java_test_support',
186 '<(DEPTH)/content/content_shell_and_tests.gyp:content_java_test_support' , 185 '<(DEPTH)/content/content_shell_and_tests.gyp:content_java_test_support' ,
187 '<(DEPTH)/net/net.gyp:net_java', 186 '<(DEPTH)/net/net.gyp:net_java',
188 '<(DEPTH)/net/net.gyp:net_java_test_support', 187 '<(DEPTH)/net/net.gyp:net_java_test_support',
189 '<(DEPTH)/sync/sync.gyp:sync_java_test_support', 188 '<(DEPTH)/sync/sync.gyp:sync_java_test_support',
190 ], 189 ],
191 'includes': [ '../../build/java.gypi' ], 190 'includes': [ '../../build/java.gypi' ],
192 }, 191 },
193 { 192 {
194 # GN: //chrome/android:chrome_template_resources 193 # GN: //chrome/android:custom_tabs_service_aidl
195 'target_name': 'chrome_template_resources', 194 'target_name': 'custom_tabs_service_aidl',
195 'type': 'none',
196 'variables': {
197 'aidl_interface_file': '<(chrome_java_dir)/src/org/chromium/chrome/brows er/customtabs/common.aidl',
198 'aidl_import_include': '<(chrome_java_dir)/src/org/chromium/chrome/brows er/customtabs',
199 },
200 'sources': [
201 '<(chrome_java_dir)/src/org/chromium/chrome/browser/customtabs/IBrowserC onnectionCallback.aidl',
202 '<(chrome_java_dir)/src/org/chromium/chrome/browser/customtabs/IBrowserC onnectionService.aidl',
203 ],
204 'includes': [ '../../build/java_aidl.gypi' ],
205 },
206 {
207 # GN: //chrome/android:chrome_public_template_resources
208 'target_name': 'chrome_public_template_resources',
196 'type': 'none', 209 'type': 'none',
197 'variables': { 210 'variables': {
198 'jinja_inputs_base_dir': '<(chrome_java_dir)/res_template', 211 'jinja_inputs_base_dir': '<(chrome_java_dir)/res_template',
199 'jinja_inputs': [ 212 'jinja_inputs': [
200 '<(jinja_inputs_base_dir)/xml/searchable.xml', 213 '<(jinja_inputs_base_dir)/xml/searchable.xml',
201 '<(jinja_inputs_base_dir)/xml/syncadapter.xml', 214 '<(jinja_inputs_base_dir)/xml/syncadapter.xml',
202 ], 215 ],
203 'jinja_outputs_zip': '<(PRODUCT_DIR)/res.java/<(_target_name).zip', 216 'jinja_outputs_zip': '<(PRODUCT_DIR)/res.java/<(_target_name).zip',
204 'jinja_variables': [ 217 'jinja_variables': [
205 'manifest_package=<(manifest_package)', 218 'manifest_package=<(manifest_package)',
206 ], 219 ],
207 }, 220 },
208 'all_dependent_settings': { 221 'all_dependent_settings': {
209 'variables': { 222 'variables': {
210 'additional_input_paths': ['<(jinja_outputs_zip)'], 223 'additional_input_paths': ['<(jinja_outputs_zip)'],
211 'dependencies_res_zip_paths': ['<(jinja_outputs_zip)'], 224 'dependencies_res_zip_paths': ['<(jinja_outputs_zip)'],
212 }, 225 },
213 }, 226 },
214 'includes': [ '../../build/android/jinja_template.gypi' ], 227 'includes': [ '../../build/android/jinja_template.gypi' ],
215 }, 228 },
216 { 229 {
217 # GN: //chrome/android:custom_tabs_service_aidl
218 'target_name': 'custom_tabs_service_aidl',
219 'type': 'none',
220 'variables': {
221 'aidl_interface_file': '<(chrome_java_dir)/src/org/chromium/chrome/brows er/customtabs/common.aidl',
222 'aidl_import_include': '<(chrome_java_dir)/src/org/chromium/chrome/brows er/customtabs',
223 },
224 'sources': [
225 '<(chrome_java_dir)/src/org/chromium/chrome/browser/customtabs/IBrowserC onnectionCallback.aidl',
226 '<(chrome_java_dir)/src/org/chromium/chrome/browser/customtabs/IBrowserC onnectionService.aidl',
227 ],
228 'includes': [ '../../build/java_aidl.gypi' ],
229 },
230 {
231 # GN: //chrome/android:chrome_public 230 # GN: //chrome/android:chrome_public
232 'target_name': 'libchrome_public', 231 'target_name': 'libchrome_public',
233 'type': 'shared_library', 232 'type': 'shared_library',
234 'dependencies': [ 233 'dependencies': [
235 'libchrome_staging', 234 'libchrome_staging',
236 '<(DEPTH)/chrome/chrome.gyp:chrome_android_core', 235 '<(DEPTH)/chrome/chrome.gyp:chrome_android_core',
237 ], 236 ],
238 'include_dirs': [ 237 'include_dirs': [
239 '<(DEPTH)', 238 '<(DEPTH)',
240 ], 239 ],
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
300 # until the number of 32 bit devices which don't support this 299 # until the number of 32 bit devices which don't support this
301 # approach falls to a minimal level - http://crbug.com/390618. 300 # approach falls to a minimal level - http://crbug.com/390618.
302 ['component != "shared_library" and profiling==0 and (target_arch == " arm64" or target_arch == "x86_64")', { 301 ['component != "shared_library" and profiling==0 and (target_arch == " arm64" or target_arch == "x86_64")', {
303 'load_library_from_zip_file': '<(chrome_apk_load_library_from_zip)', 302 'load_library_from_zip_file': '<(chrome_apk_load_library_from_zip)',
304 'load_library_from_zip': '<(chrome_apk_load_library_from_zip)', 303 'load_library_from_zip': '<(chrome_apk_load_library_from_zip)',
305 }], 304 }],
306 ], 305 ],
307 }, 306 },
308 'dependencies': [ 307 'dependencies': [
309 'chrome_android_paks_copy', 308 'chrome_android_paks_copy',
309 'chrome_public_template_resources',
310 'chrome_staging_java', 310 'chrome_staging_java',
311 ], 311 ],
312 'includes': [ 'chrome_apk.gypi' ], 312 'includes': [ 'chrome_apk.gypi' ],
313 }, 313 },
314 { 314 {
315 # GN: N/A 315 # GN: N/A
316 # chrome_public_apk creates a .jar as a side effect. Any java targets 316 # chrome_public_apk creates a .jar as a side effect. Any java targets
317 # that need that .jar in their classpath should depend on this target, 317 # that need that .jar in their classpath should depend on this target,
318 'target_name': 'chrome_public_apk_java', 318 'target_name': 'chrome_public_apk_java',
319 'type': 'none', 319 'type': 'none',
(...skipping 89 matching lines...) Expand 10 before | Expand all | Expand 10 after
409 ], 409 ],
410 }, 410 },
411 ], 411 ],
412 } 412 }
413 413
414 # Local Variables: 414 # Local Variables:
415 # tab-width:2 415 # tab-width:2
416 # indent-tabs-mode:nil 416 # indent-tabs-mode:nil
417 # End: 417 # End:
418 # vim: set expandtab tabstop=2 shiftwidth=2: 418 # vim: set expandtab tabstop=2 shiftwidth=2:
OLDNEW
« no previous file with comments | « chrome/android/BUILD.gn ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698