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

Commit 87ecb92e authored by Zaiyue Xue's avatar Zaiyue Xue
Browse files

Fix anomaly is not logged in non-debug mode. 

Proto is not supported to fully toString() in non-debug mode.

Bug: 327978756
Fix: 327978756
Test: manual
Change-Id: I1231c40769fd345b500ec120cf9c6fca3eb7c54a
parent bec4c957

src/com/android/settings/fuelgauge/batteryusage/PowerUsageAdvanced.java

+11 −3
Original line number Diff line number Diff line
@@ -274,8 +274,7 @@ public class PowerUsageAdvanced extends PowerUsageBase { 
        if (!isResumed() || anomalyEventList == null) {

            return;

        }

        Log.d(TAG, "anomalyEventList = " + anomalyEventList);



        logPowerAnomalyEventList(anomalyEventList);

        final Set<String> dismissedPowerAnomalyKeys =

                DatabaseUtils.getDismissedPowerAnomalyKeys(getContext());

        Log.d(TAG, "dismissedPowerAnomalyKeys = " + dismissedPowerAnomalyKeys);
@@ -429,10 +428,19 @@ public class PowerUsageAdvanced extends PowerUsageBase { 
                        .filter(predicate)

                        .max(Comparator.comparing(PowerAnomalyEvent::getScore))

                        .orElse(null);

        Log.d(TAG, "filterAnomalyEvent = " + filterAnomalyEvent);

        Log.d(TAG, "filterAnomalyEvent = "

                + (filterAnomalyEvent == null ? null : filterAnomalyEvent.getEventId()));

        return filterAnomalyEvent;

    }



    private static void logPowerAnomalyEventList(PowerAnomalyEventList anomalyEventList) {

        final StringBuilder stringBuilder = new StringBuilder();

        for (PowerAnomalyEvent anomalyEvent : anomalyEventList.getPowerAnomalyEventsList()) {

            stringBuilder.append(anomalyEvent.getEventId()).append(", ");

        }

        Log.d(TAG, "anomalyEventList = [" + stringBuilder + "]");

    }



    private static BatteryDiffData getAllBatteryDiffData(

            Map<Integer, Map<Integer, BatteryDiffData>> batteryUsageMap) {

        return batteryUsageMap == null