Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Isolate count of visible window from mirror of uid state" into sc-v2-dev

Browse Source
This commit is contained in:
Riddle Hsu
2022-03-14 13:08:16 +00:00
committed by Android (Google) Code Review
parent 98524bb5ff 4d2f0f9178
commit e55e1952c0
2 changed files with 31 additions and 30 deletions
Download Patch File Download Diff File Expand all files Collapse all files

55
services/core/java/com/android/server/wm/MirrorActiveUids.java
View File

@@ -19,7 +19,7 @@ package com.android.server.wm;
import static android.app.ActivityManager.PROCESS_STATE_NONEXISTENT;
import android.app.ActivityManager.ProcessState;
import android.util.SparseArray;
import android.util.SparseIntArray;
import java.io.PrintWriter;
@@ -29,15 +29,14 @@ import java.io.PrintWriter;
* adjustment) or getting state from window manager (background start check).
*/
class MirrorActiveUids {
private final SparseArray<UidRecord> mUidStates = new SparseArray<>();
/** Uid -> process state. */
private final SparseIntArray mUidStates = new SparseIntArray();
/** Uid -> number of non-app visible windows belong to the uid. */
private final SparseIntArray mNumNonAppVisibleWindowMap = new SparseIntArray();
synchronized void onUidActive(int uid, int procState) {
UidRecord r = mUidStates.get(uid);
if (r == null) {
r = new UidRecord();
mUidStates.put(uid, r);
}
r.mProcState = procState;
mUidStates.put(uid, procState);
}
synchronized void onUidInactive(int uid) {
@@ -45,22 +44,28 @@ class MirrorActiveUids {
}
synchronized void onUidProcStateChanged(int uid, int procState) {
final UidRecord r = mUidStates.get(uid);
if (r != null) {
r.mProcState = procState;
final int index = mUidStates.indexOfKey(uid);
if (index >= 0) {
mUidStates.setValueAt(index, procState);
}
}
synchronized @ProcessState int getUidState(int uid) {
final UidRecord r = mUidStates.get(uid);
return r != null ? r.mProcState : PROCESS_STATE_NONEXISTENT;
return mUidStates.get(uid, PROCESS_STATE_NONEXISTENT);
}
/** Called when the surface of non-application (exclude toast) window is shown or hidden. */
synchronized void onNonAppSurfaceVisibilityChanged(int uid, boolean visible) {
final UidRecord r = mUidStates.get(uid);
if (r != null) {
r.mNumNonAppVisibleWindow += visible ? 1 : -1;
final int index = mNumNonAppVisibleWindowMap.indexOfKey(uid);
if (index >= 0) {
final int num = mNumNonAppVisibleWindowMap.valueAt(index) + (visible ? 1 : -1);
if (num > 0) {
mNumNonAppVisibleWindowMap.setValueAt(index, num);
} else {
mNumNonAppVisibleWindowMap.removeAt(index);
}
} else if (visible) {
mNumNonAppVisibleWindowMap.append(uid, 1);
}
}
@@ -70,23 +75,15 @@ class MirrorActiveUids {
* {@link VisibleActivityProcessTracker}.
*/
synchronized boolean hasNonAppVisibleWindow(int uid) {
final UidRecord r = mUidStates.get(uid);
return r != null && r.mNumNonAppVisibleWindow > 0;
return mNumNonAppVisibleWindowMap.get(uid) > 0;
}
synchronized void dump(PrintWriter pw, String prefix) {
pw.print(prefix + "NumNonAppVisibleWindowByUid:[");
for (int i = mUidStates.size() - 1; i >= 0; i--) {
final UidRecord r = mUidStates.valueAt(i);
if (r.mNumNonAppVisibleWindow > 0) {
pw.print(" " + mUidStates.keyAt(i) + ":" + r.mNumNonAppVisibleWindow);
}
pw.print(prefix + "NumNonAppVisibleWindowUidMap:[");
for (int i = mNumNonAppVisibleWindowMap.size() - 1; i >= 0; i--) {
pw.print(" " + mNumNonAppVisibleWindowMap.keyAt(i) + ":"
+ mNumNonAppVisibleWindowMap.valueAt(i));
}
pw.println("]");
}
private static final class UidRecord {
@ProcessState int mProcState;
int mNumNonAppVisibleWindow;
}
}

6
services/tests/wmtests/src/com/android/server/wm/WindowStateTests.java
View File

@@ -800,7 +800,6 @@ public class WindowStateTests extends WindowTestsBase {
@Test
public void testHasActiveVisibleWindow() {
final int uid = ActivityBuilder.DEFAULT_FAKE_UID;
mAtm.mActiveUids.onUidActive(uid, 0 /* any proc state */);
final WindowState app = createWindow(null, TYPE_APPLICATION, "app", uid);
app.mActivityRecord.setVisible(false);
@@ -828,6 +827,11 @@ public class WindowStateTests extends WindowTestsBase {
// Make the application overlay window visible. It should be a valid active visible window.
overlay.onSurfaceShownChanged(true);
assertTrue(mAtm.hasActiveVisibleWindow(uid));
// The number of windows should be independent of the existence of uid state.
mAtm.mActiveUids.onUidInactive(uid);
mAtm.mActiveUids.onUidActive(uid, 0 /* any proc state */);
assertTrue(mAtm.mActiveUids.hasNonAppVisibleWindow(uid));
}
@UseTestDisplay(addWindows = W_ACTIVITY)