diff options
| -rw-r--r-- | services/core/java/com/android/server/dreams/DreamController.java | 26 | ||||
| -rw-r--r-- | services/core/java/com/android/server/dreams/DreamManagerService.java | 18 |
2 files changed, 25 insertions, 19 deletions
diff --git a/services/core/java/com/android/server/dreams/DreamController.java b/services/core/java/com/android/server/dreams/DreamController.java index 532045320988..5bc69943033e 100644 --- a/services/core/java/com/android/server/dreams/DreamController.java +++ b/services/core/java/com/android/server/dreams/DreamController.java @@ -62,6 +62,7 @@ final class DreamController { private final Listener mListener; private final IWindowManager mIWindowManager; private long mDreamStartTime; + private String mSavedStopReason; private final Intent mDreamingStartedIntent = new Intent(Intent.ACTION_DREAMING_STARTED) .addFlags(Intent.FLAG_RECEIVER_REGISTERED_ONLY); @@ -77,14 +78,15 @@ final class DreamController { public void run() { if (mCurrentDream != null && mCurrentDream.mBound && !mCurrentDream.mConnected) { Slog.w(TAG, "Bound dream did not connect in the time allotted"); - stopDream(true /*immediate*/); + stopDream(true /*immediate*/, "slow to connect"); } } }; private final Runnable mStopStubbornDreamRunnable = () -> { Slog.w(TAG, "Stubborn dream did not finish itself in the time allotted"); - stopDream(true /*immediate*/); + stopDream(true /*immediate*/, "slow to finish"); + mSavedStopReason = null; }; public DreamController(Context context, Handler handler, Listener listener) { @@ -116,7 +118,7 @@ final class DreamController { public void startDream(Binder token, ComponentName name, boolean isTest, boolean canDoze, int userId, PowerManager.WakeLock wakeLock) { - stopDream(true /*immediate*/); + stopDream(true /*immediate*/, "starting new dream"); Trace.traceBegin(Trace.TRACE_TAG_POWER, "startDream"); try { @@ -141,12 +143,12 @@ final class DreamController { Context.BIND_AUTO_CREATE | Context.BIND_FOREGROUND_SERVICE, new UserHandle(userId))) { Slog.e(TAG, "Unable to bind dream service: " + intent); - stopDream(true /*immediate*/); + stopDream(true /*immediate*/, "bindService failed"); return; } } catch (SecurityException ex) { Slog.e(TAG, "Unable to bind dream service: " + intent, ex); - stopDream(true /*immediate*/); + stopDream(true /*immediate*/, "unable to bind service: SecExp."); return; } @@ -157,7 +159,7 @@ final class DreamController { } } - public void stopDream(boolean immediate) { + public void stopDream(boolean immediate, String reason) { if (mCurrentDream == null) { return; } @@ -173,6 +175,7 @@ final class DreamController { // Give the dream a moment to wake up and finish itself gently. mCurrentDream.mWakingGently = true; try { + mSavedStopReason = reason; mCurrentDream.mService.wakeUp(); mHandler.postDelayed(mStopStubbornDreamRunnable, DREAM_FINISH_TIMEOUT); return; @@ -186,7 +189,9 @@ final class DreamController { mCurrentDream = null; Slog.i(TAG, "Stopping dream: name=" + oldDream.mName + ", isTest=" + oldDream.mIsTest + ", canDoze=" + oldDream.mCanDoze - + ", userId=" + oldDream.mUserId); + + ", userId=" + oldDream.mUserId + + ", reason='" + reason + "'" + + (mSavedStopReason == null ? "" : "(from '" + mSavedStopReason + "')")); MetricsLogger.hidden(mContext, oldDream.mCanDoze ? MetricsEvent.DOZING : MetricsEvent.DREAMING); MetricsLogger.histogram(mContext, @@ -195,6 +200,7 @@ final class DreamController { mHandler.removeCallbacks(mStopUnconnectedDreamRunnable); mHandler.removeCallbacks(mStopStubbornDreamRunnable); + mSavedStopReason = null; if (oldDream.mSentStartBroadcast) { mContext.sendBroadcastAsUser(mDreamingStoppedIntent, UserHandle.ALL); @@ -233,7 +239,7 @@ final class DreamController { mCurrentDream.mDreamingStartedCallback); } catch (RemoteException ex) { Slog.e(TAG, "The dream service died unexpectedly.", ex); - stopDream(true /*immediate*/); + stopDream(true /*immediate*/, "attach failed"); return; } @@ -287,7 +293,7 @@ final class DreamController { mHandler.post(() -> { mService = null; if (mCurrentDream == DreamRecord.this) { - stopDream(true /*immediate*/); + stopDream(true /*immediate*/, "binder died"); } }); } @@ -312,7 +318,7 @@ final class DreamController { mHandler.post(() -> { mService = null; if (mCurrentDream == DreamRecord.this) { - stopDream(true /*immediate*/); + stopDream(true /*immediate*/, "service disconnected"); } }); } diff --git a/services/core/java/com/android/server/dreams/DreamManagerService.java b/services/core/java/com/android/server/dreams/DreamManagerService.java index bcf262d97235..769956d797b0 100644 --- a/services/core/java/com/android/server/dreams/DreamManagerService.java +++ b/services/core/java/com/android/server/dreams/DreamManagerService.java @@ -121,7 +121,7 @@ public final class DreamManagerService extends SystemService { public void onReceive(Context context, Intent intent) { writePulseGestureEnabled(); synchronized (mLock) { - stopDreamLocked(false /*immediate*/); + stopDreamLocked(false /*immediate*/, "user switched"); } } }, new IntentFilter(Intent.ACTION_USER_SWITCHED), null, mHandler); @@ -180,7 +180,7 @@ public final class DreamManagerService extends SystemService { // for example when being undocked. long time = SystemClock.uptimeMillis(); mPowerManager.userActivity(time, false /*noChangeLights*/); - stopDreamInternal(false /*immediate*/); + stopDreamInternal(false /*immediate*/, "request awaken"); } private void finishSelfInternal(IBinder token, boolean immediate) { @@ -197,7 +197,7 @@ public final class DreamManagerService extends SystemService { // device may simply go to sleep. synchronized (mLock) { if (mCurrentDreamToken == token) { - stopDreamLocked(immediate); + stopDreamLocked(immediate, "finished self"); } } } @@ -218,9 +218,9 @@ public final class DreamManagerService extends SystemService { } } - private void stopDreamInternal(boolean immediate) { + private void stopDreamInternal(boolean immediate, String reason) { synchronized (mLock) { - stopDreamLocked(immediate); + stopDreamLocked(immediate, reason); } } @@ -373,7 +373,7 @@ public final class DreamManagerService extends SystemService { return; } - stopDreamLocked(true /*immediate*/); + stopDreamLocked(true /*immediate*/, "starting new dream"); Slog.i(TAG, "Entering dreamland."); @@ -392,7 +392,7 @@ public final class DreamManagerService extends SystemService { })); } - private void stopDreamLocked(final boolean immediate) { + private void stopDreamLocked(final boolean immediate, String reason) { if (mCurrentDreamToken != null) { if (immediate) { Slog.i(TAG, "Leaving dreamland."); @@ -408,7 +408,7 @@ public final class DreamManagerService extends SystemService { @Override public void run() { Slog.i(TAG, "Performing gentle wake from dream."); - mController.stopDream(immediate); + mController.stopDream(immediate, reason); } }); } @@ -696,7 +696,7 @@ public final class DreamManagerService extends SystemService { @Override public void stopDream(boolean immediate) { - stopDreamInternal(immediate); + stopDreamInternal(immediate, "requested stopDream"); } @Override |