Make %TypedArray%.from spec-compliant

Make %TypedArray%.from spec-compliant

This patch fixes %TypedArray%.from to follow the ES2016 draft spec
more precisely. Specifically, the input is first converted to an
ArrayLike, and then afterwards, the mapping function is run and the
results written into the TypedArray. This fixes a test262 test.

R=adamk
LOG=Y
BUG=v8:4782
Committed: https://crrev.com/2765a0609a8aed612a3e34d9f9897d9ec7fc969e
Cr-Commit-Position: refs/heads/master@{#34373}

[Dan Ehrenberg, 2016-02-26 21:28:18]

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

[commit-bot: I haz the power, 2016-02-26 21:28:21]

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

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

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

[commit-bot: I haz the power, 2016-02-26 21:53:37]

Dry run: This issue passed the CQ dry run.

[adamk, 2016-02-26 22:18:48]

https://codereview.chromium.org/1743463004/diff/1/src/js/typedarray.js
File src/js/typedarray.js (right):

https://codereview.chromium.org/1743463004/diff/1/src/js/typedarray.js#newcod...
src/js/typedarray.js:771: var array = new GlobalArray();
Can you use an InternalArray here and %MoveArrayContents below? My read is that
this is not meant to be observable.

[Dan Ehrenberg, 2016-02-29 23:24:25]

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

[commit-bot: I haz the power, 2016-02-29 23:24:36]

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

[Dan Ehrenberg, 2016-02-29 23:24:44]

https://codereview.chromium.org/1743463004/diff/1/src/js/typedarray.js
File src/js/typedarray.js (right):

https://codereview.chromium.org/1743463004/diff/1/src/js/typedarray.js#newcod...
src/js/typedarray.js:771: var array = new GlobalArray();
On 2016/02/26 at 22:18:48, adamk wrote:
> Can you use an InternalArray here and %MoveArrayContents below? My read is
that this is not meant to be observable.

Done

[commit-bot: I haz the power, 2016-02-29 23:42:12]

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

[commit-bot: I haz the power, 2016-02-29 23:42:13]

Dry run: This issue passed the CQ dry run.

[adamk, 2016-03-01 02:22:36]

lgtm

[Dan Ehrenberg, 2016-03-01 02:38:46]

The CQ bit was checked by littledan@chromium.org

[commit-bot: I haz the power, 2016-03-01 02:38:49]

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

[commit-bot: I haz the power, 2016-03-01 02:40:35]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-01 02:41:47]

Description was changed from

==========
Make %TypedArray%.from spec-compliant

This patch fixes %TypedArray%.from to follow the ES2016 draft spec
more precisely. Specifically, the input is first converted to an
ArrayLike, and then afterwards, the mapping function is run and the
results written into the TypedArray. This fixes a test262 test.

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

to

==========
Make %TypedArray%.from spec-compliant

This patch fixes %TypedArray%.from to follow the ES2016 draft spec
more precisely. Specifically, the input is first converted to an
ArrayLike, and then afterwards, the mapping function is run and the
results written into the TypedArray. This fixes a test262 test.

R=adamk
LOG=Y
BUG=v8:4782

Committed: https://crrev.com/2765a0609a8aed612a3e34d9f9897d9ec7fc969e
Cr-Commit-Position: refs/heads/master@{#34373}
==========

[commit-bot: I haz the power, 2016-03-01 02:41:48]

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