Bugfix: internationalization routines fail on monkeypatching.

Bugfix: internationalization routines fail on monkeypatching.

Calls to Object.defineProperty() and Object.apply() are not safe.

R=mstarzinger@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=21071

[mvstanton, 2014-04-29 16:52:08]

Hey Michael, here is the update, thx!
--Michael

[Michael Starzinger, 2014-04-29 20:20:28]

LGTM.

https://codereview.chromium.org/253903003/diff/1/test/mjsunit/regress/support...
File test/mjsunit/regress/supported-locales-of.js (right):

https://codereview.chromium.org/253903003/diff/1/test/mjsunit/regress/support...
test/mjsunit/regress/supported-locales-of.js:1: // Copyright 2014 the V8 project
authors. All rights reserved.
nit: The name of the test-case is misleading as it tests more than the name
indicates. How do you feel about adding it to the "regress-builtinbust-[n].js"
family?

https://codereview.chromium.org/253903003/diff/1/test/mjsunit/regress/support...
test/mjsunit/regress/supported-locales-of.js:19: 
nit: Let's drop one of the two empty newlines.

[mvstanton, 2014-04-30 07:15:58]

done, thanks!

https://codereview.chromium.org/253903003/diff/1/test/mjsunit/regress/support...
File test/mjsunit/regress/supported-locales-of.js (right):

https://codereview.chromium.org/253903003/diff/1/test/mjsunit/regress/support...
test/mjsunit/regress/supported-locales-of.js:1: // Copyright 2014 the V8 project
authors. All rights reserved.
On 2014/04/29 20:20:28, Michael Starzinger wrote:
> nit: The name of the test-case is misleading as it tests more than the name
> indicates. How do you feel about adding it to the "regress-builtinbust-[n].js"
> family?

Done.

https://codereview.chromium.org/253903003/diff/1/test/mjsunit/regress/support...
test/mjsunit/regress/supported-locales-of.js:19: 
On 2014/04/29 20:20:28, Michael Starzinger wrote:
> nit: Let's drop one of the two empty newlines.

Done.

[mvstanton, 2014-04-30 07:36:18]

Committed patchset #2 manually as r21071 (presubmit successful).