Fix a bug in double-swiping panels...

...that could cause them both to be shown at once.

Bug: 7179458
Change-Id: I32e51bd6d4f5773d0f4f5d3e99a84d299cd4b766

2 files changed, 7 insertions, 2 deletions

diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/phone/PanelView.java b/packages/SystemUI/src/com/android/systemui/statusbar/phone/PanelView.java
index ca1f75cf44f0..de621794a9d8 100644
--- a/packages/SystemUI/src/com/android/systemui/statusbar/phone/PanelView.java
+++ b/packages/SystemUI/src/com/android/systemui/statusbar/phone/PanelView.java
@@ -295,6 +295,7 @@ public class PanelView extends FrameLayout {
     }
 
     public void fling(float vel, boolean always) {
+        if (DEBUG) LOG("fling: vel=%.3f, this=%s", vel, this);
         mVel = vel;
 
         if (always||mVel != 0) {
@@ -416,7 +417,10 @@ public class PanelView extends FrameLayout {
 
     public void collapse() {
         // TODO: abort animation or ongoing touch
+        if (DEBUG) LOG("collapse: " + this);
         if (!isFullyCollapsed()) {
+            mTimeAnimator.cancel();
+            mClosing = true;
             // collapse() should never be a rubberband, even if an animation is already running
             mRubberbanding = false;
             fling(-mSelfCollapseVelocityPx, /*always=*/ true);
@@ -424,10 +428,10 @@ public class PanelView extends FrameLayout {
     }
 
     public void expand() {
+        if (DEBUG) LOG("expand: " + this);
         if (isFullyCollapsed()) {
             mBar.startOpeningPanel(this);
-            if (DEBUG) LOG("expand: calling fling(%s, true)", mSelfExpandVelocityPx);
-            fling (mSelfExpandVelocityPx, /*always=*/ true);
+            fling(mSelfExpandVelocityPx, /*always=*/ true);
         } else if (DEBUG) {
             if (DEBUG) LOG("skipping expansion: is expanded");
         }
diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/phone/PhoneStatusBarView.java b/packages/SystemUI/src/com/android/systemui/statusbar/phone/PhoneStatusBarView.java
index 15ac5c0a96a7..516b1abb3f58 100644
--- a/packages/SystemUI/src/com/android/systemui/statusbar/phone/PhoneStatusBarView.java
+++ b/packages/SystemUI/src/com/android/systemui/statusbar/phone/PhoneStatusBarView.java
@@ -149,6 +149,7 @@ public class PhoneStatusBarView extends PanelBar {
 
     @Override
     public void onPanelFullyOpened(PanelView openPanel) {
+        super.onPanelFullyOpened(openPanel);
         mFadingPanel = openPanel;
         mShouldFade = true; // now you own the fade, mister
     }