Intl: Use private symbols to memoize bound functions

Intl: Use private symbols to memoize bound functions

The Intl object used to keep around functions which are bound to the
receiver and memoized in the object (as required by the ECMA-402 spec)
in ordinary properties with names like __boundformat__. This patch
instead stores those methods in private symbol properties, so they are
not exposed to users. A search in GitHub didn't find any uses of
__boundformat__ (whereas the same search found plenty of usages of
other V8 Intl features), so I think this should be fine in terms of
web compatibility.

BUG=v8:3785
R=adamk
LOG=Y

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

[Dan Ehrenberg, 2016-02-23 22:31:23]

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

[commit-bot: I haz the power, 2016-02-23 22:31:30]

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

[commit-bot: I haz the power, 2016-02-23 22:53:17]

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

[commit-bot: I haz the power, 2016-02-23 22:53:18]

Dry run: This issue passed the CQ dry run.

[adamk, 2016-02-23 23:14:56]

lgtm

https://codereview.chromium.org/1728823002/diff/1/src/js/i18n.js
File src/js/i18n.js (right):

https://codereview.chromium.org/1728823002/diff/1/src/js/i18n.js#newcode230
src/js/i18n.js:230: // funciton macro.
Typo nit: s/funciton/function/

[Dan Ehrenberg, 2016-02-23 23:21:58]

The CQ bit was checked by littledan@chromium.org

[Dan Ehrenberg, 2016-02-23 23:21:58]

The patchset sent to the CQ was uploaded after l-g-t-m from adamk@chromium.org
Link to the patchset: https://codereview.chromium.org/1728823002/#ps20001
(title: "fix typo")

[commit-bot: I haz the power, 2016-02-23 23:22:01]

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

[commit-bot: I haz the power, 2016-02-24 00:04:41]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-02-24 00:05:15]

Description was changed from

==========
Intl: Use private symbols to memoize bound functions

The Intl object used to keep around functions which are bound to the
receiver and memoized in the object (as required by the ECMA-402 spec)
in ordinary properties with names like __boundformat__. This patch
instead stores those methods in private symbol properties, so they are
not exposed to users. A search in GitHub didn't find any uses of
__boundformat__ (whereas the same search found plenty of usages of
other V8 Intl features), so I think this should be fine in terms of
web compatibility.

BUG=v8:3785
R=adamk
LOG=Y
==========

to

==========
Intl: Use private symbols to memoize bound functions

The Intl object used to keep around functions which are bound to the
receiver and memoized in the object (as required by the ECMA-402 spec)
in ordinary properties with names like __boundformat__. This patch
instead stores those methods in private symbol properties, so they are
not exposed to users. A search in GitHub didn't find any uses of
__boundformat__ (whereas the same search found plenty of usages of
other V8 Intl features), so I think this should be fine in terms of
web compatibility.

BUG=v8:3785
R=adamk
LOG=Y

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

[commit-bot: I haz the power, 2016-02-24 00:05:16]

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