diff options
| author | jshin@chromium.org <jshin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-01-22 19:18:43 +0000 |
|---|---|---|
| committer | jshin@chromium.org <jshin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-01-22 19:18:43 +0000 |
| commit | ba6dd29f30451b12566641bad7fa58ca7e3fe6bc (patch) | |
| tree | 9004e1b3df1e4e7d625fd2164aec02d6cd21af4e /net | |
| parent | fb4eb867ed64c283aeb66497528e210c7dadbdcb (diff) | |
| download | chromium_src-ba6dd29f30451b12566641bad7fa58ca7e3fe6bc.zip chromium_src-ba6dd29f30451b12566641bad7fa58ca7e3fe6bc.tar.gz chromium_src-ba6dd29f30451b12566641bad7fa58ca7e3fe6bc.tar.bz2 | |
Enable icu_use_data_file on Windows
Set icu_use_data_file_flag to 1 on Windows in common.gypi and make the dependency on icudata conditional on 'icu_use_data_file_flag != 0' in common.gypi, base.gyp and net.gyp
This also requires a change in third_party/icu/icu.gyp
( https://codereview.chromium.org/111723007/ ), which was rolled in by https://codereview.chromium.org/118313004/
BUG=72633
TEST=All windows builds work fine and there's no perf regression (start-up etc).
Review URL: https://codereview.chromium.org/99473012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@246387 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net')
| -rw-r--r-- | net/net.gyp | 27 |
1 files changed, 18 insertions, 9 deletions
diff --git a/net/net.gyp b/net/net.gyp index 9b10a414..a40f393 100644 --- a/net/net.gyp +++ b/net/net.gyp @@ -2235,14 +2235,20 @@ 'dns/dns_config_service_posix_unittest.cc', 'http/http_auth_gssapi_posix_unittest.cc', ], - # This is needed to trigger the dll copy step on windows. - # TODO(mark): Specifying this here shouldn't be necessary. 'dependencies': [ - '../third_party/icu/icu.gyp:icudata', '../third_party/nss/nss.gyp:nspr', '../third_party/nss/nss.gyp:nss', 'third_party/nss/ssl.gyp:libssl', ], + 'conditions': [ + [ 'icu_use_data_file_flag == 0', { + # This is needed to trigger the dll copy step on windows. + # TODO(mark): Specifying this here shouldn't be necessary. + 'dependencies': [ + '../third_party/icu/icu.gyp:icudata', + ], + }], + ], # TODO(jschuh): crbug.com/167187 fix size_t to int truncations. 'msvs_disabled_warnings': [4267, ], }, @@ -2341,16 +2347,19 @@ ], }, ], - # This is needed to trigger the dll copy step on windows. - # TODO(mark): Specifying this here shouldn't be necessary. [ 'OS == "win"', { - 'dependencies': [ - '../third_party/icu/icu.gyp:icudata', + 'conditions': [ + [ 'icu_use_data_file_flag == 0', { + # This is needed to trigger the dll copy step on windows. + # TODO(mark): Specifying this here shouldn't be necessary. + 'dependencies': [ + '../third_party/icu/icu.gyp:icudata', + ], + }], ], # TODO(jschuh): crbug.com/167187 fix size_t to int truncations. 'msvs_disabled_warnings': [4267, ], - }, - ], + }], ], }, { |