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

Commit 73107044 authored by Muhammad Qureshi's avatar Muhammad Qureshi
Browse files

Use FrameworkStatsLog instead of StatsLog 

As part of statsd becoming a Mainline module in R, autogenerated
StatsLog.write() calls are going away and replaced by *StatsLog.java
that is autogenerated for each module.
This CL replaces some usages of StatsLog with FrameworkStatsLog.

Bug: 145952197
Test: m
Change-Id: Id3e0900a050441b9aef3ee6244759f5098afdfb1
parent 47c9ac72

core/java/com/android/internal/compat/ChangeReporter.java

+8 −8
Original line number Diff line number Diff line
@@ -18,10 +18,10 @@ package com.android.internal.compat; 


import android.util.Log;

import android.util.Slog;

import android.util.StatsLog;



import com.android.internal.annotations.GuardedBy;

import com.android.internal.annotations.VisibleForTesting;

import com.android.internal.util.FrameworkStatsLog;



import java.util.HashMap;

import java.util.HashSet;
@@ -85,8 +85,8 @@ public final class ChangeReporter { 
     */

    public void reportChange(int uid, long changeId, int state) {

        if (shouldWriteToStatsLog(uid, changeId, state)) {

            StatsLog.write(StatsLog.APP_COMPATIBILITY_CHANGE_REPORTED, uid, changeId,

                    state, mSource);

            FrameworkStatsLog.write(FrameworkStatsLog.APP_COMPATIBILITY_CHANGE_REPORTED, uid,

                    changeId, state, mSource);

        }

        if (shouldWriteToDebug(uid, changeId, state)) {

            debugLog(uid, changeId, state);
@@ -174,7 +174,7 @@ public final class ChangeReporter { 
    private void debugLog(int uid, long changeId, int state) {

        String message = String.format("Compat change id reported: %d; UID %d; state: %s", changeId,

                uid, stateToString(state));

        if (mSource == StatsLog.APP_COMPATIBILITY_CHANGE_REPORTED__SOURCE__SYSTEM_SERVER) {

        if (mSource == FrameworkStatsLog.APP_COMPATIBILITY_CHANGE_REPORTED__SOURCE__SYSTEM_SERVER) {

            Slog.d(TAG, message);

        } else {

            Log.d(TAG, message);
@@ -183,18 +183,18 @@ public final class ChangeReporter { 
    }



    /**

     * Transforms StatsLog.APP_COMPATIBILITY_CHANGE_REPORTED__STATE enum to a string.

     * Transforms FrameworkStatsLog.APP_COMPATIBILITY_CHANGE_REPORTED__STATE enum to a string.

     *

     * @param state to transform

     * @return a string representing the state

     */

    private static String stateToString(int state) {

        switch (state) {

            case StatsLog.APP_COMPATIBILITY_CHANGE_REPORTED__STATE__LOGGED:

            case FrameworkStatsLog.APP_COMPATIBILITY_CHANGE_REPORTED__STATE__LOGGED:

                return "LOGGED";

            case StatsLog.APP_COMPATIBILITY_CHANGE_REPORTED__STATE__ENABLED:

            case FrameworkStatsLog.APP_COMPATIBILITY_CHANGE_REPORTED__STATE__ENABLED:

                return "ENABLED";

            case StatsLog.APP_COMPATIBILITY_CHANGE_REPORTED__STATE__DISABLED:

            case FrameworkStatsLog.APP_COMPATIBILITY_CHANGE_REPORTED__STATE__DISABLED:

                return "DISABLED";

            default:

                return "UNKNOWN";

services/core/java/com/android/server/compat/PlatformCompat.java

+8 −7
Original line number Diff line number Diff line
@@ -30,7 +30,6 @@ import android.os.Binder; 
import android.os.RemoteException;

import android.os.UserHandle;

import android.util.Slog;

import android.util.StatsLog;



import com.android.internal.annotations.VisibleForTesting;

import com.android.internal.compat.AndroidBuildClassifier;
@@ -40,6 +39,7 @@ import com.android.internal.compat.CompatibilityChangeInfo; 
import com.android.internal.compat.IOverrideValidator;

import com.android.internal.compat.IPlatformCompat;

import com.android.internal.util.DumpUtils;

import com.android.internal.util.FrameworkStatsLog;

import com.android.server.LocalServices;



import java.io.FileDescriptor;
@@ -59,7 +59,7 @@ public class PlatformCompat extends IPlatformCompat.Stub { 
    public PlatformCompat(Context context) {

        mContext = context;

        mChangeReporter = new ChangeReporter(

                StatsLog.APP_COMPATIBILITY_CHANGE_REPORTED__SOURCE__SYSTEM_SERVER);

                FrameworkStatsLog.APP_COMPATIBILITY_CHANGE_REPORTED__SOURCE__SYSTEM_SERVER);

        mCompatConfig = CompatConfig.create(new AndroidBuildClassifier(), mContext);

    }
@@ -67,7 +67,7 @@ public class PlatformCompat extends IPlatformCompat.Stub { 
    PlatformCompat(Context context, CompatConfig compatConfig) {

        mContext = context;

        mChangeReporter = new ChangeReporter(

                StatsLog.APP_COMPATIBILITY_CHANGE_REPORTED__SOURCE__SYSTEM_SERVER);

                FrameworkStatsLog.APP_COMPATIBILITY_CHANGE_REPORTED__SOURCE__SYSTEM_SERVER);

        mCompatConfig = compatConfig;

    }
@@ -75,7 +75,7 @@ public class PlatformCompat extends IPlatformCompat.Stub { 
    public void reportChange(long changeId, ApplicationInfo appInfo) {

        checkCompatChangeLogPermission();

        reportChange(changeId, appInfo.uid,

                StatsLog.APP_COMPATIBILITY_CHANGE_REPORTED__STATE__LOGGED);

                FrameworkStatsLog.APP_COMPATIBILITY_CHANGE_REPORTED__STATE__LOGGED);

    }



    @Override
@@ -91,7 +91,8 @@ public class PlatformCompat extends IPlatformCompat.Stub { 
    @Override

    public void reportChangeByUid(long changeId, int uid) {

        checkCompatChangeLogPermission();

        reportChange(changeId, uid, StatsLog.APP_COMPATIBILITY_CHANGE_REPORTED__STATE__LOGGED);

        reportChange(changeId, uid,

                FrameworkStatsLog.APP_COMPATIBILITY_CHANGE_REPORTED__STATE__LOGGED);

    }



    @Override
@@ -99,11 +100,11 @@ public class PlatformCompat extends IPlatformCompat.Stub { 
        checkCompatChangeReadPermission();

        if (mCompatConfig.isChangeEnabled(changeId, appInfo)) {

            reportChange(changeId, appInfo.uid,

                    StatsLog.APP_COMPATIBILITY_CHANGE_REPORTED__STATE__ENABLED);

                    FrameworkStatsLog.APP_COMPATIBILITY_CHANGE_REPORTED__STATE__ENABLED);

            return true;

        }

        reportChange(changeId, appInfo.uid,

                StatsLog.APP_COMPATIBILITY_CHANGE_REPORTED__STATE__DISABLED);

                FrameworkStatsLog.APP_COMPATIBILITY_CHANGE_REPORTED__STATE__DISABLED);

        return false;

    }