tracing: remove sampling state profiler

tracing: remove sampling state profiler

As per discussion in [1] the tracing sampling state profiler is not used anymore.
The coverage of the TRACE_EVENT_SCOPED_SAMPLING_STATE macros became
poor over time and new alternatives came out (e.g., V8 sampling profiler).
The overall is sensible but should be re-approached in a more sustainable way.
More specifically, the new trace event filters could be leveraged to build a
sampling profiler on top of the existing TRACE_EVENT macros.

[1] https://groups.google.com/a/chromium.org/d/msg/tracing/E3K6qF1YMI8/wapmBei2AQAJ

BUG=

Committed: https://crrev.com/7bc681e5c39bda0926e77961273916dc90530558
Cr-Commit-Position: refs/heads/master@{#424212}

[Primiano Tucci (use gerrit), 2016-10-08 14:25:28]

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

[commit-bot: I haz the power, 2016-10-08 14:25:34]

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

[Primiano Tucci (use gerrit), 2016-10-08 14:29:22]

Description was changed from

==========
tracing: remove sampling state profiler

TODO description

BUG=
==========

to

==========
tracing: remove sampling state profiler

As per discussion in [1] the tracing sampling state profiler is not used
anymore.
The coverage of the TRACE_EVENT_SCOPED_SAMPLING_STATE macros became
poor over time and new alternatives came out (e.g., V8 sampling profiler).
The overall is sensible but should be re-approached in a more sustainable way.
More specifically, the new trace event filters could be leveraged to build a 
sampling profiler on top of the existing TRACE_EVENT macros.

[1]
https://groups.google.com/a/chromium.org/d/msg/tracing/E3K6qF1YMI8/wapmBei2AQAJ

BUG=
==========

[Primiano Tucci (use gerrit), 2016-10-08 14:29:22]

primiano@chromium.org changed reviewers:
+ haraken@chromium.org, oysteine@chromium.org

[commit-bot: I haz the power, 2016-10-08 14:38:37]

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

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

Dry run: Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)

[Primiano Tucci (use gerrit), 2016-10-08 15:40:08]

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

[commit-bot: I haz the power, 2016-10-08 15:40:14]

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-08 16:14:07]

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

[commit-bot: I haz the power, 2016-10-08 16:14:07]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Primiano Tucci (use gerrit), 2016-10-09 00:08:35]

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

[commit-bot: I haz the power, 2016-10-09 00:08:44]

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-09 01:39:05]

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

[commit-bot: I haz the power, 2016-10-09 01:39:06]

Dry run: This issue passed the CQ dry run.

[Primiano Tucci (use gerrit), 2016-10-09 21:49:17]

Allright, draining my flight queue

[haraken, 2016-10-10 01:46:27]

LGTM

[Primiano Tucci (use gerrit), 2016-10-10 16:41:21]

+alph FYI. I am pretty sure this should not interfere with your v8 profiler
work, but worth checking.

[oystein (OOO til 10th of July), 2016-10-10 17:00:35]

Another enthusiastic lgtm!

[Primiano Tucci (use gerrit), 2016-10-10 18:02:27]

The CQ bit was checked by primiano@chromium.org

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

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

[Primiano Tucci (use gerrit), 2016-10-10 18:07:34]

primiano@chromium.org changed reviewers:
+ brettw@chromium.org, pfeldman@chromium.org, thestig@chromium.org

[Primiano Tucci (use gerrit), 2016-10-10 18:07:35]

+pfeldman to stamp remaining diffs in content/ outside of content/tracing
+brettw for -1 line in tools/gn/bootstrap/bootstrap.py
+thestig for -2 lines in base/build.gn

[Lei Zhang, 2016-10-10 18:09:50]

On 2016/10/10 18:07:35, Primiano Tucci - slow til 10th wrote:
> +thestig for -2 lines in base/build.gn

lgtm

[commit-bot: I haz the power, 2016-10-10 18:12:22]

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

[commit-bot: I haz the power, 2016-10-10 18:12:24]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[brettw, 2016-10-10 18:20:21]

bootstrap lgtm (I'll fix owners for that file)

[alph, 2016-10-10 18:21:34]

alph@chromium.org changed reviewers:
+ alph@chromium.org

[alph, 2016-10-10 18:21:35]

Awesome! LGTM

[Primiano Tucci (use gerrit), 2016-10-10 18:30:25]

The CQ bit was checked by primiano@chromium.org

[commit-bot: I haz the power, 2016-10-10 18:30:51]

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-10 19:57:46]

Description was changed from

==========
tracing: remove sampling state profiler

As per discussion in [1] the tracing sampling state profiler is not used
anymore.
The coverage of the TRACE_EVENT_SCOPED_SAMPLING_STATE macros became
poor over time and new alternatives came out (e.g., V8 sampling profiler).
The overall is sensible but should be re-approached in a more sustainable way.
More specifically, the new trace event filters could be leveraged to build a 
sampling profiler on top of the existing TRACE_EVENT macros.

[1]
https://groups.google.com/a/chromium.org/d/msg/tracing/E3K6qF1YMI8/wapmBei2AQAJ

BUG=
==========

to

==========
tracing: remove sampling state profiler

As per discussion in [1] the tracing sampling state profiler is not used
anymore.
The coverage of the TRACE_EVENT_SCOPED_SAMPLING_STATE macros became
poor over time and new alternatives came out (e.g., V8 sampling profiler).
The overall is sensible but should be re-approached in a more sustainable way.
More specifically, the new trace event filters could be leveraged to build a 
sampling profiler on top of the existing TRACE_EVENT macros.

[1]
https://groups.google.com/a/chromium.org/d/msg/tracing/E3K6qF1YMI8/wapmBei2AQAJ

BUG=
==========

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

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-10-10 20:02:03]

Description was changed from

==========
tracing: remove sampling state profiler

As per discussion in [1] the tracing sampling state profiler is not used
anymore.
The coverage of the TRACE_EVENT_SCOPED_SAMPLING_STATE macros became
poor over time and new alternatives came out (e.g., V8 sampling profiler).
The overall is sensible but should be re-approached in a more sustainable way.
More specifically, the new trace event filters could be leveraged to build a 
sampling profiler on top of the existing TRACE_EVENT macros.

[1]
https://groups.google.com/a/chromium.org/d/msg/tracing/E3K6qF1YMI8/wapmBei2AQAJ

BUG=
==========

to

==========
tracing: remove sampling state profiler

As per discussion in [1] the tracing sampling state profiler is not used
anymore.
The coverage of the TRACE_EVENT_SCOPED_SAMPLING_STATE macros became
poor over time and new alternatives came out (e.g., V8 sampling profiler).
The overall is sensible but should be re-approached in a more sustainable way.
More specifically, the new trace event filters could be leveraged to build a
sampling profiler on top of the existing TRACE_EVENT macros.

[1]
https://groups.google.com/a/chromium.org/d/msg/tracing/E3K6qF1YMI8/wapmBei2AQAJ

BUG=

Committed: https://crrev.com/7bc681e5c39bda0926e77961273916dc90530558
Cr-Commit-Position: refs/heads/master@{#424212}
==========

[commit-bot: I haz the power, 2016-10-10 20:02:05]

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