Delete the tracing service

Delete the tracing service

This tracing service is not used by any one, AFAIK, and I am creating
a new tracing service that is going to replace the tracing controller
that currently exists in content/browser/tracing.

To get an idea of the new service, please take a look at the prototype:
https://codereview.chromium.org/2833873003/

BUG=640235
Review-Url: https://codereview.chromium.org/2852183002
Cr-Commit-Position: refs/heads/master@{#468684}
Committed: https://chromium.googlesource.com/chromium/src/+/50510ac8fcc50e132c27edbb80d759cec829e47a

[chiniforooshan, 2017-05-01 21:50:23]

Elliot: for //components and //services/tracing
Tom: for the two .mojom files
Scott: for the rest

Thanks a lot!

[chiniforooshan, 2017-05-01 21:52:34]

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

[commit-bot: I haz the power, 2017-05-01 21:53:20]

Dry run: CQ is trying da patch.

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

[chiniforooshan, 2017-05-01 21:53:52]

chiniforooshan@chromium.org changed reviewers:
+ erg@chromium.org, sky@chromium.org, tsepez@chromium.org

[chiniforooshan, 2017-05-01 21:53:52]

oops, forgot to actually add reviewers. PTAL; thanks.

[Tom Sepez, 2017-05-01 21:57:32]

RS LGTM on deleting .mojom files.

[Elliot Glaysher, 2017-05-01 21:59:30]

lgtm and good riddance. thank you for doing this and good luck with the
replacement.

[commit-bot: I haz the power, 2017-05-01 23:21:55]

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

[commit-bot: I haz the power, 2017-05-01 23:21:56]

Dry run: This issue passed the CQ dry run.

[sky, 2017-05-01 23:37:24]

LGTM

[chiniforooshan, 2017-05-02 14:20:54]

The CQ bit was checked by chiniforooshan@chromium.org

[commit-bot: I haz the power, 2017-05-02 14:21:11]

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, 2017-05-02 14:32:59]

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

[commit-bot: I haz the power, 2017-05-02 14:33:04]

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...)

[chiniforooshan, 2017-05-02 14:40:14]

chiniforooshan@chromium.org changed reviewers:
+ rockot@chromium.org

[chiniforooshan, 2017-05-02 14:40:15]

+Ken for //services/tracing (thought an lgtm from erg@ would cover this, but it
won't). Thank you.

Missing LGTM from an OWNER for these files:
    services/tracing/BUILD.gn
    services/tracing/OWNERS
    services/tracing/data_sink.cc
    services/tracing/data_sink.h
    services/tracing/main.cc
    services/tracing/manifest.json
    services/tracing/public/cpp/BUILD.gn
    services/tracing/public/cpp/provider.cc
    services/tracing/public/cpp/provider.h
    services/tracing/public/cpp/switches.cc
    services/tracing/public/cpp/switches.h
    services/tracing/public/interfaces/BUILD.gn
    services/tracing/public/interfaces/OWNERS
    services/tracing/public/interfaces/constants.mojom
    services/tracing/public/interfaces/tracing.mojom
    services/tracing/recorder.cc
    services/tracing/recorder.h
    services/tracing/service.cc
    services/tracing/service.h

[Ken Rockot(use gerrit already), 2017-05-02 16:49:28]

On 2017/05/02 at 14:40:15, chiniforooshan wrote:
> +Ken for //services/tracing (thought an lgtm from erg@ would cover this, but
it won't). Thank you.
> 
> Missing LGTM from an OWNER for these files:
>     services/tracing/BUILD.gn
>     services/tracing/OWNERS
>     services/tracing/data_sink.cc
>     services/tracing/data_sink.h
>     services/tracing/main.cc
>     services/tracing/manifest.json
>     services/tracing/public/cpp/BUILD.gn
>     services/tracing/public/cpp/provider.cc
>     services/tracing/public/cpp/provider.h
>     services/tracing/public/cpp/switches.cc
>     services/tracing/public/cpp/switches.h
>     services/tracing/public/interfaces/BUILD.gn
>     services/tracing/public/interfaces/OWNERS
>     services/tracing/public/interfaces/constants.mojom
>     services/tracing/public/interfaces/tracing.mojom
>     services/tracing/recorder.cc
>     services/tracing/recorder.h
>     services/tracing/service.cc
>     services/tracing/service.h

Weird. I guess because it removes the OWNERS file, its contents are ignored in
the owners check. LGTM!

[chiniforooshan, 2017-05-02 16:50:52]

The CQ bit was checked by chiniforooshan@chromium.org

[commit-bot: I haz the power, 2017-05-02 16:51:23]

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, 2017-05-02 16:58:40]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493743852544340, "parent_rev":
"31a45636e84fdf6cf56cac35ece3604baddf9174", "commit_rev":
"50510ac8fcc50e132c27edbb80d759cec829e47a"}

[commit-bot: I haz the power, 2017-05-02 16:58:50]

Description was changed from

==========
Delete the tracing service

This tracing service is not used by any one, AFAIK, and I am creating
a new tracing service that is going to replace the tracing controller
that currently exists in content/browser/tracing.

To get an idea of the new service, please take a look at the prototype:
https://codereview.chromium.org/2833873003/

BUG=640235
==========

to

==========
Delete the tracing service

This tracing service is not used by any one, AFAIK, and I am creating
a new tracing service that is going to replace the tracing controller
that currently exists in content/browser/tracing.

To get an idea of the new service, please take a look at the prototype:
https://codereview.chromium.org/2833873003/

BUG=640235

Review-Url: https://codereview.chromium.org/2852183002
Cr-Commit-Position: refs/heads/master@{#468684}
Committed:
https://chromium.googlesource.com/chromium/src/+/50510ac8fcc50e132c27edbb80d7...
==========

[commit-bot: I haz the power, 2017-05-02 16:58:52]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/50510ac8fcc50e132c27edbb80d7...