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

Commit 2169abd3 authored by Caitlin Shkuratov's avatar Caitlin Shkuratov
Browse files

[SB] Add logs about battery unknown state. 

Bug: 433159946
Flag: EXEMPT logging
Test: Dump BatteryController log, verify unknown status is logged
Change-Id: I8e15f525e98b84f421286bb28b7a5a610ec433b4
parent 47ec2d75

packages/SystemUI/src/com/android/systemui/statusbar/policy/BatteryControllerImpl.java

+1 −0
Original line number Original line Diff line number Diff line
@@ -490,6 +490,7 @@ public class BatteryControllerImpl extends BroadcastReceiver implements BatteryC 
    }

    }





    private void fireBatteryUnknownStateChanged() {

    private void fireBatteryUnknownStateChanged() {

        mLogger.logBatteryUnknownStateChangedCallback(mStateUnknown);

        dispatchSafeChange((callback) -> callback.onBatteryUnknownStateChanged(mStateUnknown));

        dispatchSafeChange((callback) -> callback.onBatteryUnknownStateChanged(mStateUnknown));

    }

    }

packages/SystemUI/src/com/android/systemui/statusbar/policy/BatteryControllerLogger.kt

+16 −7
Original line number Original line Diff line number Diff line
@@ -35,7 +35,7 @@ constructor(@BatteryControllerLog private val logBuffer: LogBuffer) { 
            TAG,

            TAG,

            LogLevel.DEBUG,

            LogLevel.DEBUG,

            { int1 = System.identityHashCode(controller) },

            { int1 = System.identityHashCode(controller) },

            { "BatteryController CREATE (${Integer.toHexString(int1)})" }

            { "BatteryController CREATE (${Integer.toHexString(int1)})" },

        )

        )

    }

    }
@@ -47,7 +47,7 @@ constructor(@BatteryControllerLog private val logBuffer: LogBuffer) { 
                int1 = System.identityHashCode(controller)

                int1 = System.identityHashCode(controller)

                bool1 = hasReceivedBattery

                bool1 = hasReceivedBattery

            },

            },

            { "BatteryController INIT (${Integer.toHexString(int1)}) hasReceivedBattery=$bool1" }

            { "BatteryController INIT (${Integer.toHexString(int1)}) hasReceivedBattery=$bool1" },

        )

        )

    }

    }
@@ -63,7 +63,7 @@ constructor(@BatteryControllerLog private val logBuffer: LogBuffer) { 
                int1 = intent.getIntExtra(EXTRA_LEVEL, DEFAULT)

                int1 = intent.getIntExtra(EXTRA_LEVEL, DEFAULT)

                int2 = intent.getIntExtra(EXTRA_SCALE, DEFAULT)

                int2 = intent.getIntExtra(EXTRA_SCALE, DEFAULT)

            },

            },

            { "Processing BATTERY_CHANGED intent. level=${int1.report()} scale=${int2.report()}" }

            { "Processing BATTERY_CHANGED intent. level=${int1.report()} scale=${int2.report()}" },

        )

        )

    }

    }
@@ -72,7 +72,7 @@ constructor(@BatteryControllerLog private val logBuffer: LogBuffer) { 
            TAG,

            TAG,

            LogLevel.DEBUG,

            LogLevel.DEBUG,

            {},

            {},

            { "Detected test intent. Will not execute battery level callbacks." }

            { "Detected test intent. Will not execute battery level callbacks." },

        )

        )

    }

    }
@@ -81,7 +81,7 @@ constructor(@BatteryControllerLog private val logBuffer: LogBuffer) { 
            TAG,

            TAG,

            LogLevel.DEBUG,

            LogLevel.DEBUG,

            {},

            {},

            { "Entering test mode for BATTERY_LEVEL_TEST intent" }

            { "Entering test mode for BATTERY_LEVEL_TEST intent" },

        )

        )

    }

    }
@@ -101,7 +101,16 @@ constructor(@BatteryControllerLog private val logBuffer: LogBuffer) { 
            {

            {

                "Sending onBatteryLevelChanged callbacks " +

                "Sending onBatteryLevelChanged callbacks " +

                    "with level=$int1, plugged=$bool1, charging=$bool2"

                    "with level=$int1, plugged=$bool1, charging=$bool2"

            },

        )

    }

    }



    fun logBatteryUnknownStateChangedCallback(isUnknown: Boolean) {

        logBuffer.log(

            TAG,

            LogLevel.DEBUG,

            { bool1 = isUnknown },

            { "Sending onBatteryStateUnknown callback with isUnknown=$bool1" },

        )

        )

    }

    }
@@ -110,7 +119,7 @@ constructor(@BatteryControllerLog private val logBuffer: LogBuffer) { 
            TAG,

            TAG,

            LogLevel.DEBUG,

            LogLevel.DEBUG,

            { bool1 = isPowerSave },

            { bool1 = isPowerSave },

            { "Sending onPowerSaveChanged callback with powerSave=$bool1" }

            { "Sending onPowerSaveChanged callback with powerSave=$bool1" },

        )

        )

    }

    }