sdk: presubmit fixups for the C SDK shrink stack

Run tools/format-sources, tools/gen_bazel, tools/gen_android_bp on the
C-SDK-shrink stack output:

  - tools/gn format applied to the BUILD.gn files touched by the
    refactor (one trailing-newline / wrap fixup each in
    include/perfetto/ext/tracing/{core,ipc}/BUILD.gn,
    protos/perfetto/trace/BUILD.gn, src/protozero/filtering/BUILD.gn,
    src/tracing/BUILD.gn).
  - Android.bp regenerated to reflect the new sibling targets
    (base_minimal, client_api_minimal, tracing_minimal, core_minimal,
    service_minimal, the trace zero/cpp/lite library splits, etc.).
  - BUILD (Bazel) regenerated for the same.

tools/run_presubmit is clean after this commit.

Author: dreveman

Android.bp

@@ -46,6 +46,7 @@ source_set("core_minimal") {
 source_set("core") {
   public_deps = [
     ":core_minimal",
+
     # Full proto / include surface that consumers (trace_processor,
     # perfetto_cmd, etc.) expect transitively from this header set.
     "../../../../../protos/perfetto/common:cpp",

BUILD

@@ -15,6 +15,7 @@
 source_set("ipc") {
   public_deps = [
     "../../base",
+
     # Pull the slim core variant so the C SDK closure (which reaches
     # this target via the system_backend -> tracing/ipc:common ->
     # ext/tracing/ipc chain) doesn't transitively pull

include/perfetto/ext/tracing/core/BUILD.gn

@@ -76,19 +76,19 @@ minimal_proto_sources = [
 ]
 
 non_minimal_core_proto_sources = [
+  "extension_descriptor.proto",
   "remote_clock_sync.proto",
-  "trace_packet_defaults.proto",
-  "trace_packet.proto",
   "trace.proto",
-  "extension_descriptor.proto",
+  "trace_packet.proto",
+  "trace_packet_defaults.proto",
 ]
 
 non_minimal_extras_proto_sources = [
+  "evdev.proto",
+  "memory_graph.proto",
   "test_event.proto",
   "test_extensions.proto",
-  "memory_graph.proto",
   "ui_state.proto",
-  "evdev.proto",
 ]
 
 # Per-domain trace proto sub-deps that trace_packet.proto's cppgen output
@@ -117,6 +117,7 @@ non_minimal_per_domain_proto_deps = [
 
 perfetto_proto_library("minimal_@TYPE@") {
   sources = minimal_proto_sources
+
   # Only common/builtin_clock.proto is imported (by clock_snapshot.proto);
   # depending on the slim config aggregator avoids leaking the full
   # config closure into trace:zero_minimal -> //src/shared_lib.
@@ -127,6 +128,7 @@ perfetto_proto_library("non_minimal_core_@TYPE@") {
   sources = non_minimal_core_proto_sources
   deps = [
     ":minimal_@TYPE@",
+
     # Only trace_config (minimal) is imported transitively from this
     # target's sources; depending on the slim config aggregator keeps
     # the per-data-source config sub-deps out of the C SDK closure
@@ -150,8 +152,7 @@ perfetto_proto_library("non_minimal_extras_@TYPE@") {
 # per-domain fields can resolve without pulling per-domain deps into the
 # zero-only :non_minimal_core_@TYPE@ that the C SDK closure depends on.
 perfetto_proto_library("non_minimal_full_cpp_only_@TYPE@") {
-  sources =
-      non_minimal_core_proto_sources + non_minimal_extras_proto_sources
+  sources = non_minimal_core_proto_sources + non_minimal_extras_proto_sources
   deps = non_minimal_per_domain_proto_deps + [
            "../config:@TYPE@",
            "interned_data:@TYPE@",
@@ -165,8 +166,7 @@ perfetto_proto_library("non_minimal_full_cpp_only_@TYPE@") {
 }
 
 perfetto_proto_library("lite_only_@TYPE@") {
-  sources =
-      non_minimal_core_proto_sources + non_minimal_extras_proto_sources
+  sources = non_minimal_core_proto_sources + non_minimal_extras_proto_sources
   deps = non_minimal_per_domain_proto_deps + [
            "../config:@TYPE@",
            "interned_data:@TYPE@",

include/perfetto/ext/tracing/ipc/BUILD.gn

@@ -42,6 +42,7 @@ source_set("message_filter_config") {
     ":message_filter",
     ":string_filter",
     "../../../gn:default_deps",
+
     # Only TraceConfig / TracePerfettoConfig types are used; the
     # per-data-source config sub-deps that come with the full :cpp
     # group aren't needed here, and depending on :minimal_cpp keeps

protos/perfetto/trace/BUILD.gn

@@ -95,6 +95,7 @@ source_set("client_api_without_backends_minimal") {
   public_deps = [
     "../../gn:default_deps",
     "../../include/perfetto/tracing:tracing_minimal",
+
     # The .h surfaces of these proto deps are reachable from the
     # console_interceptor.cc / track_event_*.cc sources in the
     # :client_api_without_backends wrapper that depends on this target,