Fix DISABLE_EXPAND in the tablet status bar.

The codepath that honored this flag had been routed around
when notification peeks were disabled. (The bug was allowing
notifications to be accessed from a secure lockscreen.)

Bug: 5173959
Change-Id: I35dee343ee4f0288530580054f8bb9f511ebe6af

1 files changed, 6 insertions, 14 deletions

diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/tablet/TabletStatusBar.java b/packages/SystemUI/src/com/android/systemui/statusbar/tablet/TabletStatusBar.java
index 201ff2d..76dec5e 100644
--- a/packages/SystemUI/src/com/android/systemui/statusbar/tablet/TabletStatusBar.java
+++ b/packages/SystemUI/src/com/android/systemui/statusbar/tablet/TabletStatusBar.java
@@ -1220,9 +1220,7 @@ public class TabletStatusBar extends StatusBar implements
 
     private View.OnClickListener mOnClickListener = new View.OnClickListener() {
         public void onClick(View v) {
-            if (v == mNotificationTrigger) {
-                onClickNotificationTrigger();
-            } else if (v == mRecentButton) {
+            if (v == mRecentButton) {
                 onClickRecentButton();
             } else if (v == mInputMethodSwitchButton) {
                 onClickInputMethodSwitchButton();
@@ -1232,17 +1230,6 @@ public class TabletStatusBar extends StatusBar implements
         }
     };
 
-    public void onClickNotificationTrigger() {
-        if (DEBUG) Slog.d(TAG, "clicked notification icons; disabled=" + mDisabled);
-        if ((mDisabled & StatusBarManager.DISABLE_EXPAND) == 0) {
-            int msg = !mNotificationPanel.isShowing()
-                ? MSG_OPEN_NOTIFICATION_PANEL
-                : MSG_CLOSE_NOTIFICATION_PANEL;
-            mHandler.removeMessages(msg);
-            mHandler.sendEmptyMessage(msg);
-        }
-    }
-
     public void onClickRecentButton() {
         if (DEBUG) Slog.d(TAG, "clicked recent apps; disabled=" + mDisabled);
         if ((mDisabled & StatusBarManager.DISABLE_EXPAND) == 0) {
@@ -1374,6 +1361,11 @@ public class TabletStatusBar extends StatusBar implements
 //                        event.getY(),
 //                        mInitialTouchX,
 //                        mInitialTouchY));
+
+            if ((mDisabled & StatusBarManager.DISABLE_EXPAND) != 0) {
+                return true;
+            }
+
             final int action = event.getAction();
             switch (action) {
                 case MotionEvent.ACTION_DOWN: