Re-land %TypedArray%.prototype.{map,filter,some}

Re-land %TypedArray%.prototype.{map,filter,some}

Previously this patch was attempted with reduce and reduceRight included;
however, some of those tests crashed in the trybots. This version has
just map, fiter and some, together with their tests.

R=arv@chromium.org
BUG=v8:3578
LOG=Y

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

[dehrenberg, 2015-05-20 06:35:41]

dehrenberg@chromium.org changed reviewers:
+ adamk@chromium.org
- arv@chromium.org

[dehrenberg, 2015-05-20 06:35:42]

The CQ bit was checked by dehrenberg@chromium.org

[dehrenberg, 2015-05-20 06:35:59]

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

[commit-bot: I haz the power, 2015-05-20 06:36:04]

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

[commit-bot: I haz the power, 2015-05-20 06:36:06]

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

[commit-bot: I haz the power, 2015-05-20 06:36:08]

Dry run: No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[adamk, 2015-05-20 18:50:07]

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

https://codereview.chromium.org/1145013002/diff/1/src/harmony-typedarray.js#n...
src/harmony-typedarray.js:33: function ConstructTypedArray(constructor, array) {
Where is this called?

https://codereview.chromium.org/1145013002/diff/1/src/harmony-typedarray.js#n...
src/harmony-typedarray.js:36: if (!IS_SPEC_OBJECT(constructor) ||
IS_UNDEFINED(constructor.prototype) ||
Can you use %IsConstructor here? These checks look quite odd. (if this function
is unused, you can of course ignore this comment)

https://codereview.chromium.org/1145013002/diff/1/src/harmony-typedarray.js#n...
src/harmony-typedarray.js:265: "fill", TypedArrayFill,
This is a dup of the sorted line above.

[arv (Not doing code reviews), 2015-05-20 19:22:31]

arv@chromium.org changed reviewers:
+ arv@chromium.org

[arv (Not doing code reviews), 2015-05-20 19:22:31]

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

https://codereview.chromium.org/1145013002/diff/1/src/harmony-typedarray.js#n...
src/harmony-typedarray.js:33: function ConstructTypedArray(constructor, array) {
Also, this function is defined twice... was there a merge?

[dehrenberg, 2015-05-20 23:39:01]

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

https://codereview.chromium.org/1145013002/diff/1/src/harmony-typedarray.js#n...
src/harmony-typedarray.js:33: function ConstructTypedArray(constructor, array) {
On 2015/05/20 19:22:31, arv wrote:
> Also, this function is defined twice... was there a merge?

Yes. OK, I learned my lesson, not to do git conflict resolution late at night.
It was intended to be called in ConstructTypedArrayLike, but apparently I forgot
to call it there (a bug that goes back to the first time I tried to land this
code). Before you ask: it'd be hard to test this code directly until @@species
goes in; I have a test in TypedArray.from, but no test that TypedArray.map
reaches this code.

https://codereview.chromium.org/1145013002/diff/1/src/harmony-typedarray.js#n...
src/harmony-typedarray.js:36: if (!IS_SPEC_OBJECT(constructor) ||
IS_UNDEFINED(constructor.prototype) ||
On 2015/05/20 18:50:07, adamk wrote:
> Can you use %IsConstructor here? These checks look quite odd. (if this
function
> is unused, you can of course ignore this comment)

The check is supposed to be an approximation of checking that this is a
TypedArray class. I replaced IS_SPEC_OBJECT with %IsConstructor, but I still
need the next couple checks.

https://codereview.chromium.org/1145013002/diff/1/src/harmony-typedarray.js#n...
src/harmony-typedarray.js:265: "fill", TypedArrayFill,
On 2015/05/20 18:50:07, adamk wrote:
> This is a dup of the sorted line above.

Done.

[adamk, 2015-05-21 00:03:19]

lgtm

[dehrenberg, 2015-05-21 00:07:21]

The CQ bit was checked by dehrenberg@chromium.org

[commit-bot: I haz the power, 2015-05-21 00:07:41]

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

[commit-bot: I haz the power, 2015-05-21 01:38:26]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-05-21 01:38:42]

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