tracing: Introduce mojo Interfaces

tracing: Introduce mojo Interfaces

Memory-infra IPCs were translated to mojo in a series of patches,
before. This CL introduces the mojo interface for all other parts,
except for background tracing, which will be dealt with in a separate
CL. For documentation of the interface, please refer to the comments
inside the mojom file.

BUG=640235
Review-Url: https://codereview.chromium.org/2723773002
Cr-Commit-Position: refs/heads/master@{#455800}
Committed: https://chromium.googlesource.com/chromium/src/+/474d0213e93f623d92acb750fcdcd48c60a34c6c

[chiniforooshan, 2017-02-28 18:59:37]

chiniforooshan@chromium.org changed reviewers:
+ fmeawad@google.com, oysteine@chromium.org

[chiniforooshan, 2017-02-28 18:59:38]

Hi Oystein, Fadi,

This is just a proposal for the eventual tracing mojo interface (background
tracing is excluded). I thought it would be easier to decide on the interface in
codereview, instead of docs/emails/etc. Could you please take a look. Thanks!

[chiniforooshan, 2017-02-28 19:14:06]

CC Primiano

[chiniforooshan, 2017-02-28 20:19:30]

+chrome-grc-reviews@

[chiniforooshan, 2017-03-06 15:23:36]

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

[chiniforooshan, 2017-03-06 15:23:37]

ping

[lpy, 2017-03-06 18:07:56]

Just curious, is there any doc about tracing servicification?

[chiniforooshan, 2017-03-06 18:19:38]

On 2017/03/06 18:07:56, lpy wrote:
> Just curious, is there any doc about tracing servicification?

There is this doc:
https://docs.google.com/a/google.com/document/d/1Mz64egjuZ4WsYw9AKKWdTvl0Il70...

I haven't update it very recently. Please feel free to comment on the doc if
something is not clear and I'll update that section.

[oystein (OOO til 10th of July), 2017-03-07 21:05:01]

Apologies for the late response; lgtm.

[oystein (OOO til 10th of July), 2017-03-07 21:05:01]

Apologies for the late response; lgtm.

[chiniforooshan, 2017-03-07 21:10:57]

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

[chiniforooshan, 2017-03-07 21:10:57]

Thanks!

+kenrb for security review of tracing.mojom.

[kenrb, 2017-03-09 16:37:31]

lgtm, bug please ensure there is security review for the interface
implementations even if there are no further changes to the mojom files.

[chiniforooshan, 2017-03-09 17:20:26]

On 2017/03/09 16:37:31, kenrb wrote:
> lgtm, bug please ensure there is security review for the interface
> implementations even if there are no further changes to the mojom files.

Acknowledged.

[chiniforooshan, 2017-03-09 17:20:45]

The CQ bit was checked by chiniforooshan@chromium.org

[commit-bot: I haz the power, 2017-03-09 17:21:10]

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-03-09 18:30:33]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1489080045657640, "parent_rev":
"af8c9798ce11b04fb6ad0fa8ddd5e7f0d7faf8af", "commit_rev":
"474d0213e93f623d92acb750fcdcd48c60a34c6c"}

[commit-bot: I haz the power, 2017-03-09 18:31:13]

Description was changed from

==========
tracing: Introduce mojo Interfaces

Memory-infra IPCs were translated to mojo in a series of patches,
before. This CL introduces the mojo interface for all other parts,
except for background tracing, which will be dealt with in a separate
CL. For documentation of the interface, please refer to the comments
inside the mojom file.

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

to

==========
tracing: Introduce mojo Interfaces

Memory-infra IPCs were translated to mojo in a series of patches,
before. This CL introduces the mojo interface for all other parts,
except for background tracing, which will be dealt with in a separate
CL. For documentation of the interface, please refer to the comments
inside the mojom file.

BUG=640235

Review-Url: https://codereview.chromium.org/2723773002
Cr-Commit-Position: refs/heads/master@{#455800}
Committed:
https://chromium.googlesource.com/chromium/src/+/474d0213e93f623d92acb750fcdc...
==========

[commit-bot: I haz the power, 2017-03-09 18:31:14]

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