Make Intl install properties more like how other builtins do

Make Intl install properties more like how other builtins do

Intl has been somewhat of an oddball for how it integrates with V8.
One aspect is that it largely didn't use utils to install itself
into the snapshot, which led to some missing names, which new
test262 tests check for, and duplicated code. This patch brings
Intl a bit closer to how the rest of the builtins do things, though
not entirely as it is currently structured to do unusual things,
such as creating new constructors from JavaScript rather than C++.
New test262 tests check for some of the names that are added in
this patch.

R=adamk
CC=jshin
BUG=v8:4778
LOG=Y

Committed: https://crrev.com/a40830577d80f699282dd83864619656b7a7966c
Cr-Commit-Position: refs/heads/master@{#34311}

[Dan Ehrenberg, 2016-02-26 02:50:18]

Description was changed from

==========
Make Intl install properties more like how other builtins do

Intl has been somewhat of an oddball for how it integrates with V8.
One aspect is that it largely didn't use utils to install itself
into the snapshot, which led to some missing names, which new
test262 tests check for, and duplicated code. This patch brings
Intl a bit closer to how the rest of the builtins do things, though
not entirely as it is currently structured to do unusual things,
such as creating new constructors from JavaScript rather than C++.
New test262 tests check for some of the names that are added in
this patch.

R=adamk
CC=jshin
==========

to

==========
Make Intl install properties more like how other builtins do

Intl has been somewhat of an oddball for how it integrates with V8.
One aspect is that it largely didn't use utils to install itself
into the snapshot, which led to some missing names, which new
test262 tests check for, and duplicated code. This patch brings
Intl a bit closer to how the rest of the builtins do things, though
not entirely as it is currently structured to do unusual things,
such as creating new constructors from JavaScript rather than C++.
New test262 tests check for some of the names that are added in
this patch.

R=adamk
CC=jshin
BUG=v8:4778
==========

[Dan Ehrenberg, 2016-02-26 02:50:31]

The CQ bit was checked by littledan@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-02-26 02:50:36]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1733293003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1733293003/1

[commit-bot: I haz the power, 2016-02-26 02:54:22]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-02-26 02:54:23]

Dry run: Try jobs failed on following builders:
  v8_presubmit on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/11539)

[Dan Ehrenberg, 2016-02-26 03:02:17]

Description was changed from

==========
Make Intl install properties more like how other builtins do

Intl has been somewhat of an oddball for how it integrates with V8.
One aspect is that it largely didn't use utils to install itself
into the snapshot, which led to some missing names, which new
test262 tests check for, and duplicated code. This patch brings
Intl a bit closer to how the rest of the builtins do things, though
not entirely as it is currently structured to do unusual things,
such as creating new constructors from JavaScript rather than C++.
New test262 tests check for some of the names that are added in
this patch.

R=adamk
CC=jshin
BUG=v8:4778
==========

to

==========
Make Intl install properties more like how other builtins do

Intl has been somewhat of an oddball for how it integrates with V8.
One aspect is that it largely didn't use utils to install itself
into the snapshot, which led to some missing names, which new
test262 tests check for, and duplicated code. This patch brings
Intl a bit closer to how the rest of the builtins do things, though
not entirely as it is currently structured to do unusual things,
such as creating new constructors from JavaScript rather than C++.
New test262 tests check for some of the names that are added in
this patch.

R=adamk
CC=jshin
BUG=v8:4778
LOG=Y
==========

[adamk, 2016-02-26 04:32:41]

lgtm

[Dan Ehrenberg, 2016-02-26 04:38:44]

The CQ bit was checked by littledan@chromium.org

[commit-bot: I haz the power, 2016-02-26 04:38:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1733293003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1733293003/1

[commit-bot: I haz the power, 2016-02-26 04:40:57]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-26 04:41:54]

Description was changed from

==========
Make Intl install properties more like how other builtins do

Intl has been somewhat of an oddball for how it integrates with V8.
One aspect is that it largely didn't use utils to install itself
into the snapshot, which led to some missing names, which new
test262 tests check for, and duplicated code. This patch brings
Intl a bit closer to how the rest of the builtins do things, though
not entirely as it is currently structured to do unusual things,
such as creating new constructors from JavaScript rather than C++.
New test262 tests check for some of the names that are added in
this patch.

R=adamk
CC=jshin
BUG=v8:4778
LOG=Y
==========

to

==========
Make Intl install properties more like how other builtins do

Intl has been somewhat of an oddball for how it integrates with V8.
One aspect is that it largely didn't use utils to install itself
into the snapshot, which led to some missing names, which new
test262 tests check for, and duplicated code. This patch brings
Intl a bit closer to how the rest of the builtins do things, though
not entirely as it is currently structured to do unusual things,
such as creating new constructors from JavaScript rather than C++.
New test262 tests check for some of the names that are added in
this patch.

R=adamk
CC=jshin
BUG=v8:4778
LOG=Y

Committed: https://crrev.com/a40830577d80f699282dd83864619656b7a7966c
Cr-Commit-Position: refs/heads/master@{#34311}
==========

[commit-bot: I haz the power, 2016-02-26 04:41:55]

Patchset 1 (id:??) landed as
https://crrev.com/a40830577d80f699282dd83864619656b7a7966c
Cr-Commit-Position: refs/heads/master@{#34311}

[Dan Ehrenberg, 2016-02-26 05:25:30]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1737873003/ by littledan@chromium.org.

The reason for reverting is: Breaks a bot:
https://build.chromium.org/p/client.v8/builders/V8%20Linux%20-%20nosnap/build....

[Dan Ehrenberg, 2016-02-26 19:19:12]

On 2016/02/26 at 05:25:30, Dan Ehrenberg wrote:
> A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1737873003/ by littledan@chromium.org.
> 
> The reason for reverting is: Breaks a bot:
https://build.chromium.org/p/client.v8/builders/V8%20Linux%20-%20nosnap/build....

Reland: https://codereview.chromium.org/1745483002/