Add back UTF-32 for now and fix a MSVC build issue

Add back UTF-32 for now and fix a MSVC build issue

See http://crbug.com/417850 for UTF-32
See http://bugs.icu-project.org/trac/ticket/12147 for C4275 warning
from MSVC.

BUG=575007
TEST=Layout tests fast/encoding/*
TEST=MSVC does not emit C4275 warning any more
R=jsbell@chromium.org

Committed: https://chromium.googlesource.com/chromium/deps/icu/+/ce2d2fd0d3d3bada940bf6d592172ea3a61a4a82

[jungshik at Google, 2016-02-04 18:05:48]

Description was changed from

==========
Add back UTF-32 for now and fix a MSVC build issue

See http://bugs.icu-project.org/trac/ticket/11893 for UTF-32
See http://bugs.icu-project.org/trac/ticket/11893 for C4275 warning
from MSVC.

BUG=575007
TEST=Layout tests fast/encoding/*
TEST=MSVC does not emit C4275 warning any more
==========

to

==========
Add back UTF-32 for now and fix a MSVC build issue

See http://crbug.com/417850 for UTF-32
See http://bugs.icu-project.org/trac/ticket/11893 for C4275 warning
from MSVC.

BUG=575007
TEST=Layout tests fast/encoding/*
TEST=MSVC does not emit C4275 warning any more
==========

[jungshik at Google, 2016-02-04 19:57:50]

Description was changed from

==========
Add back UTF-32 for now and fix a MSVC build issue

See http://crbug.com/417850 for UTF-32
See http://bugs.icu-project.org/trac/ticket/11893 for C4275 warning
from MSVC.

BUG=575007
TEST=Layout tests fast/encoding/*
TEST=MSVC does not emit C4275 warning any more
==========

to

==========
Add back UTF-32 for now and fix a MSVC build issue

See http://crbug.com/417850 for UTF-32
See http://bugs.icu-project.org/trac/ticket/12147 for C4275 warning
from MSVC.

BUG=575007
TEST=Layout tests fast/encoding/*
TEST=MSVC does not emit C4275 warning any more
==========

[jungshik at Google, 2016-02-04 20:02:13]

jshin@chromium.org changed reviewers:
+ jsbell@chromium.org

[jungshik at Google, 2016-02-04 20:02:13]

Joshua, PTAL. I'm turning back on UTF-32 in ICU for now until I have a 
resolution of http://crbug.com/417850 (a bit of clarification might be needed
from the spec as to what to do when a real UTF-32 document is encountered).

[jsbell, 2016-02-04 20:10:48]

lgtm

[jungshik at Google, 2016-02-04 21:22:52]

Description was changed from

==========
Add back UTF-32 for now and fix a MSVC build issue

See http://crbug.com/417850 for UTF-32
See http://bugs.icu-project.org/trac/ticket/12147 for C4275 warning
from MSVC.

BUG=575007
TEST=Layout tests fast/encoding/*
TEST=MSVC does not emit C4275 warning any more
==========

to

==========
Add back UTF-32 for now and fix a MSVC build issue

See http://crbug.com/417850 for UTF-32
See http://bugs.icu-project.org/trac/ticket/12147 for C4275 warning
from MSVC.

BUG=575007
TEST=Layout tests fast/encoding/*
TEST=MSVC does not emit C4275 warning any more
R=jsbell@chromium.org

Committed:
https://chromium.googlesource.com/chromium/deps/icu/+/ce2d2fd0d3d3bada940bf6d...
==========

[jungshik at Google, 2016-02-04 21:22:54]

Committed patchset #1 (id:1) manually as
ce2d2fd0d3d3bada940bf6d592172ea3a61a4a82 (presubmit successful).