Use TracedValue in runtime statistics.

Use TracedValue in runtime statistics.

We introduced TracedValue into V8 tracing previously, this patch uses it to
build JSON string of runtime statistics instead of using stringstream as buffer.

BUG=v8:5089
LOG=N

Committed: https://crrev.com/65b3af466e9aea7bc451558d671fcd9866b6cbe3
Cr-Commit-Position: refs/heads/master@{#40443}

[lpy, 2016-10-14 22:37:58]

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

[commit-bot: I haz the power, 2016-10-14 22:38:03]

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

[lpy, 2016-10-14 22:40:11]

lpy@chromium.org changed reviewers:
+ cbruni@chromium.org, fmeawad@chromium.org

[lpy, 2016-10-14 22:40:12]

PTAL.

[commit-bot: I haz the power, 2016-10-14 23:19:30]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-14 23:19:31]

Dry run: This issue passed the CQ dry run.

[Camillo Bruni, 2016-10-18 07:52:24]

lgtm with nit

https://codereview.chromium.org/2418303002/diff/1/src/counters.cc
File src/counters.cc (right):

https://codereview.chromium.org/2418303002/diff/1/src/counters.cc#newcode369
src/counters.cc:369: }
Please leave the separate function in place as the macros blow up the binary
size quite a bit :P. RuntimeCallStats::Dump easily ends up being around 80KB the
last time I measured (`nm --size-sort --print-size` is your friend).

[lpy, 2016-10-18 19:20:03]

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

[commit-bot: I haz the power, 2016-10-18 19:20:06]

Dry run: 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-10-18 19:21:35]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-18 19:21:36]

Dry run: Try jobs failed on following builders:
  v8_mac_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_mac_rel_ng/builds/11060)

[lpy, 2016-10-18 19:41:00]

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

[commit-bot: I haz the power, 2016-10-18 19:41:02]

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

[lpy, 2016-10-18 19:42:20]

Patchset #3 (id:40001) has been deleted

[lpy, 2016-10-18 19:42:28]

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

[commit-bot: I haz the power, 2016-10-18 19:42:33]

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

[lpy, 2016-10-18 19:43:45]

thanks, ptal.

https://codereview.chromium.org/2418303002/diff/1/src/counters.cc
File src/counters.cc (right):

https://codereview.chromium.org/2418303002/diff/1/src/counters.cc#newcode369
src/counters.cc:369: }
On 2016/10/18 07:52:24, Camillo Bruni wrote:
> Please leave the separate function in place as the macros blow up the binary
> size quite a bit :P. RuntimeCallStats::Dump easily ends up being around 80KB
the
> last time I measured (`nm --size-sort --print-size` is your friend).

Done.

[commit-bot: I haz the power, 2016-10-18 20:16:56]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-18 20:16:57]

Dry run: This issue passed the CQ dry run.

[Camillo Bruni, 2016-10-19 11:30:56]

thanks, lgtm again.

[lpy, 2016-10-19 14:51:58]

The CQ bit was checked by lpy@chromium.org

[commit-bot: I haz the power, 2016-10-19 14:52:05]

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-10-19 14:56:24]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2016-11-17 22:06:52]

Description was changed from

==========
Use TracedValue in runtime statistics.

We introduced TracedValue into V8 tracing previously, this patch uses it to
build JSON string of runtime statistics instead of using stringstream as buffer.

BUG=v8:5089
LOG=N
==========

to

==========
Use TracedValue in runtime statistics.

We introduced TracedValue into V8 tracing previously, this patch uses it to
build JSON string of runtime statistics instead of using stringstream as buffer.

BUG=v8:5089
LOG=N

Committed: https://crrev.com/65b3af466e9aea7bc451558d671fcd9866b6cbe3
Cr-Commit-Position: refs/heads/master@{#40443}
==========

[commit-bot: I haz the power, 2016-11-17 22:06:53]

Patchset 3 (id:??) landed as
https://crrev.com/65b3af466e9aea7bc451558d671fcd9866b6cbe3
Cr-Commit-Position: refs/heads/master@{#40443}