Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 20de6bde authored by Dave Mankoff's avatar Dave Mankoff
Browse files

Fix double tapping notifications on lock screen. 

http://ag/10273995 broke double tapping on notifications. This cl
restores the behavior by replacing a call to
ActivatableNotificationView#performClick with
ExpandableOutlineView#performClick.

That is to say, we needed to call the super method, as we did before
the breaking cl was checked in.

Bug: 149744457
Test: atest SystemUITests
Change-Id: Ie77ade20f74b04eb0edbe0f1806938523068175a
parent 4daaa833

packages/SystemUI/src/com/android/systemui/statusbar/notification/row/ActivatableNotificationView.java

+4 −0
Original line number Diff line number Diff line
@@ -336,6 +336,10 @@ public abstract class ActivatableNotificationView extends ExpandableOutlineView 
        return false;

    }



    boolean superPerformClick() {

        return super.performClick();

    }



    /**

     * Cancels the hotspot and makes the notification inactive.

     */

packages/SystemUI/src/com/android/systemui/statusbar/notification/row/ActivatableNotificationViewController.java

+2 −1
Original line number Diff line number Diff line
@@ -72,7 +72,8 @@ public class ActivatableNotificationViewController { 
            } else {

                mView.makeInactive(true /* animate */);

            }

        }, mView::performClick, mView::handleSlideBack, mFalsingManager::onNotificationDoubleTap);

        }, mView::superPerformClick, mView::handleSlideBack,

                mFalsingManager::onNotificationDoubleTap);

        mView.setOnTouchListener(mTouchHandler);

        mView.setTouchHandler(mTouchHandler);

        mView.setOnDimmedListener(dimmed -> {