Use ICU case conversion/transliterator for case conversion behind a flag

tools/gyp/v8.gyp

Index: tools/gyp/v8.gyp
diff --git a/tools/gyp/v8.gyp b/tools/gyp/v8.gyp
index 837daa343249a707e0a5db1c6095570ab52ad3a0..9122caa1d3a2f0c7cd859dc6abbc061305088d37 100644
--- a/tools/gyp/v8.gyp
+++ b/tools/gyp/v8.gyp
@@ -1971,17 +1971,6 @@
         }, {

[jochen (gone - plz use gerrit), 2016/04/29 06:58:09]

this file doesn't exist anymore, it's now in src/

[jungshik at Google, 2016/04/29 18:03:23]

rebased. Thanks !

           'toolsets': ['target'],
         }],
-        ['v8_enable_i18n_support==1', {
-          'variables': {
-            'i18n_library_files': [
-              '../../src/js/i18n.js',
-            ],
-          },
-        }, {
-          'variables': {
-            'i18n_library_files': [],
-          },
-        }],
       ],
       'variables': {
         'library_files': [
@@ -2031,6 +2020,12 @@
         'libraries_experimental_bin_file': '<(SHARED_INTERMEDIATE_DIR)/libraries-experimental.bin',
         'libraries_extras_bin_file': '<(SHARED_INTERMEDIATE_DIR)/libraries-extras.bin',
         'libraries_experimental_extras_bin_file': '<(SHARED_INTERMEDIATE_DIR)/libraries-experimental-extras.bin',
+        'conditions': [
+          ['v8_enable_i18n_support==1', {
+            'library_files': ['../../src/js/i18n.js'],
+            'experimental_library_files': ['../../src/js/icu-case-mapping.js'],

[jungshik at Google, 2016/04/29 06:23:17]

For some reason, icu-case-mapping.js is never added to the list of js files to
be compiled by 'js2c' for
experimental-libraries.cc and libraries-experimental.bin. 

When I manually built those two files ( cc and bin) using the following commands
and d8 after that, d8 worked as expected. Without --icu_case_mapping, I got the
current
behavior and with --icu_case_mapping, I got a new correct behaviro for
to{Lower,Upper}Case. 

python ../tools/js2c.py ../out/Release/gen/experimental-libraries.cc
EXPERIMENTAL js/macros.py messages.h js/harmony-atomics.js
js/harmony-regexp-exec.js js/harmony-sharedarraybuffer.js js/harmony-simd.js
js/harmony-species.js js/harmony-unicode-regexps.js js/harmony-string-padding.js
js/promise-extra.js js/icu-case-mapping.js 

python ../tools/js2c.py ../out/Release/gen/experimental-libraries.cc
EXPERIMENTAL js/macros.py messages.h js/harmony-atomics.js
js/harmony-regexp-exec.js js/harmony-sharedarraybuffer.js js/harmony-simd.js
js/harmony-species.js js/harmony-unicode-regexps.js js/harmony-string-padding.js
js/promise-extra.js js/icu-case-mapping.js --startup_blob
../out/Release/gen/libraries-experimental.bin --nojs

I also wanted to try GN build, but it turned out that GN does not yet support a
standalone v8 build. 

[jochen (gone - plz use gerrit), 2016/04/29 06:58:09]

gyp is a bit peculiar here. I'd recommend to keep  i18n_library_files and add
i18n_experimental_library_files

+          }],
+        ],
       },
       'actions': [
         {
@@ -2038,7 +2033,6 @@
           'inputs': [
             '../../tools/js2c.py',
             '<@(library_files)',
-            '<@(i18n_library_files)'
           ],
           'outputs': ['<(SHARED_INTERMEDIATE_DIR)/libraries.cc'],
           'action': [
@@ -2047,7 +2041,6 @@
             '<(SHARED_INTERMEDIATE_DIR)/libraries.cc',
             'CORE',
             '<@(library_files)',
-            '<@(i18n_library_files)'
           ],
         },
         {
@@ -2055,7 +2048,6 @@
           'inputs': [
             '../../tools/js2c.py',
             '<@(library_files)',
-            '<@(i18n_library_files)'
           ],
           'outputs': ['<@(libraries_bin_file)'],
           'action': [
@@ -2064,7 +2056,6 @@
             '<(SHARED_INTERMEDIATE_DIR)/libraries.cc',
             'CORE',
             '<@(library_files)',
-            '<@(i18n_library_files)',
             '--startup_blob', '<@(libraries_bin_file)',
             '--nojs',
           ],