[inspector] cleanup all task related data when limit reached

[inspector] cleanup all task related data when limit reached

- and reduce limit to 128 * 1024.

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

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

[kozy, 2017-04-18 21:04:19]

Dmitry, please take a look. It's preparation for for removing cancel event.

[dgozman, 2017-04-19 20:09:53]

lgtm, but test changes look suspicious

https://codereview.chromium.org/2824293002/diff/20001/test/inspector/debugger...
File test/inspector/debugger/collect-old-async-call-chains-expected.txt (right):

https://codereview.chromium.org/2824293002/diff/20001/test/inspector/debugger...
test/inspector/debugger/collect-old-async-call-chains-expected.txt:174: actual
async chain len: 0
Why this change?

[kozy, 2017-04-19 22:50:22]

Description was changed from

==========
[inspector] cleanup all task related data when limit reached

- and reduce limit to 512 * 1024.

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

to

==========
[inspector] cleanup all task related data when limit reached

- and reduce limit to 128 * 1024.

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

[kozy, 2017-04-19 23:41:57]

Patchset #5 (id:80001) has been deleted

[kozy, 2017-04-19 23:42:08]

Patchset #4 (id:60001) has been deleted

[kozy, 2017-04-19 23:42:20]

Patchset #3 (id:40001) has been deleted

[kozy, 2017-04-19 23:45:50]

Patchset #3 (id:100001) has been deleted

[kozy, 2017-04-19 23:46:06]

https://codereview.chromium.org/2824293002/diff/20001/test/inspector/debugger...
File test/inspector/debugger/collect-old-async-call-chains-expected.txt (right):

https://codereview.chromium.org/2824293002/diff/20001/test/inspector/debugger...
test/inspector/debugger/collect-old-async-call-chains-expected.txt:174: actual
async chain len: 0
On 2017/04/19 20:09:53, dgozman wrote:
> Why this change?

fixed

[kozy, 2017-04-19 23:46:10]

The CQ bit was checked by kozyatinskiy@chromium.org

[kozy, 2017-04-19 23:46: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/2824293002/#ps120001
(title: "fixed test")

[commit-bot: I haz the power, 2017-04-19 23:46: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-04-20 00:14:50]

CQ is committing da patch.

Bot data: {"patchset_id": 120001, "attempt_start_ts": 1492645570832040,
"parent_rev": "5806d86208c3845b94c8da6953830d428721c112", "commit_rev":
"d1ba8c8546706ea6fa819ca896b33c5df4c5d3c7"}

[commit-bot: I haz the power, 2017-04-20 00:14:57]

Description was changed from

==========
[inspector] cleanup all task related data when limit reached

- and reduce limit to 128 * 1024.

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

to

==========
[inspector] cleanup all task related data when limit reached

- and reduce limit to 128 * 1024.

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

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

[commit-bot: I haz the power, 2017-04-20 00:14:58]

Committed patchset #3 (id:120001) as
https://chromium.googlesource.com/v8/v8/+/d1ba8c8546706ea6fa819ca896b33c5df4c...

[dgozman, 2017-04-20 00:42:23]

Please fix :-)

https://codereview.chromium.org/2824293002/diff/120001/src/inspector/v8-debug...
File src/inspector/v8-debugger.cc (right):

https://codereview.chromium.org/2824293002/diff/120001/src/inspector/v8-debug...
src/inspector/v8-debugger.cc:1033: if (m_asyncTaskCreationStacks.find(it.second)
== m_asyncTaskStacks.end()) {
You are comparing iterator from one map to another's end().

[kozy, 2017-04-20 00:59:02]

thanks!

https://codereview.chromium.org/2824293002/diff/120001/src/inspector/v8-debug...
File src/inspector/v8-debugger.cc (right):

https://codereview.chromium.org/2824293002/diff/120001/src/inspector/v8-debug...
src/inspector/v8-debugger.cc:1033: if (m_asyncTaskCreationStacks.find(it.second)
== m_asyncTaskStacks.end()) {
On 2017/04/20 00:42:23, dgozman wrote:
> You are comparing iterator from one map to another's end().

C++ is my best friend since it allows it :)
fixed: https://codereview.chromium.org/2828133002/