Reland of [V8][Metric] Initial implementation of the RuntimeStatsMetric

Reland of [V8][Metric] Initial implementation of the RuntimeStatsMetric (patchset #1 id:1 of https://chromiumcodereview.appspot.com/2439953002/ )

Reason for revert:
The CL conflicted with the CL that landed right before it (https://github.com/catapult-project/catapult/commit/f5d3b3605c72ff72fce258c4d78a66e137688226)

Fixing the conflict and relanding.

Original issue's description:
> Revert of [V8][Metric] Initial implementation of the RuntimeStatsMetric (patchset #6 id:100001 of https://chromiumcodereview.appspot.com/2415963002/ )
> 
> Reason for revert:
> This is causing test failures on the CQ and locally.
> 
> Link to bot with CQ failures:
> https://build.chromium.org/p/tryserver.client.catapult/builders/Catapult%20Linux%20Tryserver/builds
> 
> From a local run on linux:
> 
> TypeError: tr.v.ValueSet is not a constructor
>     at Object.testFn_ (http://127.0.0.1:8003/tracing/metrics/v8/runtime_stats_metric_test.html:179:18)
>     at Object.run (http://127.0.0.1:8003/tracing/base/unittest/test_case.html:79:19)
>     at Object.<anonymous> (http://127.0.0.1:8003/tracing/base/unittest/test_runner.html:218:52)
>     at Object.runCurrentTestCase_ (http://127.0.0.1:8003/tracing/base/unittest/test_runner.html:216:14)
>     at Object.runOneTestCase_ (http://127.0.0.1:8003/tracing/base/unittest/test_runner.html:169:12)
> 
> Original issue's description:
> > [V8][Metric] Initial implementation of the RuntimeStatsMetric
> > 
> > This CL implements a runtime stats metric that extracts the runtime
> > stats info from the V8ThreadSlices up to interactive time.
> > Then these values are grouped by 'type'. Each type is a single value
> > histogram. Histograms are repeated for count and duration.
> > 
> > Each 'Type' histogram has a RelatedValueMap that describes the
> > individual runtime stats used to calculate that 'type' data.
> > 
> > BUG=catapult:#2938
> > 
> > Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/55c4800f013cba6cb2a4b77e8c49b643d56a8873
> 
> TBR=eakuefner@chromium.org,benjhayden@chromium.org,cbruni@chromium.org,lpy@chromium.org,nednguyen@google.com,fmeawad@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=catapult:#2938
> 
> Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/96f25d9aa22bdc48bb9e6fa417585fb150caa0a1

TBR=eakuefner@chromium.org,benjhayden@chromium.org,cbruni@chromium.org,lpy@chromium.org,nednguyen@google.com,aiolos@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=catapult:#2938

Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/a5f3fc1beeb59a31efc8e253df67587434ec6e64

[fmeawad, 2016-10-20 22:15:26]

Created Reland of [V8][Metric] Initial implementation of the RuntimeStatsMetric

[fmeawad, 2016-10-20 22:19:53]

Description was changed from

==========
Reland of [V8][Metric] Initial implementation of the RuntimeStatsMetric
(patchset #1 id:1 of https://chromiumcodereview.appspot.com/2439953002/ )

Reason for revert:
The CL conflicted with the CL that landed right before it
(https://github.com/catapult-project/catapult/commit/f5d3b3605c72ff72fce258c4d...)

Fixing the conflict and relanding.

Original issue's description:
> Revert of [V8][Metric] Initial implementation of the RuntimeStatsMetric
(patchset #6 id:100001 of https://chromiumcodereview.appspot.com/2415963002/ )
> 
> Reason for revert:
> This is causing test failures on the CQ and locally.
> 
> Link to bot with CQ failures:
>
https://build.chromium.org/p/tryserver.client.catapult/builders/Catapult%20Li...
> 
> From a local run on linux:
> 
> TypeError: tr.v.ValueSet is not a constructor
>     at Object.testFn_
(http://127.0.0.1:8003/tracing/metrics/v8/runtime_stats_metric_test.html:179:18)
>     at Object.run
(http://127.0.0.1:8003/tracing/base/unittest/test_case.html:79:19)
>     at Object.<anonymous>
(http://127.0.0.1:8003/tracing/base/unittest/test_runner.html:218:52)
>     at Object.runCurrentTestCase_
(http://127.0.0.1:8003/tracing/base/unittest/test_runner.html:216:14)
>     at Object.runOneTestCase_
(http://127.0.0.1:8003/tracing/base/unittest/test_runner.html:169:12)
> 
> Original issue's description:
> > [V8][Metric] Initial implementation of the RuntimeStatsMetric
> > 
> > This CL implements a runtime stats metric that extracts the runtime
> > stats info from the V8ThreadSlices up to interactive time.
> > Then these values are grouped by 'type'. Each type is a single value
> > histogram. Histograms are repeated for count and duration.
> > 
> > Each 'Type' histogram has a RelatedValueMap that describes the
> > individual runtime stats used to calculate that 'type' data.
> > 
> > BUG=catapult:#2938
> > 
> > Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
> 
>
TBR=eakuefner@chromium.org,benjhayden@chromium.org,cbruni@chromium.org,lpy@ch...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=catapult:#2938
> 
> Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...

TBR=eakuefner@chromium.org,benjhayden@chromium.org,cbruni@chromium.org,lpy@ch...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=catapult:#2938
==========

to

==========
Reland of [V8][Metric] Initial implementation of the RuntimeStatsMetric
(patchset #1 id:1 of https://chromiumcodereview.appspot.com/2439953002/ )

Reason for revert:
The CL conflicted with the CL that landed right before it
(https://github.com/catapult-project/catapult/commit/f5d3b3605c72ff72fce258c4d...)

TBR=benjhayden@chromium.org, eakuefner@chromium.org
Fixing the conflict and relanding.

Original issue's description:
> Revert of [V8][Metric] Initial implementation of the RuntimeStatsMetric
(patchset #6 id:100001 of https://chromiumcodereview.appspot.com/2415963002/ )
> 
> Reason for revert:
> This is causing test failures on the CQ and locally.
> 
> Link to bot with CQ failures:
>
https://build.chromium.org/p/tryserver.client.catapult/builders/Catapult%20Li...
> 
> From a local run on linux:
> 
> TypeError: tr.v.ValueSet is not a constructor
>     at Object.testFn_
(http://127.0.0.1:8003/tracing/metrics/v8/runtime_stats_metric_test.html:179:18)
>     at Object.run
(http://127.0.0.1:8003/tracing/base/unittest/test_case.html:79:19)
>     at Object.<anonymous>
(http://127.0.0.1:8003/tracing/base/unittest/test_runner.html:218:52)
>     at Object.runCurrentTestCase_
(http://127.0.0.1:8003/tracing/base/unittest/test_runner.html:216:14)
>     at Object.runOneTestCase_
(http://127.0.0.1:8003/tracing/base/unittest/test_runner.html:169:12)
> 
> Original issue's description:
> > [V8][Metric] Initial implementation of the RuntimeStatsMetric
> > 
> > This CL implements a runtime stats metric that extracts the runtime
> > stats info from the V8ThreadSlices up to interactive time.
> > Then these values are grouped by 'type'. Each type is a single value
> > histogram. Histograms are repeated for count and duration.
> > 
> > Each 'Type' histogram has a RelatedValueMap that describes the
> > individual runtime stats used to calculate that 'type' data.
> > 
> > BUG=catapult:#2938
> > 
> > Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
> 
>
TBR=eakuefner@chromium.org,benjhayden@chromium.org,cbruni@chromium.org,lpy@ch...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=catapult:#2938
> 
> Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...

TBR=eakuefner@chromium.org,benjhayden@chromium.org,cbruni@chromium.org,lpy@ch...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=catapult:#2938
==========

[fmeawad, 2016-10-20 22:20:28]

Description was changed from

==========
Reland of [V8][Metric] Initial implementation of the RuntimeStatsMetric
(patchset #1 id:1 of https://chromiumcodereview.appspot.com/2439953002/ )

Reason for revert:
The CL conflicted with the CL that landed right before it
(https://github.com/catapult-project/catapult/commit/f5d3b3605c72ff72fce258c4d...)

TBR=benjhayden@chromium.org, eakuefner@chromium.org
Fixing the conflict and relanding.

Original issue's description:
> Revert of [V8][Metric] Initial implementation of the RuntimeStatsMetric
(patchset #6 id:100001 of https://chromiumcodereview.appspot.com/2415963002/ )
> 
> Reason for revert:
> This is causing test failures on the CQ and locally.
> 
> Link to bot with CQ failures:
>
https://build.chromium.org/p/tryserver.client.catapult/builders/Catapult%20Li...
> 
> From a local run on linux:
> 
> TypeError: tr.v.ValueSet is not a constructor
>     at Object.testFn_
(http://127.0.0.1:8003/tracing/metrics/v8/runtime_stats_metric_test.html:179:18)
>     at Object.run
(http://127.0.0.1:8003/tracing/base/unittest/test_case.html:79:19)
>     at Object.<anonymous>
(http://127.0.0.1:8003/tracing/base/unittest/test_runner.html:218:52)
>     at Object.runCurrentTestCase_
(http://127.0.0.1:8003/tracing/base/unittest/test_runner.html:216:14)
>     at Object.runOneTestCase_
(http://127.0.0.1:8003/tracing/base/unittest/test_runner.html:169:12)
> 
> Original issue's description:
> > [V8][Metric] Initial implementation of the RuntimeStatsMetric
> > 
> > This CL implements a runtime stats metric that extracts the runtime
> > stats info from the V8ThreadSlices up to interactive time.
> > Then these values are grouped by 'type'. Each type is a single value
> > histogram. Histograms are repeated for count and duration.
> > 
> > Each 'Type' histogram has a RelatedValueMap that describes the
> > individual runtime stats used to calculate that 'type' data.
> > 
> > BUG=catapult:#2938
> > 
> > Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
> 
>
TBR=eakuefner@chromium.org,benjhayden@chromium.org,cbruni@chromium.org,lpy@ch...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=catapult:#2938
> 
> Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...

TBR=eakuefner@chromium.org,benjhayden@chromium.org,cbruni@chromium.org,lpy@ch...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=catapult:#2938
==========

to

==========
Reland of [V8][Metric] Initial implementation of the RuntimeStatsMetric
(patchset #1 id:1 of https://chromiumcodereview.appspot.com/2439953002/ )

Reason for revert:
The CL conflicted with the CL that landed right before it
(https://github.com/catapult-project/catapult/commit/f5d3b3605c72ff72fce258c4d...)

Fixing the conflict and relanding.

Original issue's description:
> Revert of [V8][Metric] Initial implementation of the RuntimeStatsMetric
(patchset #6 id:100001 of https://chromiumcodereview.appspot.com/2415963002/ )
> 
> Reason for revert:
> This is causing test failures on the CQ and locally.
> 
> Link to bot with CQ failures:
>
https://build.chromium.org/p/tryserver.client.catapult/builders/Catapult%20Li...
> 
> From a local run on linux:
> 
> TypeError: tr.v.ValueSet is not a constructor
>     at Object.testFn_
(http://127.0.0.1:8003/tracing/metrics/v8/runtime_stats_metric_test.html:179:18)
>     at Object.run
(http://127.0.0.1:8003/tracing/base/unittest/test_case.html:79:19)
>     at Object.<anonymous>
(http://127.0.0.1:8003/tracing/base/unittest/test_runner.html:218:52)
>     at Object.runCurrentTestCase_
(http://127.0.0.1:8003/tracing/base/unittest/test_runner.html:216:14)
>     at Object.runOneTestCase_
(http://127.0.0.1:8003/tracing/base/unittest/test_runner.html:169:12)
> 
> Original issue's description:
> > [V8][Metric] Initial implementation of the RuntimeStatsMetric
> > 
> > This CL implements a runtime stats metric that extracts the runtime
> > stats info from the V8ThreadSlices up to interactive time.
> > Then these values are grouped by 'type'. Each type is a single value
> > histogram. Histograms are repeated for count and duration.
> > 
> > Each 'Type' histogram has a RelatedValueMap that describes the
> > individual runtime stats used to calculate that 'type' data.
> > 
> > BUG=catapult:#2938
> > 
> > Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
> 
>
TBR=eakuefner@chromium.org,benjhayden@chromium.org,cbruni@chromium.org,lpy@ch...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=catapult:#2938
> 
> Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...

TBR=eakuefner@chromium.org,benjhayden@chromium.org,cbruni@chromium.org,lpy@ch...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=catapult:#2938
==========

[fmeawad, 2016-10-20 22:20:33]

The CQ bit was checked by fmeawad@chromium.org

[commit-bot: I haz the power, 2016-10-20 22:20:50]

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-20 22:21:07]

Description was changed from

==========
Reland of [V8][Metric] Initial implementation of the RuntimeStatsMetric
(patchset #1 id:1 of https://chromiumcodereview.appspot.com/2439953002/ )

Reason for revert:
The CL conflicted with the CL that landed right before it
(https://github.com/catapult-project/catapult/commit/f5d3b3605c72ff72fce258c4d...)

Fixing the conflict and relanding.

Original issue's description:
> Revert of [V8][Metric] Initial implementation of the RuntimeStatsMetric
(patchset #6 id:100001 of https://chromiumcodereview.appspot.com/2415963002/ )
> 
> Reason for revert:
> This is causing test failures on the CQ and locally.
> 
> Link to bot with CQ failures:
>
https://build.chromium.org/p/tryserver.client.catapult/builders/Catapult%20Li...
> 
> From a local run on linux:
> 
> TypeError: tr.v.ValueSet is not a constructor
>     at Object.testFn_
(http://127.0.0.1:8003/tracing/metrics/v8/runtime_stats_metric_test.html:179:18)
>     at Object.run
(http://127.0.0.1:8003/tracing/base/unittest/test_case.html:79:19)
>     at Object.<anonymous>
(http://127.0.0.1:8003/tracing/base/unittest/test_runner.html:218:52)
>     at Object.runCurrentTestCase_
(http://127.0.0.1:8003/tracing/base/unittest/test_runner.html:216:14)
>     at Object.runOneTestCase_
(http://127.0.0.1:8003/tracing/base/unittest/test_runner.html:169:12)
> 
> Original issue's description:
> > [V8][Metric] Initial implementation of the RuntimeStatsMetric
> > 
> > This CL implements a runtime stats metric that extracts the runtime
> > stats info from the V8ThreadSlices up to interactive time.
> > Then these values are grouped by 'type'. Each type is a single value
> > histogram. Histograms are repeated for count and duration.
> > 
> > Each 'Type' histogram has a RelatedValueMap that describes the
> > individual runtime stats used to calculate that 'type' data.
> > 
> > BUG=catapult:#2938
> > 
> > Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
> 
>
TBR=eakuefner@chromium.org,benjhayden@chromium.org,cbruni@chromium.org,lpy@ch...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=catapult:#2938
> 
> Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...

TBR=eakuefner@chromium.org,benjhayden@chromium.org,cbruni@chromium.org,lpy@ch...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=catapult:#2938
==========

to

==========
Reland of [V8][Metric] Initial implementation of the RuntimeStatsMetric
(patchset #1 id:1 of https://chromiumcodereview.appspot.com/2439953002/ )

Reason for revert:
The CL conflicted with the CL that landed right before it
(https://github.com/catapult-project/catapult/commit/f5d3b3605c72ff72fce258c4d...)

Fixing the conflict and relanding.

Original issue's description:
> Revert of [V8][Metric] Initial implementation of the RuntimeStatsMetric
(patchset #6 id:100001 of https://chromiumcodereview.appspot.com/2415963002/ )
> 
> Reason for revert:
> This is causing test failures on the CQ and locally.
> 
> Link to bot with CQ failures:
>
https://build.chromium.org/p/tryserver.client.catapult/builders/Catapult%20Li...
> 
> From a local run on linux:
> 
> TypeError: tr.v.ValueSet is not a constructor
>     at Object.testFn_
(http://127.0.0.1:8003/tracing/metrics/v8/runtime_stats_metric_test.html:179:18)
>     at Object.run
(http://127.0.0.1:8003/tracing/base/unittest/test_case.html:79:19)
>     at Object.<anonymous>
(http://127.0.0.1:8003/tracing/base/unittest/test_runner.html:218:52)
>     at Object.runCurrentTestCase_
(http://127.0.0.1:8003/tracing/base/unittest/test_runner.html:216:14)
>     at Object.runOneTestCase_
(http://127.0.0.1:8003/tracing/base/unittest/test_runner.html:169:12)
> 
> Original issue's description:
> > [V8][Metric] Initial implementation of the RuntimeStatsMetric
> > 
> > This CL implements a runtime stats metric that extracts the runtime
> > stats info from the V8ThreadSlices up to interactive time.
> > Then these values are grouped by 'type'. Each type is a single value
> > histogram. Histograms are repeated for count and duration.
> > 
> > Each 'Type' histogram has a RelatedValueMap that describes the
> > individual runtime stats used to calculate that 'type' data.
> > 
> > BUG=catapult:#2938
> > 
> > Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
> 
>
TBR=eakuefner@chromium.org,benjhayden@chromium.org,cbruni@chromium.org,lpy@ch...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=catapult:#2938
> 
> Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...

TBR=eakuefner@chromium.org,benjhayden@chromium.org,cbruni@chromium.org,lpy@ch...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=catapult:#2938

Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
==========

[commit-bot: I haz the power, 2016-10-20 22:21:08]

Committed patchset #2 (id:190001) as
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...