commit cfeb079df151643cbdf42c5deb5f2848c9cd5859
author Danae Savvidi <dsavvidi@google.com> Wed Aug 20 17:57:30 2025 +0000
committer Danae Savvidi <dsavvidi@google.com> Wed Aug 20 17:57:30 2025 +0000
tree 50679a623862bb34b9c74dc6a3d640c0f5ca350f
parent 07d74c4cbe00360e27701a5b210c8032c39defe5

Update logging line to add client_count in DisplayEventCallbackOccurred atom

To log number of unique clients that received the event.

Bug: 433483856
Flag: com.android.server.display.feature.flags.enable_logging_for_display_events
Test: atest DisplayServiceTests
Test: atest DisplayFrameworkStatsLoggerTest
Change-Id: Ic3214257261aa5cf49515f61c30d894e2e600a1a

services/core/java/com/android/server/display/DisplayFrameworkStatsLogger.java

diff --git a/services/core/java/com/android/server/display/DisplayFrameworkStatsLogger.java b/services/core/java/com/android/server/display/DisplayFrameworkStatsLogger.java
index fde6964..98f97ef 100644
--- a/services/core/java/com/android/server/display/DisplayFrameworkStatsLogger.java
+++ b/services/core/java/com/android/server/display/DisplayFrameworkStatsLogger.java
@@ -30,7 +30,8 @@
         FrameworkStatsLog.write(
                 FrameworkStatsLog.DISPLAY_EVENT_CALLBACK_OCCURRED,
                 toProtoEventType(event),
-                notifiedUids.copyKeys());
+                notifiedUids.copyKeys(),
+                notifiedUids.size());
     }
 
     /**

services/tests/displayservicetests/src/com/android/server/display/DisplayFrameworkStatsLoggerTest.java

diff --git a/services/tests/displayservicetests/src/com/android/server/display/DisplayFrameworkStatsLoggerTest.java b/services/tests/displayservicetests/src/com/android/server/display/DisplayFrameworkStatsLoggerTest.java
index 61c43da..6b9bbc5 100644
--- a/services/tests/displayservicetests/src/com/android/server/display/DisplayFrameworkStatsLoggerTest.java
+++ b/services/tests/displayservicetests/src/com/android/server/display/DisplayFrameworkStatsLoggerTest.java
@@ -70,7 +70,7 @@
                 .write(
                         FrameworkStatsLog.DISPLAY_EVENT_CALLBACK_OCCURRED,
                         expectedProtoType,
-                        uidMap.copyKeys());
+                        uidMap.copyKeys(), 2);
     }
 
     @Test
@@ -92,7 +92,7 @@
                 .write(
                         FrameworkStatsLog.DISPLAY_EVENT_CALLBACK_OCCURRED,
                         expectedProtoType,
-                        uidMap.copyKeys());
+                        uidMap.copyKeys(), 1);
     }
 
     @Test
@@ -113,6 +113,6 @@
                 .write(
                         FrameworkStatsLog.DISPLAY_EVENT_CALLBACK_OCCURRED,
                         expectedProtoType,
-                        uidMap.copyKeys());
+                        uidMap.copyKeys(), 1);
     }
 }