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

Side by Side Diff: android_webview/android_webview.gyp

Issue 1048463003: Use jinja2 to template webview manifest files. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Simplify intermediate dir Created 5 years, 8 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 | « no previous file | android_webview/apk/java/AndroidManifest.xml » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2009 The Chromium Authors. All rights reserved. 1 # Copyright 2009 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 'variables': { 5 'variables': {
6 'chromium_code': 1, 6 'chromium_code': 1,
7 }, 7 },
8 'targets': [ 8 'targets': [
9 { 9 {
10 'target_name': 'android_webview_pak', 10 'target_name': 'android_webview_pak',
(...skipping 370 matching lines...) Expand 10 before | Expand all | Expand 10 after
381 'R_package': 'org.chromium.android_webview', 381 'R_package': 'org.chromium.android_webview',
382 'R_package_relpath': 'org/chromium/android_webview', 382 'R_package_relpath': 'org/chromium/android_webview',
383 }, 383 },
384 'includes': [ '../build/java.gypi' ], 384 'includes': [ '../build/java.gypi' ],
385 }, 385 },
386 { 386 {
387 'target_name': 'system_webview_apk', 387 'target_name': 'system_webview_apk',
388 'variables': { 388 'variables': {
389 'apk_name': 'SystemWebView', 389 'apk_name': 'SystemWebView',
390 'android_sdk_jar': '../third_party/android_platform/webview/framewor ks_5.1.0_r1.jar', 390 'android_sdk_jar': '../third_party/android_platform/webview/framewor ks_5.1.0_r1.jar',
391 'android_manifest_path': 'apk/java/AndroidManifest.xml', 391 'jinja_inputs': ['apk/java/AndroidManifest.xml'],
Torne 2015/03/30 13:31:09 Thinking about it more, all this logic should go i
Tobias Sargeant 2015/03/30 13:53:07 Done.
392 'jinja_output': '<(INTERMEDIATE_DIR)/AndroidManifest.xml',
393 'jinja_variables': [ ],
394 'android_manifest_path': '<(INTERMEDIATE_DIR)/AndroidManifest.xml',
392 'java_in_dir': 'glue/java', 395 'java_in_dir': 'glue/java',
393 'resource_dir': 'apk/java/res', 396 'resource_dir': 'apk/java/res',
394 }, 397 },
395 'includes': [ 'apk/system_webview_apk_common.gypi' ], 398 'includes': [
399 'apk/system_webview_apk_common.gypi',
400 '../build/android/jinja_template.gypi',
401 ],
396 }, 402 },
397 ], 403 ],
398 }, { # android_webview_build==1 404 }, { # android_webview_build==1
399 'targets': [ 405 'targets': [
400 { 406 {
401 'target_name': 'android_webview_jarjar_ui_resources', 407 'target_name': 'android_webview_jarjar_ui_resources',
402 'android_unmangled_name': 1, 408 'android_unmangled_name': 1,
403 'type': 'none', 409 'type': 'none',
404 'variables': { 410 'variables': {
405 'res_dir': '../ui/android/java/res', 411 'res_dir': '../ui/android/java/res',
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
437 'android_webview_jarjar_content_resources', 443 'android_webview_jarjar_content_resources',
438 'android_webview_jarjar_ui_resources', 444 'android_webview_jarjar_ui_resources',
439 'android_webview_jarjar_web_contents_delegate_android_resources', 445 'android_webview_jarjar_web_contents_delegate_android_resources',
440 'android_webview_strings_grd', 446 'android_webview_strings_grd',
441 ], 447 ],
442 }, 448 },
443 ], 449 ],
444 }], 450 }],
445 ], 451 ],
446 } 452 }
OLDNEW
« no previous file with comments | « no previous file | android_webview/apk/java/AndroidManifest.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698