Add support for icu_use_data_file=1 to icu.gyp on Windows

icu.gyp

Index: icu.gyp
===================================================================
--- icu.gyp	(revision 228516)
+++ icu.gyp	(working copy)
@@ -71,7 +71,7 @@
             }, {
               'toolsets': ['host', 'target'],
             }],
-            [ 'OS == "win"', {
+            [ 'OS == "win" and icu_use_data_file_flag==0', {
               'type': 'none',
               'copies': [
                 {
@@ -82,28 +82,27 @@
                 },
               ],
             }],
-            [ 'OS != "win" and icu_use_data_file_flag', {
+            [ 'icu_use_data_file_flag', {
               # Remove any assembly data file.
               'sources/': [['exclude', 'icudt46l_dat']],
               # Compile in the stub data symbol.
               'sources': ['source/stubdata/stubdata.c'],
+             
               # Make sure any binary depending on this gets the data file.
-              'link_settings': {
-                'target_conditions': [
-                  ['(OS == "mac" and _mac_bundle) or OS=="ios"', {
-                    'mac_bundle_resources': [
-                      'source/data/in/icudt46l.dat',
+              'conditions': [
+                ['OS=="ios"', {
+                  'mac_bundle_resources': [

[Mark Mentovai, 2013/12/18 21:58:28]

This is a static_library, but a static_library can’t be a mac_bundle, so this is
meaningless. The file won’t be copied anywhere.

[jungshik at Google, 2013/12/18 22:06:13]

Hmm.. Then, I wonder how iOS could have worked so far? iOS has had
icu_use_data_file_flag set to 1 for a long while (before I did anything).  In
this CL, I removed link_settings and replaced 'target_conditions' with
'conditions'. 

+                    'source/data/in/icudtl.dat',
+                  ],
+                }, {

[jungshik at Google, 2013/12/18 21:33:36]

Trybot failed for a CL at https://codereview.chromium.org/118313004/ that rolls
icu to this version. It's ios_debug_simulator. base_unittests or net_unittests
failed to find icudtl.dat.  See
http://build.chromium.org/p/tryserver.chromium/builders/ios_dbg_simulator/bui...

I wonder which of two changes, removing link_settings and replacing
target_conditions with conditions is to blame for the failure. 

 

+                  'copies': [{
+                    'destination': '<(PRODUCT_DIR)',
+                    'files': [
+                      'source/data/in/icudtl.dat',
                     ],
-                  }, {
-                    'copies': [{
-                      'destination': '<(PRODUCT_DIR)',
-                      'files': [
-                        'source/data/in/icudt46l.dat',
-                      ],
-                    }],
                   }],
-                ],  # target_conditions
-              },  # link_settings
+                }],
+              ],  # target_conditions
             }],
           ],
           'target_conditions': [