Pablo Gamito

This commit **enhances the Android Window Manager shell transition tracing** by adding new debug fields, including geometry data via `RectProto`, to the `ShellTransition.Change` proto message. It also **refactors** the proto by renaming the `ShellTransition.targets` field to `ShellTransition.changes`. The **Trace Processor** is updated to parse these new fields from `shell_transition.proto` and expose them through the `android_window_manager_shell_transition_participants` SQL view. This **new capability** provides richer metadata for analyzing window transitions, allowing for more detailed debugging and analysis of UI state changes.

This commit introduces a **new field**, `animating_types`, to the `WindowStateProto` message within the **Perfetto tracing system**'s `protos/perfetto/trace/android/server/windowmanagerservice.proto` definition. This **enhancement** allows the **Android WindowManagerService** trace data to capture and report the specific types of animations currently active for a window. By extending the protocol buffer, Perfetto gains the ability to provide more granular insights into window animation states, improving the diagnostic capabilities for UI performance analysis.

This commit **fixes a crash** in the **ProtoLogMessageDecoder** within the trace processor's Winscope importer. Previously, processing ProtoLog messages with a mismatch between format specifiers and parameters would lead to a segmentation fault due to reading past parameter vectors. The change introduces **defensive checks** in the `Decode` function to validate parameter existence, preventing these crashes by using a `[MISSING_PARAM]` placeholder for missing parameters. A new statistic, `winscope_protolog_param_mismatch`, is also added to the **trace processor's statistics system** to track these occurrences, enhancing the robustness of trace analysis. This is a **bug fix** and a **defensive improvement** to prevent data processing failures.

This commit **exposes the `trace_processor` library to the Android build system**, enabling its direct use within C++ Android applications. It introduces a new `cc_library_static` target in `Android.bp` to compile `trace_processor` as a static library for Android. The `tools/gen_android_bp` script is also updated to ensure this new target is correctly included in the default and host-supported build configurations. This **new capability** significantly enhances **Android development** by allowing C++ code to easily integrate and leverage the powerful trace analysis and processing functionalities of `trace_processor` on Android devices.

This commit **downgrades the severity** of specific **ProtoLog parsing and decoding errors** within the **trace processor** from `kError` to `kInfo`. This **maintenance adjustment** addresses an issue where these errors were occurring too frequently in traces, leading to excessive data loss and noise. By treating these events as informational rather than critical, the change **improves the robustness of trace processing** and ensures more complete data collection by preventing premature termination or over-reporting of non-critical issues.

This commit **corrects** the classification of several **Winscope statistics** within the `trace_processor` component. It redefines various parsing and decoding issues, previously logged as informational messages, to be correctly classified as **errors**. This **bug fix** ensures that critical problems encountered during trace processing are accurately reported, improving the visibility and urgency of these issues for developers and users. The update specifically targets definitions in `src/trace_processor/storage/stats.h`, enhancing the reliability of error reporting for the **Winscope tracing tool**.

This commit performs a **documentation update** within the **Perfetto tracing system** configuration, specifically for **SysUI persistent tracing**. It **updates comments** in `persistent_cfg.pbtxt` to reflect the new, more descriptive name of a persistent tracing flag. The outdated flag `persist.debug.perfetto.persistent` is replaced with `persist.debug.perfetto.persistent_sysui_tracing_for_bugreport` in the documentation. This **maintenance** task ensures that developers and users refer to the correct and current flag name when configuring persistent tracing for bug reports, improving clarity and preventing misconfigurations.

This commit **fixes a critical bug** in the **`Winscope` module's `Trace Processor` importer** that caused incomplete `ShellTransition` data in `Flicker` and `Winscope` traces. Previously, `ArgsTracker` instances were misused, leading to argument data being prematurely overwritten. To resolve this, the **`ShellTransitionsTracker`** has been refactored to manage **dedicated `ArgsTracker` instances for each transition**, ensuring all arguments are correctly accumulated and flushed. This **bug fix** guarantees **complete and accurate `ShellTransition` data** for trace analysis tools.

This commit **updates the Perfetto tracing protocol definition** for Android's WindowManagerService. It **synchronizes** the `protos/perfetto/trace/android/server/windowmanagerservice.proto` file by adding `prepare_sync_seq_id` and `sync_seq_id` fields to the `WindowStateProto` message. This **feature enhancement** allows Perfetto to capture more granular data related to window state synchronization. The change improves the diagnostic capabilities for the **Android UI/windowing system** by providing new sequence IDs for tracing window management events.