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

Commit a64839b5 authored by jackqdyulei's avatar jackqdyulei
Browse files

Use default package in BatteryEntry 

Before this cl, we use the first package name if there is a name
list in BatterySipper.

We should use BatteryEntry.defaultPackageName instead, which could
describle the batterysipper more accurately.

Bug: 62417935
Test: RunSettingsRoboTests
Change-Id: I44fa151c0cf8f14cba305f179c7e94e5bc3ce434
parent 85395155

src/com/android/settings/fuelgauge/AdvancedPowerUsageDetail.java

+1 −1
Original line number Diff line number Diff line
@@ -119,7 +119,7 @@ public class AdvancedPowerUsageDetail extends DashboardFragment implements 
            args.putString(EXTRA_PACKAGE_NAME, null);

        } else {

            // populate data for normal app

            args.putString(EXTRA_PACKAGE_NAME, sipper.mPackages[0]);

            args.putString(EXTRA_PACKAGE_NAME, entry.defaultPackageName);

        }



        args.putInt(EXTRA_UID, sipper.getUid());

tests/robotests/src/com/android/settings/fuelgauge/AdvancedPowerUsageDetailTest.java

+1 −0
Original line number Diff line number Diff line
@@ -248,6 +248,7 @@ public class AdvancedPowerUsageDetailTest { 
    @Test

    public void testStartBatteryDetailPage_NormalApp() {

        mBatterySipper.mPackages = PACKAGE_NAME;

        mBatteryEntry.defaultPackageName = PACKAGE_NAME[0];

        AdvancedPowerUsageDetail.startBatteryDetailPage(mTestActivity, null, mBatteryStatsHelper, 0,

                mBatteryEntry, USAGE_PERCENT);