commit f178fa9ffc5af2dca40159ff6ba48ab51ff9fb9d
author Prabir Pradhan <prabirmsp@google.com> Fri May 10 16:57:37 2024 +0000
committer Prabir Pradhan <prabirmsp@google.com> Tue May 14 16:33:56 2024 +0000
tree 332f972bf2f362dca23b4fc78a8dd4e9d0b3b42c
parent 887bfb079414695383a377823386d00add266775

Introduce ArgsParser to write parsed protos to TraceStorage

The logic for interning parsed key strings and writing the key-value
pair into TraceStorage is duplicated several times. Introduce a new
class to consolidate the repeated code.

Bug: 332714237
Test: presubmit
Change-Id: I5de1ab3a1ddcfe3c1882a60c4aea5d678379ab01

Android.bp

diff --git a/Android.bp b/Android.bp
index b379ef7..ad476e1 100644
--- a/Android.bp
+++ b/Android.bp
@@ -12371,6 +12371,7 @@
     name: "perfetto_src_trace_processor_importers_proto_minimal",
     srcs: [
         "src/trace_processor/importers/proto/active_chrome_processes_tracker.cc",
+        "src/trace_processor/importers/proto/args_parser.cc",
         "src/trace_processor/importers/proto/chrome_string_lookup.cc",
         "src/trace_processor/importers/proto/chrome_system_probes_module.cc",
         "src/trace_processor/importers/proto/chrome_system_probes_parser.cc",
@@ -12436,7 +12437,6 @@
         "src/trace_processor/importers/proto/winscope/shell_transitions_tracker.cc",
         "src/trace_processor/importers/proto/winscope/surfaceflinger_layers_parser.cc",
         "src/trace_processor/importers/proto/winscope/surfaceflinger_transactions_parser.cc",
-        "src/trace_processor/importers/proto/winscope/winscope_args_parser.cc",
         "src/trace_processor/importers/proto/winscope/winscope_module.cc",
     ],
 }