[profiler] fix memory leak for code entries for runtime callstats.

[profiler] fix memory leak for code entries for runtime callstats.

Track allocated code entries and delete at the end. This is what we
do in ProfileListener too.

R=alph@chromium.org, cbruni@chromium.org
BUG=v8:5753
Review-Url: https://codereview.chromium.org/2586923002
Cr-Commit-Position: refs/heads/master@{#41793}
Committed: https://chromium.googlesource.com/v8/v8/+/d0bb789f0358851b6580a274e2b5f2a8f728f44b

[Yang, 2016-12-19 09:53:38]

The CQ bit was checked by yangguo@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-19 09:53:41]

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

[Camillo Bruni, 2016-12-19 10:02:01]

lgtm

[Yang, 2016-12-19 10:05:19]

The CQ bit was unchecked by yangguo@chromium.org

[Yang, 2016-12-19 10:44:39]

The CQ bit was checked by yangguo@chromium.org

[commit-bot: I haz the power, 2016-12-19 10:44:48]

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, 2016-12-19 10:46:56]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1482144279885540, "parent_rev":
"d4493222b958877e51ebd67399d12a80ec848e30", "commit_rev":
"d0bb789f0358851b6580a274e2b5f2a8f728f44b"}

[commit-bot: I haz the power, 2016-12-19 10:47:03]

Description was changed from

==========
[profiler] fix memory leak for code entries for runtime callstats.

Track allocated code entries and delete at the end. This is what we
do in ProfileListener too.

R=alph@chromium.org, cbruni@chromium.org
BUG=v8:5753
==========

to

==========
[profiler] fix memory leak for code entries for runtime callstats.

Track allocated code entries and delete at the end. This is what we
do in ProfileListener too.

R=alph@chromium.org, cbruni@chromium.org
BUG=v8:5753

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

[commit-bot: I haz the power, 2016-12-19 10:47:04]

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

[Michael Achenbach, 2016-12-19 13:49:22]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2590483002/ by machenbach@chromium.org.

The reason for reverting is: Looks like the layout tests want these leaks:
https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/bui...

See:
https://github.com/v8/v8/wiki/Blink-layout-tests.

[Yang, 2016-12-19 14:18:39]

Description was changed from

==========
[profiler] fix memory leak for code entries for runtime callstats.

Track allocated code entries and delete at the end. This is what we
do in ProfileListener too.

R=alph@chromium.org, cbruni@chromium.org
BUG=v8:5753

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

to

==========
[profiler] fix memory leak for code entries for runtime callstats.

Track allocated code entries and delete at the end. This is what we
do in ProfileListener too.

R=alph@chromium.org, cbruni@chromium.org
BUG=v8:5753

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