[DevTools] increase AsyncCallStackDepth to 8

[DevTools] increase AsyncCallStackDepth to 8

Amount of stored stack traces are limits in v8 after [1].
After landing better async stacks for promises [2] we won't omit empty asynchronous stack traces to provide information where promise is chained, so we need bigger limit to be useful.

[1] https://codereview.chromium.org/2579403002/
[2] https://codereview.chromium.org/2648873002/

BUG=v8:5738
R=dgozman@chromium.org

Review-Url: https://codereview.chromium.org/2650943009
Cr-Commit-Position: refs/heads/master@{#446224}
Committed: https://chromium.googlesource.com/chromium/src/+/00427c5f26e25fd1c188a46ce5e2798e32ca05b4

[kozy, 2017-01-25 02:00:15]

Dmitry, please take a look. Surprisingly all our tests currently dump less then
5 async call chains.

[dgozman, 2017-01-25 23:28:41]

On 2017/01/25 02:00:15, kozy wrote:
> Dmitry, please take a look. Surprisingly all our tests currently dump less
then
> 5 async call chains.
Suspicious! I think we set depth in tests separately. Let's bump that one as
well?

change lgtm

[kozy, 2017-01-26 01:37:06]

Increased limit for all tests.

[kozy, 2017-01-26 01:37:11]

The CQ bit was checked by kozyatinskiy@chromium.org

[kozy, 2017-01-26 01:37:11]

The patchset sent to the CQ was uploaded after l-g-t-m from dgozman@chromium.org
Link to the patchset: https://codereview.chromium.org/2650943009/#ps40001
(title: "changed stack depth in tests")

[commit-bot: I haz the power, 2017-01-26 01:38:04]

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

[kozy, 2017-01-26 01:57:20]

The CQ bit was checked by kozyatinskiy@chromium.org

[kozy, 2017-01-26 01:57:21]

The patchset sent to the CQ was uploaded after l-g-t-m from dgozman@chromium.org
Link to the patchset: https://codereview.chromium.org/2650943009/#ps60001
(title: "added missing test")

[commit-bot: I haz the power, 2017-01-26 01:58:19]

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-01-26 04:18:05]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1485395840757460,
"parent_rev": "81183e45cf14c416dfce69b990a0c71a4e1c3399", "commit_rev":
"00427c5f26e25fd1c188a46ce5e2798e32ca05b4"}

[commit-bot: I haz the power, 2017-01-26 04:18:48]

Description was changed from

==========
[DevTools] increase AsyncCallStackDepth to 8

Amount of stored stack traces are limits in v8 after [1].
After landing better async stacks for promises [2] we won't omit empty
asynchronous stack traces to provide information where promise is chained, so we
need bigger limit to be useful.

[1] https://codereview.chromium.org/2579403002/
[2] https://codereview.chromium.org/2648873002/

BUG=v8:5738
R=dgozman@chromium.org
==========

to

==========
[DevTools] increase AsyncCallStackDepth to 8

Amount of stored stack traces are limits in v8 after [1].
After landing better async stacks for promises [2] we won't omit empty
asynchronous stack traces to provide information where promise is chained, so we
need bigger limit to be useful.

[1] https://codereview.chromium.org/2579403002/
[2] https://codereview.chromium.org/2648873002/

BUG=v8:5738
R=dgozman@chromium.org

Review-Url: https://codereview.chromium.org/2650943009
Cr-Commit-Position: refs/heads/master@{#446224}
Committed:
https://chromium.googlesource.com/chromium/src/+/00427c5f26e25fd1c188a46ce5e2...
==========

[commit-bot: I haz the power, 2017-01-26 04:18:49]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/00427c5f26e25fd1c188a46ce5e2...