Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 2a72e610 authored by Treehugger Robot's avatar Treehugger Robot Committed by Android (Google) Code Review
Browse files

Merge "Remove PackageChangedBroadcastReported metrics." into main

parents 1f66a39f 1b426b5a
Loading
Loading
Loading
Loading
+0 −4
Original line number Diff line number Diff line
@@ -363,10 +363,6 @@ public final class BroadcastHelper {
        final boolean isForWholeApp = componentNames.contains(packageName);
        final String callingPackageNameForTrace = mContext.getPackageManager().getNameForUid(
                callingUidForTrace);

        PackageMetrics.reportPackageChangedBroadcast(callingUidForTrace, packageUid, isForWholeApp,
                reasonForTrace);

        if (isForWholeApp) {
            tracePackageChangedBroadcastEvent(true /* applyFlag */, reasonForTrace, packageName,
                    "<implicit>" /* targetPackageName */, "whole" /* targetComponent */,
+0 −31
Original line number Diff line number Diff line
@@ -621,13 +621,6 @@ public final class PackageMetrics {
                uid, componentOldState, componentNewState, isLauncher, isForWholeApp, callingUid);
    }

    public static void reportPackageChangedBroadcast(int callingUid, int targetUid,
            boolean isForWholeApp, String reason) {
        FrameworkStatsLog.write(FrameworkStatsLog.PACKAGE_CHANGED_BROADCAST_REPORTED,
                callingUid, targetUid,
                convertPackageChangedReasonStringToInteger(reason, isForWholeApp));
    }

    /**
     * Metrics for reporting what kind of reason to call the invalidation.
     */
@@ -635,30 +628,6 @@ public final class PackageMetrics {
        storeInvalidationMetrics(cacheType, invalidationReason);
    }

    private static int convertPackageChangedReasonStringToInteger(String reason,
            boolean isForWholeApp) {
        return switch (reason) {
            case STRING_COMPONENT_STATE_CHANGED -> isForWholeApp
                    ?
                    FrameworkStatsLog.PACKAGE_CHANGED_BROADCAST_REPORTED__REASON__PACKAGE_CHANGED_REASON_PACKAGE_STATE_CHANGED
                    : FrameworkStatsLog.PACKAGE_CHANGED_BROADCAST_REPORTED__REASON__PACKAGE_CHANGED_REASON_COMPONENT_STATE_CHANGED;
            case STRING_COMPONENT_LABEL_ICON_CHANGED ->
                    FrameworkStatsLog.PACKAGE_CHANGED_BROADCAST_REPORTED__REASON__PACKAGE_CHANGED_REASON_COMPONENT_LABEL_ICON_CHANGED;
            case STRING_RESET_COMPONENT_STATE_CHANGED ->
                    FrameworkStatsLog.PACKAGE_CHANGED_BROADCAST_REPORTED__REASON__PACKAGE_CHANGED_REASON_COMPONENT_STATE_RESET;
            case STRING_MIME_GROUP_CHANGED ->
                    FrameworkStatsLog.PACKAGE_CHANGED_BROADCAST_REPORTED__REASON__PACKAGE_CHANGED_REASON_MIME_GROUP_CHANGED;
            case STRING_OVERLAY_CHANGED ->
                    FrameworkStatsLog.PACKAGE_CHANGED_BROADCAST_REPORTED__REASON__PACKAGE_CHANGED_REASON_OVERLAY_CHANGED;
            case STRING_STATIC_SHARED_LIBRARY_CHANGED ->
                    FrameworkStatsLog.PACKAGE_CHANGED_BROADCAST_REPORTED__REASON__PACKAGE_CHANGED_REASON_STATIC_SHARED_LIBRARY_CHANGED;
            case STRING_TEST ->
                    FrameworkStatsLog.PACKAGE_CHANGED_BROADCAST_REPORTED__REASON__PACKAGE_CHANGED_REASON_TEST;
            default ->
                    FrameworkStatsLog.PACKAGE_CHANGED_BROADCAST_REPORTED__REASON__PACKAGE_CHANGED_REASON_UNSPECIFIED;
        };
    }

    private static void storeInvalidationMetrics(int cacheType, int invalidationReason) {
        synchronized (sInvalidationMetrics) {
            SparseIntArray invalidationMetrics = sInvalidationMetrics.get(cacheType);