Turn on UCONFIG_NO_NON_HTML5_CONVERTER to save 100kB

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,
     'icu_use_data_file_flag%': 0,
     'want_separate_host_toolset%': 1,
   },
   'target_defaults': {
     'direct_dependent_settings': {
       'defines': [
         # Tell ICU to not insert |using namespace icu;| into its headers,
         # so that chrome's source explicitly has to use |icu::|.
         'U_USING_ICU_NAMESPACE=0',
         # We don't use ICU plugins and dyload is only necessary for them.
         # NaCl-related builds also fail looking for dlfcn.h when it's enabled.
         'U_ENABLE_DYLOAD=0',
+        # Only build encoding coverters and detectors necessary for HTML5.
+        'UCONFIG_NO_NON_HTML5_CONVERSION=1',

[Mark Mentovai, 2014/09/23 21:59:03]

Is this needed?

It’s only needed if UCONFIG_NO_NON_HTML5_CONVERSION is visible to headers that
are #included from outside of any ICU targets. Is it?

[jungshik at Google, 2014/09/24 20:56:48]

That's what I guessed, but was not sure about direct_dep..settings. 

Anyway, no public header depends on UCONFIG_NO...*. So, I will remove that line.
Thanks. 

[jungshik at Google, 2014/09/24 21:06:11]

To be precise, public headers (in source/{common,i18n}/unicode) do depend on
uconfig.h that sets the default value for UCONFIG_NO..*, but
UCONFIG_NO_NON_HTML5_CONVERSION is used only *internally*. So, no need to set it
in 'direct_dependent_settings'.  

       ],
     },
     'defines': [
       'U_USING_ICU_NAMESPACE=0',
       'HAVE_DLOPEN=0',
+      'UCONFIG_NO_NON_HTML5_CONVERSION=1',
     ],
     'conditions': [
       ['component=="static_library"', {
         'defines': [
           'U_STATIC_IMPLEMENTATION',
         ],
       }],
       ['(OS=="linux" or OS=="freebsd" or OS=="openbsd" or OS=="solaris" \
          or OS=="netbsd" or OS=="mac" or OS=="android" or OS=="qnx") and \
         (target_arch=="arm" or target_arch=="ia32" or \
@@ skipping 583 matching lines @@
           },
           'includes': [
             '../../build/shim_headers.gypi',
           ],
           'toolsets': ['target'],
         },
       ], # targets
     }],
   ], # conditions
 }