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

Commit 9d97ee2a authored by Joe Onorato's avatar Joe Onorato
Browse files

Turn this wtf into just a log. 

We're seeing this a bunch, but the overhead from the WTF is concerning.
So turn it off.  It means that somewhere there is a race in tearing
down the app, or maybe some place fails to call back in some scenario.

Bug: 28932059
Change-Id: Ice14ade95bb9377ad622d440fb022953ad51c34c
parent 8b655e0f

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

+2 −2
Original line number Diff line number Diff line
@@ -2201,11 +2201,11 @@ public final class ActiveServices { 
                    // If the app is null, then it was probably removed because the process died,

                    // otherwise wtf

                    if (r.app != null) {

                        Slog.wtfStack(TAG, "Service done with onDestroy, but not inDestroying: "

                        Slog.w(TAG, "Service done with onDestroy, but not inDestroying: "

                                + r + ", app=" + r.app);

                    }

                } else if (r.executeNesting != 1) {

                    Slog.wtfStack(TAG, "Service done with onDestroy, but executeNesting="

                    Slog.w(TAG, "Service done with onDestroy, but executeNesting="

                            + r.executeNesting + ": " + r);

                    // Fake it to keep from ANR due to orphaned entry.

                    r.executeNesting = 1;