Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Add package, activity count in allow list for ContentCapture logging" into tm-dev

Browse Source
This commit is contained in:
Joanne Chung
2022-05-13 11:05:57 +00:00
committed by Android (Google) Code Review
parent 7d1b14eb1f 5b9fe2f235
commit 790afd00ed
1 changed files with 2 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
services/contentcapture/java/com/android/server/contentcapture/ContentCaptureMetricsLogger.java
View File

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