Remove the ENABLE_JAVASCRIPT_I18N_API compile time flag.

Remove the ENABLE_JAVASCRIPT_I18N_API compile time flag.

All platforms ship this. We'll continue to have the run-time flag because
of the --disable-javascript-i18n-api command line flag.

BUG=

[eseidel, 2013-04-09 07:23:50]

What does this even do?  :)  Can we remove the --disable?  Is this still an
"experimental" v8 feature?

[jochen (gone - plz use gerrit), 2013-04-09 09:39:17]

+danno

I think we can even drop the runtime flag

[jochen (gone - plz use gerrit), 2013-04-11 14:23:43]

+cira

Do you know whether the command line flag to disable the v8-i18n extension is
used, or whether we can just delete it?

[Nebojša Ćirić, 2013-04-15 17:54:23]

Sorry didn't see it before.

I don't think we need the flag anymore. The feature has been launched. The only
reason would be if some implementation decides not to use it, for say size
reasons (you need ICU to enable this). But Chrome already bundles ICU in the
build so it may be a moot point.
As far as I know even Chrome on Android has this enabled.


On 2013/04/11 14:23:43, jochen wrote:
> +cira
> 
> Do you know whether the command line flag to disable the v8-i18n extension is
> used, or whether we can just delete it?

[Peter Beverloo, 2013-04-15 21:11:56]

Correct, Chromium for Android ships this as well.  I'll remove the command line
flag, and then update this patch to remove any related plumbing as well.  Thank
you!

[abarth-chromium, 2013-04-15 23:56:51]

LGTM

We can also land this patch (perhaps with the default set to "true"), then
remove the command-line switch, then remove the runtime plumbing.

[Peter Beverloo, 2013-04-23 13:25:44]

Apologies for the late reply.  I've uploaded two patches removing the plumbing
for both Chromium and Blink.  I tried updating this CL, but "git cl upload"
continued to complain that it couldn't find the base file.

Chromium side:
https://codereview.chromium.org/13843008/

Blink side:
https://codereview.chromium.org/14429004/

I'm abandoning this CL, but will refer to it in both new CLs for reference.