Convert components/tracing from static lib to component

Convert components/tracing from static lib to component

This is a prerequisite for https://crrev.com/1292583002.

BUG=518823
Committed: https://crrev.com/da965e85fdb2008392c3d313a04a67f8e794ddad
Cr-Commit-Position: refs/heads/master@{#343700}

[Ruud van Asseldonk, 2015-08-17 13:09:39]

ruuda@google.com changed reviewers:
+ dsinclair@chromium.org, primiano@chromium.org

[dsinclair, 2015-08-17 13:39:07]

lgtm

[Primiano Tucci (use gerrit), 2015-08-17 13:51:21]

LGTM with 1 minor comment. Many thanks for doing this.

https://codereview.chromium.org/1298683002/diff/1/components/BUILD.gn
File components/BUILD.gn (right):

https://codereview.chromium.org/1298683002/diff/1/components/BUILD.gn#newcode467
components/BUILD.gn:467: "//components/tracing",
I'd probably move this to the other CL that is introducing the test

https://codereview.chromium.org/1298683002/diff/1/components/components_tests...
File components/components_tests.gyp (right):

https://codereview.chromium.org/1298683002/diff/1/components/components_tests...
components/components_tests.gyp:1435: 'tracing.gyp:tracing',
same here

[Ruud van Asseldonk, 2015-08-17 14:58:38]

https://codereview.chromium.org/1298683002/diff/1/components/BUILD.gn
File components/BUILD.gn (right):

https://codereview.chromium.org/1298683002/diff/1/components/BUILD.gn#newcode467
components/BUILD.gn:467: "//components/tracing",
On 2015/08/17 13:51:21, Primiano Tucci wrote:
> I'd probably move this to the other CL that is introducing the test

Done.

https://codereview.chromium.org/1298683002/diff/1/components/components_tests...
File components/components_tests.gyp (right):

https://codereview.chromium.org/1298683002/diff/1/components/components_tests...
components/components_tests.gyp:1435: 'tracing.gyp:tracing',
On 2015/08/17 13:51:21, Primiano Tucci wrote:
> same here

Done.

[Ruud van Asseldonk, 2015-08-17 14:59:36]

The CQ bit was checked by ruuda@google.com

[Ruud van Asseldonk, 2015-08-17 14:59:36]

The patchset sent to the CQ was uploaded after l-g-t-m from
primiano@chromium.org, dsinclair@chromium.org
Link to the patchset: https://codereview.chromium.org/1298683002/#ps20001
(title: "Fix review comments")

[commit-bot: I haz the power, 2015-08-17 14:59:40]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1298683002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1298683002/20001

[commit-bot: I haz the power, 2015-08-17 15:14:40]

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

[commit-bot: I haz the power, 2015-08-17 15:14:41]

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

[Ruud van Asseldonk, 2015-08-17 15:32:36]

ruuda@google.com changed reviewers:
+ dcheng@chromium.org

[Ruud van Asseldonk, 2015-08-17 15:32:36]

+dcheng: for owner stamp tracing_messages.h

Can you please take a look at three-line change? This only adds an include, the
IPC is left untouched.

[blundell, 2015-08-17 15:35:52]

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

[blundell, 2015-08-17 15:35:52]

//components lgtm

[dcheng, 2015-08-17 16:20:34]

IPC changes LGTM

[Ruud van Asseldonk, 2015-08-17 16:24:13]

The CQ bit was checked by ruuda@google.com

[commit-bot: I haz the power, 2015-08-17 16:24:21]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1298683002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1298683002/20001

[commit-bot: I haz the power, 2015-08-17 17:02:06]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-08-17 17:02:47]

Patchset 2 (id:??) landed as
https://crrev.com/da965e85fdb2008392c3d313a04a67f8e794ddad
Cr-Commit-Position: refs/heads/master@{#343700}