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

Unified Diff: url/url.gyp

Issue 257673002: Make it possible to build url/ without ICU on android. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Missed a spot Created 6 years, 7 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
« no previous file with comments | « url/android/url_jni_registrar.cc ('k') | url/url_canon_icu_alternatives_android.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: url/url.gyp
diff --git a/url/url.gyp b/url/url.gyp
index 93bbee2e17135802680bb9245edf6ec070983f0e..d1c3bf1e3d6018720d07ee52dd3d759b9f9d8c9d 100644
--- a/url/url.gyp
+++ b/url/url.gyp
@@ -33,6 +33,28 @@
'defines': [
'URL_IMPLEMENTATION',
],
+ 'conditions': [
+ ['use_icu_alternatives_on_android==1', {
+ 'sources!': [
+ 'url_canon_icu.cc',
+ 'url_canon_icu.h',
+ ],
+ 'dependencies!': [
+ '../third_party/icu/icu.gyp:icui18n',
+ '../third_party/icu/icu.gyp:icuuc',
+ ],
+ }],
+ ['use_icu_alternatives_on_android==1 and OS=="android"', {
+ 'dependencies': [
+ 'url_java',
+ 'url_jni_headers',
+ ],
+ 'sources': [
+ 'url_canon_icu_alternatives_android.cc',
+ 'url_canon_icu_alternatives_android.h',
+ ],
+ }],
+ ],
# TODO(jschuh): crbug.com/167187 fix size_t to int truncations.
'msvs_disabled_warnings': [4267, ],
},
@@ -40,7 +62,6 @@
'target_name': 'url_unittests',
'type': 'executable',
'dependencies': [
- '../base/base.gyp:base_i18n',
'../base/base.gyp:run_all_unittests',
'../testing/gtest.gyp:gtest',
'../third_party/icu/icu.gyp:icuuc',
@@ -49,6 +70,7 @@
'sources': [
'gurl_unittest.cc',
'origin_unittest.cc',
+ 'url_canon_icu_unittest.cc',
'url_canon_unittest.cc',
'url_parse_unittest.cc',
'url_test_utils.h',
@@ -63,9 +85,47 @@
],
}
],
+ ['use_icu_alternatives_on_android==1',
+ {
+ 'sources!': [
+ 'url_canon_icu_unittest.cc',
+ ],
+ 'dependencies!': [
+ '../third_party/icu/icu.gyp:icuuc',
+ ],
+ }
+ ],
],
# TODO(jschuh): crbug.com/167187 fix size_t to int truncations.
'msvs_disabled_warnings': [4267, ],
},
],
+ 'conditions': [
+ ['use_icu_alternatives_on_android==1 and OS=="android"', {
+ 'targets': [
+ {
+ 'target_name': 'url_jni_headers',
+ 'type': 'none',
+ 'sources': [
+ 'android/java/src/org/chromium/url/IDNStringUtil.java'
+ ],
+ 'variables': {
+ 'jni_gen_package': 'url',
+ },
+ 'includes': [ '../build/jni_generator.gypi' ],
+ },
+ {
+ 'target_name': 'url_java',
+ 'type': 'none',
+ 'variables': {
+ 'java_in_dir': '../url/android/java',
+ },
+ 'dependencies': [
+ '../base/base.gyp:base',
+ ],
+ 'includes': [ '../build/java.gypi' ],
+ },
+ ],
+ }],
+ ],
}
« no previous file with comments | « url/android/url_jni_registrar.cc ('k') | url/url_canon_icu_alternatives_android.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698