[test] Speed up some slow tests.

[test] Speed up some slow tests.

- split up cctest/test-api/Threading*
- shortened mjsunit/math-floor-part*
- split up cctest/test-serialize/CustomSnapshotDataBlob
- skipped mjsunit/regress/regress-crbug-474297 unless gc-stress
- shortened cctest/test-serialize/CodeSerializerLargeCodeObject

R=jkummerow@chromium.org

Committed: https://crrev.com/62937cf5335271698d47a4596813bdf2bacba4d5
Cr-Commit-Position: refs/heads/master@{#40753}

[Jakob Kummerow, 2016-11-03 13:28:40]

Yay! LGTM with comments.

I'll change the description to include [test] for you.

https://codereview.chromium.org/2470143004/diff/1/test/cctest/test-serialize.cc
File test/cctest/test-serialize.cc (right):

https://codereview.chromium.org/2470143004/diff/1/test/cctest/test-serialize....
test/cctest/test-serialize.cc:613: DisableTurbofan();
As discussed offline, it seems scary that these tests manually disable Turbofan.

https://codereview.chromium.org/2470143004/diff/1/test/mjsunit/mjsunit.status
File test/mjsunit/mjsunit.status (right):

https://codereview.chromium.org/2470143004/diff/1/test/mjsunit/mjsunit.status...
test/mjsunit/mjsunit.status:175: 'regress/regress-crbug-474297': [SKIP,
['gc_stress == True', PASS]],
Doesn't this always skip the test? I think you need:

[PASS, ['gc_stress == False', SKIP]]

[Jakob Kummerow, 2016-11-03 13:29:03]

Description was changed from

==========
Speed up some slow tests.

- split up cctest/test-api/Threading*
- shortened mjsunit/math-floor-part*
- split up cctest/test-serialize/CustomSnapshotDataBlob
- skipped mjsunit/regress/regress-crbug-474297 unless gc-stress
- shortend cctest/test-serialize/CodeSerializerLargeCodeObject

R=jkummerow@chromium.org
==========

to

==========
[test] Speed up some slow tests.

- split up cctest/test-api/Threading*
- shortened mjsunit/math-floor-part*
- split up cctest/test-serialize/CustomSnapshotDataBlob
- skipped mjsunit/regress/regress-crbug-474297 unless gc-stress
- shortened cctest/test-serialize/CodeSerializerLargeCodeObject

R=jkummerow@chromium.org
==========

[Yang, 2016-11-04 08:39:34]

The CQ bit was checked by yangguo@chromium.org

[Yang, 2016-11-04 08:39:34]

The patchset sent to the CQ was uploaded after l-g-t-m from
jkummerow@chromium.org
Link to the patchset: https://codereview.chromium.org/2470143004/#ps20001
(title: "address comment")

[commit-bot: I haz the power, 2016-11-04 08:39:39]

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

[Yang, 2016-11-04 08:39:41]

https://codereview.chromium.org/2470143004/diff/1/test/cctest/test-serialize.cc
File test/cctest/test-serialize.cc (right):

https://codereview.chromium.org/2470143004/diff/1/test/cctest/test-serialize....
test/cctest/test-serialize.cc:613: DisableTurbofan();
On 2016/11/03 13:28:40, Jakob Kummerow wrote:
> As discussed offline, it seems scary that these tests manually disable
Turbofan.

Will address this in a follow-up.

https://codereview.chromium.org/2470143004/diff/1/test/mjsunit/mjsunit.status
File test/mjsunit/mjsunit.status (right):

https://codereview.chromium.org/2470143004/diff/1/test/mjsunit/mjsunit.status...
test/mjsunit/mjsunit.status:175: 'regress/regress-crbug-474297': [SKIP,
['gc_stress == True', PASS]],
On 2016/11/03 13:28:40, Jakob Kummerow wrote:
> Doesn't this always skip the test? I think you need:
> 
> [PASS, ['gc_stress == False', SKIP]]

Done. Though I wouldn't have thought this has a different meaning :)

[commit-bot: I haz the power, 2016-11-04 09:04:17]

Description was changed from

==========
[test] Speed up some slow tests.

- split up cctest/test-api/Threading*
- shortened mjsunit/math-floor-part*
- split up cctest/test-serialize/CustomSnapshotDataBlob
- skipped mjsunit/regress/regress-crbug-474297 unless gc-stress
- shortened cctest/test-serialize/CodeSerializerLargeCodeObject

R=jkummerow@chromium.org
==========

to

==========
[test] Speed up some slow tests.

- split up cctest/test-api/Threading*
- shortened mjsunit/math-floor-part*
- split up cctest/test-serialize/CustomSnapshotDataBlob
- skipped mjsunit/regress/regress-crbug-474297 unless gc-stress
- shortened cctest/test-serialize/CodeSerializerLargeCodeObject

R=jkummerow@chromium.org
==========

[commit-bot: I haz the power, 2016-11-04 09:04:18]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-17 22:22:10]

Description was changed from

==========
[test] Speed up some slow tests.

- split up cctest/test-api/Threading*
- shortened mjsunit/math-floor-part*
- split up cctest/test-serialize/CustomSnapshotDataBlob
- skipped mjsunit/regress/regress-crbug-474297 unless gc-stress
- shortened cctest/test-serialize/CodeSerializerLargeCodeObject

R=jkummerow@chromium.org
==========

to

==========
[test] Speed up some slow tests.

- split up cctest/test-api/Threading*
- shortened mjsunit/math-floor-part*
- split up cctest/test-serialize/CustomSnapshotDataBlob
- skipped mjsunit/regress/regress-crbug-474297 unless gc-stress
- shortened cctest/test-serialize/CodeSerializerLargeCodeObject

R=jkummerow@chromium.org

Committed: https://crrev.com/62937cf5335271698d47a4596813bdf2bacba4d5
Cr-Commit-Position: refs/heads/master@{#40753}
==========

[commit-bot: I haz the power, 2016-11-17 22:22:12]

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