diff options
author | Joe Onorato <joeo@android.com> | 2010-06-07 11:52:41 -0700 |
---|---|---|
committer | Joe Onorato <joeo@android.com> | 2010-06-09 14:33:30 -0700 |
commit | f1f259165ffaa4095afbd50fea47ed091cbc14b3 (patch) | |
tree | f9778c41051b321c95df81f4452661a70ca5fab1 /packages | |
parent | 7a0f36bd93ad8a5b8cb3e1fe56dbdb43a0ad3a57 (diff) | |
download | frameworks_base-f1f259165ffaa4095afbd50fea47ed091cbc14b3.zip frameworks_base-f1f259165ffaa4095afbd50fea47ed091cbc14b3.tar.gz frameworks_base-f1f259165ffaa4095afbd50fea47ed091cbc14b3.tar.bz2 |
Call into the notification manager when the panel is revealed.
This lets it turn off the LED. However, it seems like somebody broke
the notification LEDs. GRRR.
Change-Id: I3f7066c2b2e1673dc0144a34cf59946351a647be
Diffstat (limited to 'packages')
-rw-r--r-- | packages/SystemUI/src/com/android/systemui/statusbar/PhoneStatusBarService.java | 13 |
1 files changed, 10 insertions, 3 deletions
diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/PhoneStatusBarService.java b/packages/SystemUI/src/com/android/systemui/statusbar/PhoneStatusBarService.java index ae97400..d93a6c9 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/PhoneStatusBarService.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/PhoneStatusBarService.java @@ -153,6 +153,7 @@ public class PhoneStatusBarService extends StatusBarService { TrackingView mTrackingView; WindowManager.LayoutParams mTrackingParams; int mTrackingPosition; // the position of the top of the tracking view. + private boolean mPanelSlightlyVisible; // ticker private Ticker mTicker; @@ -1297,8 +1298,15 @@ public class PhoneStatusBarService extends StatusBarService { // because the window itself extends below the content view. mExpandedParams.y = -disph; } - visibilityChanged(visible); mExpandedDialog.getWindow().setAttributes(mExpandedParams); + + // As long as this isn't just a repositioning that's not supposed to affect + // the user's perception of what's showing, call to say that the visibility + // has changed. (Otherwise, someone else will call to do that). + if (expandedPosition != EXPANDED_LEAVE_ALONE) { + Slog.d(TAG, "updateExpandedViewPos visibilityChanged(" + visible + ")"); + visibilityChanged(visible); + } } if (SPEW) { @@ -1328,12 +1336,11 @@ public class PhoneStatusBarService extends StatusBarService { * turned off. If any other notifications happen, the lights will turn back on. Steve says * this is what he wants. (see bug 1131461) */ - private boolean mPanelSlightlyVisible; void visibilityChanged(boolean visible) { if (mPanelSlightlyVisible != visible) { mPanelSlightlyVisible = visible; try { - mBarService.visibilityChanged(visible); + mBarService.onPanelRevealed(); } catch (RemoteException ex) { // Won't fail unless the world has ended. } |