arc: enable Android tracing in verified-boot mode

components/arc/common/trace.mojom

Index: components/arc/common/trace.mojom
diff --git a/components/arc/common/trace.mojom b/components/arc/common/trace.mojom
new file mode 100644
index 0000000000000000000000000000000000000000..3b634dd0a4be762aa26d4164c453fd95f8e01bdb
--- /dev/null
+++ b/components/arc/common/trace.mojom
@@ -0,0 +1,16 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+

[Yusuke Sato, 2016/10/12 05:58:13]

// Next MinVersion: 1

[shunhsingou, 2016/12/22 04:04:34]

Done.

+module arc.mojom;
+
+interface TraceHost {

[Yusuke Sato, 2016/10/12 05:58:13]

remove line 7-8? I'm not sure defining an empty host makes sense.

[shunhsingou, 2016/12/22 04:04:34]

Removed

+};
+
+interface TraceInstance {

[Yusuke Sato, 2016/10/12 05:58:13]

Mind adding comments to the interface and all methods? That would help the
security review etc.

[shunhsingou, 2016/12/22 04:04:34]

Done.

+  StartTracing@0(array<string> categories) => (bool success);

[Luis Héctor Chávez, 2016/10/12 20:32:07]

Is there any chance the TraceInstance will want to call any method of the
TraceHost in the future? If so, it might be a good idea to declare

Init@0(TraceHost host);

just like the other interfaces. If there is exactly 0% chance, let's get rid of
TraceHost altogether.

[shunhsingou, 2016/12/22 04:04:34]

Removed

+
+  StopTracing@1() => (bool success);
+
+  QueryAvailableCategories@2() => (array<string> categories);

[Yusuke Sato, 2016/10/12 05:58:13]

sort

[shunhsingou, 2016/12/22 04:04:34]

Done.

+};