[inspector] AsyncStackTrace should always have non-zero context group id

[inspector] AsyncStackTrace should always have non-zero context group id

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

Review-Url: https://codereview.chromium.org/2822073002
Cr-Commit-Position: refs/heads/master@{#44733}
Committed: https://chromium.googlesource.com/v8/v8/+/c38e88651b68ef8e0be4c51067423f2cd938c157

[kozy, 2017-04-18 01:32:31]

Dmitry, please take a look.

[dgozman, 2017-04-18 18:28:19]

Should the same happen to V8StackTraceImpl?

[kozy, 2017-04-18 18:29:56]

On 2017/04/18 18:28:19, dgozman wrote:
> Should the same happen to V8StackTraceImpl?

For V8StackTraceImpl we need only to check that when we're creating it we don't
accidentally append async parent or async creation with wrong context group id.
We don't store context group id in V8StackTraceImpl because it could not be a
parent for any other stack.

[dgozman, 2017-04-19 19:42:27]

lgtm

[kozy, 2017-04-19 20:12:24]

The CQ bit was checked by kozyatinskiy@chromium.org

[commit-bot: I haz the power, 2017-04-19 20:12:30]

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-04-19 20:35:59]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492632744105710, "parent_rev":
"9b51273278bcf99d1eb3f705de1f11a8c5df8964", "commit_rev":
"c38e88651b68ef8e0be4c51067423f2cd938c157"}

[commit-bot: I haz the power, 2017-04-19 20:36:05]

Description was changed from

==========
[inspector] AsyncStackTrace should always have non-zero context group id

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

to

==========
[inspector] AsyncStackTrace should always have non-zero context group id

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

Review-Url: https://codereview.chromium.org/2822073002
Cr-Commit-Position: refs/heads/master@{#44733}
Committed:
https://chromium.googlesource.com/v8/v8/+/c38e88651b68ef8e0be4c51067423f2cd93...
==========

[commit-bot: I haz the power, 2017-04-19 20:36:05]

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