Remove tiny unit test for MinSimple/MaxSimple

Remove tiny unit test for MinSimple/MaxSimple

The source code of those functions is simple a ternary operator, and
the test seems to have been designed as a "regression" test for the
previous implementation (which used a macro, and thus evaluated its
arguments multiple times).

I don't think the test is worth the weight of supporting importing
JS natives from tests.

This patch removes the last test that depends on JS natives.

R=littledan@chromium.org

Review-Url: https://codereview.chromium.org/2711833002
Cr-Commit-Position: refs/heads/master@{#43394}
Committed: https://chromium.googlesource.com/v8/v8/+/837760ecb7ff2ccaf7921ab46e7b473b313d4a4d

[adamk, 2017-02-22 22:52:39]

Description was changed from

==========
Remove tiny unit test for MinSimple/MaxSimple

The source code of those functions is simple a ternary operator, and
the test seems to have been designed as a "regression" test for the
previous implementation (which used a macro). I don't think the test
is worth the weight of supporting importing JS natives from tests.

R=littledan@chromium.org
==========

to

==========
Remove tiny unit test for MinSimple/MaxSimple

The source code of those functions is simple a ternary operator, and
the test seems to have been designed as a "regression" test for the
previous implementation (which used a macro). I don't think the test
is worth the weight of supporting importing JS natives from tests.

This patch removes the last test that depends on JS natives.

R=littledan@chromium.org
==========

[adamk, 2017-02-22 23:39:25]

Description was changed from

==========
Remove tiny unit test for MinSimple/MaxSimple

The source code of those functions is simple a ternary operator, and
the test seems to have been designed as a "regression" test for the
previous implementation (which used a macro). I don't think the test
is worth the weight of supporting importing JS natives from tests.

This patch removes the last test that depends on JS natives.

R=littledan@chromium.org
==========

to

==========
Remove tiny unit test for MinSimple/MaxSimple

The source code of those functions is simple a ternary operator, and
the test seems to have been designed as a "regression" test for the
previous implementation (which used a macro, and thus evaluated its
arguments multiple times).

I don't think the test is worth the weight of supporting importing
JS natives from tests.

This patch removes the last test that depends on JS natives.

R=littledan@chromium.org
==========

[Yang, 2017-02-23 06:07:57]

On 2017/02/22 22:52:05, adamk wrote:

lgtm

[Dan Ehrenberg, 2017-02-23 10:32:17]

lgtm

[adamk, 2017-02-23 17:49:36]

The CQ bit was checked by adamk@chromium.org

[commit-bot: I haz the power, 2017-02-23 17:49:40]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-23 18:19:17]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487872176952760, "parent_rev":
"184e80a60d95deee5b5b5390d3672aafb6b7da3f", "commit_rev":
"837760ecb7ff2ccaf7921ab46e7b473b313d4a4d"}

[commit-bot: I haz the power, 2017-02-23 18:19:22]

Description was changed from

==========
Remove tiny unit test for MinSimple/MaxSimple

The source code of those functions is simple a ternary operator, and
the test seems to have been designed as a "regression" test for the
previous implementation (which used a macro, and thus evaluated its
arguments multiple times).

I don't think the test is worth the weight of supporting importing
JS natives from tests.

This patch removes the last test that depends on JS natives.

R=littledan@chromium.org
==========

to

==========
Remove tiny unit test for MinSimple/MaxSimple

The source code of those functions is simple a ternary operator, and
the test seems to have been designed as a "regression" test for the
previous implementation (which used a macro, and thus evaluated its
arguments multiple times).

I don't think the test is worth the weight of supporting importing
JS natives from tests.

This patch removes the last test that depends on JS natives.

R=littledan@chromium.org

Review-Url: https://codereview.chromium.org/2711833002
Cr-Commit-Position: refs/heads/master@{#43394}
Committed:
https://chromium.googlesource.com/v8/v8/+/837760ecb7ff2ccaf7921ab46e7b473b313...
==========

[commit-bot: I haz the power, 2017-02-23 18:19:23]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/837760ecb7ff2ccaf7921ab46e7b473b313...