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

Commit 814404d1 authored by Achim Thesmann's avatar Achim Thesmann
Browse files

Remove redundant logs 

The `allowBasedOnCaller`/`allowBasedOnRealCaller` methods already dump the state if `DEBUG_ACTIVITY_STARTS` is `true`. DOuble logging only makes the logs more difficult to read.

Test: atest BackgroundActivityStartControllerTest
Change-Id: Ia5c38a8717a3d50369c5d60fa6780c6c49148cea
parent afa74609

services/core/java/com/android/server/wm/BackgroundActivityStartController.java

+0 −9
Original line number Diff line number Diff line
@@ -714,9 +714,6 @@ public class BackgroundActivityStartController { 


        if (!state.hasRealCaller()) {

            if (resultForCaller.allows()) {

                if (DEBUG_ACTIVITY_STARTS) {

                    Slog.d(TAG, "Background activity start allowed. " + state);

                }

                return allowBasedOnCaller(state);

            }

            return abortLaunch(state);
@@ -742,15 +739,9 @@ public class BackgroundActivityStartController { 


        // Handle cases with explicit opt-in

        if (resultForCaller.allows() && state.callerExplicitOptInOrAutoOptIn()) {

            if (DEBUG_ACTIVITY_STARTS) {

                Slog.d(TAG, "Activity start explicitly allowed by caller. " + state);

            }

            return allowBasedOnCaller(state);

        }

        if (resultForRealCaller.allows() && state.realCallerExplicitOptInOrAutoOptIn()) {

            if (DEBUG_ACTIVITY_STARTS) {

                Slog.d(TAG, "Activity start explicitly allowed by real caller. " + state);

            }

            return allowBasedOnRealCaller(state);

        }

        // Handle PendingIntent cases with default behavior next