diff --git a/services/contentcapture/java/com/android/server/contentcapture/ContentCaptureMetricsLogger.java b/services/contentcapture/java/com/android/server/contentcapture/ContentCaptureMetricsLogger.java index db3db4ec01494..10bec64936b38 100644 --- a/services/contentcapture/java/com/android/server/contentcapture/ContentCaptureMetricsLogger.java +++ b/services/contentcapture/java/com/android/server/contentcapture/ContentCaptureMetricsLogger.java @@ -37,7 +37,7 @@ public final class ContentCaptureMetricsLogger { public static void writeServiceEvent(int eventType, @NonNull String serviceName) { // we should not logging the application package name FrameworkStatsLog.write(FrameworkStatsLog.CONTENT_CAPTURE_SERVICE_EVENTS, eventType, - serviceName, /* componentName= */ null); + serviceName, /* componentName= */ null, 0, 0); } /** @hide */ @@ -49,14 +49,13 @@ public final class ContentCaptureMetricsLogger { public static void writeSetWhitelistEvent(@Nullable ComponentName service, @Nullable List packages, @Nullable List activities) { final String serviceName = ComponentName.flattenToShortString(service); - // TODO: log allow list count. int packageCount = packages != null ? packages.size() : 0; int activityCount = activities != null ? activities.size() : 0; // we should not logging the application package name // log the allow list package and activity count instead FrameworkStatsLog.write(FrameworkStatsLog.CONTENT_CAPTURE_SERVICE_EVENTS, FrameworkStatsLog.CONTENT_CAPTURE_SERVICE_EVENTS__EVENT__SET_WHITELIST, - serviceName, /* allowListStr= */ null); + serviceName, /* allowListStr= */ null, packageCount, activityCount); } /** @hide */