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

Commit 79d47446 authored by Yu-Ting Tseng's avatar Yu-Ting Tseng Committed by Android (Google) Code Review
Browse files

Merge "Fix a memory leak in UidState."

parents a698ab8b 1635233c

core/java/com/android/internal/app/procstats/UidState.java

+1 −0
Original line number Diff line number Diff line
@@ -150,6 +150,7 @@ public final class UidState { 
    public void resetSafely(long now) {

        mDurations.resetTable();

        mStartTime = now;

        mProcesses.removeIf(p -> !p.isInUse());

    }



    /**

core/tests/coretests/src/com/android/internal/app/procstats/ProcessStatsTest.java

+15 −0
Original line number Diff line number Diff line
@@ -156,4 +156,19 @@ public class ProcessStatsTest extends TestCase { 
                        eq(0),

                        eq(APP_1_PROCESS_NAME));

    }



    @SmallTest

    public void testSafelyResetClearsProcessInUidState() throws Exception {

        ProcessStats processStats = new ProcessStats();

        ProcessState processState =

                processStats.getProcessStateLocked(

                        APP_1_PACKAGE_NAME, APP_1_UID, APP_1_VERSION, APP_1_PROCESS_NAME);

        processState.makeActive();

        UidState uidState = processStats.mUidStates.get(APP_1_UID);

        assertTrue(uidState.isInUse());

        processState.makeInactive();

        uidState.resetSafely(NOW_MS);

        processState.makeActive();

        assertFalse(uidState.isInUse());

    }

}