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

Commit 19ee06db authored by Charles Chen's avatar Charles Chen
Browse files

Fix multiuser CTS tests 

cts/hostsidetests/multiuser relies on parsing log to detect
app crashes. In [1], we removes the log for refactoring,
which leads to the detecting mechanism in multiuser CTS no more
working.

This CL adds the log it back.

Test: atest cts/hostsidetests/multiuser
fixes: 217452444
Change-Id: I24a1c94a35a627e5bd02b4321db24eb79c12c7da
parent c50b959b

services/core/java/com/android/server/am/AppErrors.java

+2 −0
Original line number Diff line number Diff line
@@ -1018,6 +1018,7 @@ class AppErrors { 
                        Settings.Secure.SHOW_FIRST_CRASH_DIALOG_DEV_OPTION,

                        0,

                        mService.mUserController.getCurrentUserId()) != 0;

                final String packageName = proc.info.packageName;

                final boolean crashSilenced = mAppsNotReportingCrashes != null

                        && mAppsNotReportingCrashes.contains(proc.info.packageName);

                final long now = SystemClock.uptimeMillis();
@@ -1026,6 +1027,7 @@ class AppErrors { 
                if ((mService.mAtmInternal.canShowErrorDialogs() || showBackground)

                        && !crashSilenced && !shouldThottle

                        && (showFirstCrash || showFirstCrashDevOption || data.repeating)) {

                    Slog.i(TAG, "Showing crash dialog for package " + packageName + " u" + userId);

                    errState.getDialogController().showCrashDialogs(data);

                    if (!proc.isolated) {

                        mProcessCrashShowDialogTimes.put(proc.processName, proc.uid, now);