commit 3970dfd2185dea9c67452fb0caceaea7e1a81b3e
author Primiano Tucci <primiano@google.com> Tue Mar 28 13:02:27 2023 +0100
committer Primiano Tucci <primiano@google.com> Tue Mar 28 12:03:21 2023 +0000
tree a30708ef1d1b828ef0a9d79bd7fb9ae3322ad199
parent c9968957f0f9aac66be5d2c6c4116bfcaea42c6d

perfetto_cmd: use CloneSession for --save-for-bugreport

Moves over the --save-for-bugreport cmdline option
to use the new CloneSession() IPC.
This moves the file writing from the tracing service
to the cmdline client.
Upcming CLs will remove the old
SaveTraceForBugreport IPC.

Bug: 260112703
Test: (1) perfetto_integrationtests --gtest_filter=PerfettoCmdlineTest.SaveForBugreport*
Test: (2) manual:
 1. adb shell perfetto --background -c - --txt < bugreport.cfg
 2. adb bugreport br.zip and check that the br contains a trace.

Change-Id: I34d1b2bb8c21aeedc24936fc48015fa38bb998c7

Android.bp

diff --git a/Android.bp b/Android.bp
index ad54f8d..55c57b0 100644
--- a/Android.bp
+++ b/Android.bp
@@ -1005,6 +1005,7 @@
         ":perfetto_src_base_version",
         ":perfetto_src_ipc_client",
         ":perfetto_src_ipc_common",
+        ":perfetto_src_perfetto_cmd_bugreport_path",
         ":perfetto_src_perfetto_cmd_perfetto_cmd",
         ":perfetto_src_perfetto_cmd_protos_cpp_gen",
         ":perfetto_src_perfetto_cmd_trigger_producer",
@@ -1191,6 +1192,7 @@
         ":perfetto_src_ipc_perfetto_ipc",
         ":perfetto_src_kallsyms_kallsyms",
         ":perfetto_src_kernel_utils_syscall_table",
+        ":perfetto_src_perfetto_cmd_bugreport_path",
         ":perfetto_src_protozero_filtering_bytecode_common",
         ":perfetto_src_protozero_filtering_bytecode_generator",
         ":perfetto_src_protozero_filtering_bytecode_parser",
@@ -1990,6 +1992,7 @@
         ":perfetto_src_ipc_perfetto_ipc",
         ":perfetto_src_kallsyms_kallsyms",
         ":perfetto_src_kernel_utils_syscall_table",
+        ":perfetto_src_perfetto_cmd_bugreport_path",
         ":perfetto_src_profiling_common_callstack_trie",
         ":perfetto_src_profiling_common_interner",
         ":perfetto_src_profiling_common_interning_output",
@@ -8467,6 +8470,11 @@
     ],
 }
 
+// GN: //src/perfetto_cmd:bugreport_path
+filegroup {
+    name: "perfetto_src_perfetto_cmd_bugreport_path",
+}
+
 // GN: //src/perfetto_cmd:gen_cc_config_descriptor
 genrule {
     name: "perfetto_src_perfetto_cmd_gen_cc_config_descriptor",
@@ -11623,6 +11631,7 @@
         ":perfetto_src_kallsyms_kallsyms",
         ":perfetto_src_kallsyms_unittests",
         ":perfetto_src_kernel_utils_syscall_table",
+        ":perfetto_src_perfetto_cmd_bugreport_path",
         ":perfetto_src_perfetto_cmd_perfetto_cmd",
         ":perfetto_src_perfetto_cmd_protos_cpp_gen",
         ":perfetto_src_perfetto_cmd_trigger_producer",