diff options
author | Prashant Malani <pmalani@google.com> | 2014-08-06 16:49:53 -0700 |
---|---|---|
committer | Prashant Malani <pmalani@google.com> | 2014-08-06 22:22:18 -0700 |
commit | 7450614a496d0b6a5836703c1145e9bcd4f104e9 (patch) | |
tree | fc4219d662146bd594e6f70899198f0bb22312cb /services/core/java/com/android/server/DockObserver.java | |
parent | 6ebeefa04cb3c360d8028eb77e60bacf8a800cb1 (diff) | |
download | frameworks_base-7450614a496d0b6a5836703c1145e9bcd4f104e9.zip frameworks_base-7450614a496d0b6a5836703c1145e9bcd4f104e9.tar.gz frameworks_base-7450614a496d0b6a5836703c1145e9bcd4f104e9.tar.bz2 |
Move wakelock release to handleMessage
There were certain code paths in handleDockStateChange() that were
exiting without releasing the wakelock. So we move the release to the
calling function to prevent stray held wakelocks.
Bug: 16841895
Change-Id: I8ae4d2ab8f775c42a893d3bcdef34321c5f631a6
Signed-off-by: Prashant Malani <pmalani@google.com>
Diffstat (limited to 'services/core/java/com/android/server/DockObserver.java')
-rw-r--r-- | services/core/java/com/android/server/DockObserver.java | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/services/core/java/com/android/server/DockObserver.java b/services/core/java/com/android/server/DockObserver.java index af38664..d05c280 100644 --- a/services/core/java/com/android/server/DockObserver.java +++ b/services/core/java/com/android/server/DockObserver.java @@ -201,9 +201,6 @@ final class DockObserver extends SystemService { // There are many components in the system watching for this so as to // adjust audio routing, screen orientation, etc. getContext().sendStickyBroadcastAsUser(intent, UserHandle.ALL); - - // Release the wake lock that was acquired when the message was posted. - mWakeLock.release(); } } @@ -213,6 +210,7 @@ final class DockObserver extends SystemService { switch (msg.what) { case MSG_DOCK_STATE_CHANGED: handleDockStateChange(); + mWakeLock.release(); break; } } |