[interpreter] Fix stale DCHECK in tracing.

[interpreter] Adds test for tracing.

Fixes a stale DCHECK and a memory leak in tracing output.

LOG=N
BUG=v8:4280
TBR=rmcilroy@chromium.org

Committed: https://crrev.com/8cf73ae1e99b9adf5ec3aad66dae175c736ba188
Cr-Commit-Position: refs/heads/master@{#35161}

[oth, 2016-03-31 10:18:32]

oth@chromium.org changed reviewers:
+ mythria@chromium.org
- rmcilroy@chromium.org

[oth, 2016-03-31 10:18:32]

Mythri, PTAL. We should have a test for this, but need to figure out how to
plumb tracing output to a stream of our choosing per isolate.

Thanks

[oth, 2016-03-31 11:08:28]

On 2016/03/31 10:18:32, oth wrote:
> Mythri, PTAL. We should have a test for this, but need to figure out how to
> plumb tracing output to a stream of our choosing per isolate.
> 
> Thanks

PTAL - test added and a memory leak fixed. Thanks!

[mythria, 2016-03-31 11:15:15]

lgtm

[oth, 2016-03-31 11:32:36]

Description was changed from

==========
[interpreter] Fix stale DCHECK in tracing.

LOG=N
BUG=v8:4280
TBR=rmcilroy@chromium.org
==========

to

==========
[interpreter] Adds test for tracing.

Fixes a stale DCHECK and a memory leak in tracing output.

LOG=N
BUG=v8:4280
TBR=rmcilroy@chromium.org
==========

[oth, 2016-03-31 11:54:14]

The CQ bit was checked by oth@chromium.org

[commit-bot: I haz the power, 2016-03-31 11:54:26]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1844023003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1844023003/20001

[commit-bot: I haz the power, 2016-03-31 11:56:06]

Description was changed from

==========
[interpreter] Adds test for tracing.

Fixes a stale DCHECK and a memory leak in tracing output.

LOG=N
BUG=v8:4280
TBR=rmcilroy@chromium.org
==========

to

==========
[interpreter] Adds test for tracing.

Fixes a stale DCHECK and a memory leak in tracing output.

LOG=N
BUG=v8:4280
TBR=rmcilroy@chromium.org
==========

[commit-bot: I haz the power, 2016-03-31 11:56:07]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-31 11:58:10]

Description was changed from

==========
[interpreter] Adds test for tracing.

Fixes a stale DCHECK and a memory leak in tracing output.

LOG=N
BUG=v8:4280
TBR=rmcilroy@chromium.org
==========

to

==========
[interpreter] Adds test for tracing.

Fixes a stale DCHECK and a memory leak in tracing output.

LOG=N
BUG=v8:4280
TBR=rmcilroy@chromium.org

Committed: https://crrev.com/8cf73ae1e99b9adf5ec3aad66dae175c736ba188
Cr-Commit-Position: refs/heads/master@{#35161}
==========

[commit-bot: I haz the power, 2016-03-31 11:58:10]

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