[Tracing] Export TraceConfig to model.metadata

[Tracing] Export TraceConfig to model.metadata

BUG=catapult:#2336

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

[nednguyen, 2016-05-24 20:52:17]

nednguyen@google.com changed reviewers:
+ benjhayden@chromium.org, fmeawad@chromium.org

[nednguyen, 2016-05-24 20:52:28]

The CQ bit was checked by nednguyen@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-05-24 20:52:40]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2013553002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2013553002/1

[fmeawad, 2016-05-24 21:03:51]

https://codereview.chromium.org/2013553002/diff/1/tracing/tracing/extras/impo...
File tracing/tracing/extras/importer/trace_event_importer_test.html (right):

https://codereview.chromium.org/2013553002/diff/1/tracing/tracing/extras/impo...
tracing/tracing/extras/importer/trace_event_importer_test.html:5311: { name:
'TraceConfig', args: {value: traceConfigData },
Can you add a real example of a current TraceConfig so that we can set the
expectation on what is included?

nit: missing space ": { value"

[nednguyen, 2016-05-24 21:11:51]

https://codereview.chromium.org/2013553002/diff/1/tracing/tracing/extras/impo...
File tracing/tracing/extras/importer/trace_event_importer_test.html (right):

https://codereview.chromium.org/2013553002/diff/1/tracing/tracing/extras/impo...
tracing/tracing/extras/importer/trace_event_importer_test.html:5311: { name:
'TraceConfig', args: {value: traceConfigData },
On 2016/05/24 21:03:51, fmeawad wrote:
> Can you add a real example of a current TraceConfig so that we can set the
> expectation on what is included?
> 
> nit: missing space ": { value"

This is from a real example:
"{"pid":16662,"tid":16662,"ts":1107984591957,"ph":"M","cat":"__metadata","name":"TraceConfig","args":{"value":{"enable_argument_filter":false,"enable_sampling":false,"enable_systrace":false,"record_mode":"record-until-full"}},"tts":340738944}"

[fmeawad, 2016-05-24 21:17:47]

lgtm

[nednguyen, 2016-05-24 21:20:22]

The CQ bit was checked by nednguyen@google.com

[commit-bot: I haz the power, 2016-05-24 21:20:33]

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

[commit-bot: I haz the power, 2016-05-24 21:42:30]

Description was changed from

==========
[Tracing] Export TraceConfig to model.metadata

BUG=catapult:#2336
==========

to

==========
[Tracing] Export TraceConfig to model.metadata

BUG=catapult:#2336

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

[commit-bot: I haz the power, 2016-05-24 21:42:31]

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