Fix compilation errors on Android and Windows

README.chromium

 Name: icu
 URL: http://site.icu-project.org/
 Version: 56.1
 License: MIT
 Security Critical: yes
 
 ***NOTE***
 ICU is in the middle of being updated to 56.1 and does not work, yet.
 If you have an urgent fix to apply, contact jshin@chromium.org to
 create a branch for 54.1 to apply a fix on top of.
@@ skipping 247 matching lines @@
   http://source.icu-project.org/repos/icu/data/trunk/tzdata/icunew/2015g/44/
 
 5. Build-related changes
 
   - patches/wpo.patch
     upstream bugs : http://bugs.icu-project.org/trac/ticket/8043
                     http://bugs.icu-project.org/trac/ticket/5701
   - patches/vscomp.patch for building with Visual Studio on Windows.
      a. do not use WINDOWS_LOCALE_API in locmap.c
      b. do not redefine stringpiece::npos
+     c. fix http://bugs.icu-project.org/trac/ticket/12129 (C4138 warning)
 
   - patches/data.build.patch :
       Remove unnecessary resources : unames, collator rule source
   - patches/data.build.win.patch :
       Windows-only data build patch.
   - patches/data_symb.patch :
       Put ICU_DATA_ENTRY_POINT(icudtXX_dat) in common when we use
       the icu data file or icudt.dll
 
 6. Apply a timezone detection API fix
   - patches/tzdetect.patch
   - upstream bugs
     http://bugs.icu-project.org/trac/ticket/11623
 
 7. Fix 'bad cast' found in Transliterator with a cfi build
   - patches/xlit_badcast.patch
   - upstream bug (yet to be resolved)
     http://bugs.icu-project.org/trac/ticket/11937
 
 8. TODO: If removing UTF-32 from Blink is more involved than expected,
    add back UTF-32 temporarily even when UCONFIG_ONLY_HTML_CONVERSION is
    defined See
     http://www.icu-project.org/trac/ticket/11296