Promise constructor should not be enumerable.

Promise constructor should not be enumerable.

Promise should not be enumerable in the global object.

BUG=352597
R=rossberg@chromium.org
LOG=Y

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

[yhirano, 2014-03-26 02:28:19]

PTAL

This fixes webexposed/ layout tests in blink, so it should be landed and rolled
before https://codereview.chromium.org/209853010/ lands.

[rossberg, 2014-03-26 09:59:37]

https://codereview.chromium.org/211943004/diff/1/src/promise.js
File src/promise.js (right):

https://codereview.chromium.org/211943004/diff/1/src/promise.js#newcode311
src/promise.js:311: $Object.defineProperty(global_receiver, 'Promise', {
You can use %SetProperty directly, like I did in the rolled-back CL:

https://codereview.chromium.org/206163004/diff/1/src/promise.js#newcode308

Just replace NONE with DONT_ENUM.

[yhirano, 2014-03-26 10:09:11]

https://codereview.chromium.org/211943004/diff/1/src/promise.js
File src/promise.js (right):

https://codereview.chromium.org/211943004/diff/1/src/promise.js#newcode311
src/promise.js:311: $Object.defineProperty(global_receiver, 'Promise', {
On 2014/03/26 09:59:37, rossberg wrote:
> You can use %SetProperty directly, like I did in the rolled-back CL:
> 
> https://codereview.chromium.org/206163004/diff/1/src/promise.js#newcode308
> 
> Just replace NONE with DONT_ENUM.

Thanks, done.

[rossberg, 2014-03-26 10:11:45]

lgtm

[rossberg, 2014-03-26 10:14:10]

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