Only add stubdata.c once on Windows (when icu_use_data_file_flag is set)

icu.gyp

 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 {
   'includes': [
     'icu.gypi',
   ],
   'variables': {
     'use_system_icu%': 0,
@@ skipping 308 matching lines @@
           'conditions': [
             [ 'use_system_icu==1 and want_separate_host_toolset==1', {
               'toolsets': ['host'],
             }],
             [ 'use_system_icu==0 and want_separate_host_toolset==1', {
               'toolsets': ['host', 'target'],
             }],
             [ 'use_system_icu==0 and want_separate_host_toolset==0', {
               'toolsets': ['target'],
             }],
-            [ 'OS == "win"', {
+            [ 'OS == "win" and icu_use_data_file_flag==0', {

[scottmg, 2014/08/19 04:02:30]

it's added in icudata at line 138 when icu_use_data_file_flag==1, so on windows,
it's added twice, which causes a link error when linking incrementally. i'm not
sure if it's important which target it's in?

               'sources': [
                 'source/stubdata/stubdata.c',
               ],
             }],
             ['OS == "android" and use_system_stlport == 1', {
               'target_conditions': [
                 ['_toolset == "target"', {
                   # ICU requires RTTI, which is not present in the system's
                   # stlport, so we have to include gabi++.
                   'include_dirs': [
@@ skipping 255 matching lines @@
           },
           'includes': [
             '../../build/shim_headers.gypi',
           ],
           'toolsets': ['target'],
         },
       ], # targets
     }],
   ], # conditions
 }