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

Commit 93c5ed75 authored by Lyn's avatar Lyn
Browse files

Log after click: entry and shade state 

Bug: 292534086
Test: click notification, inspect buffer logs
Change-Id: Ia88b4af4eb8ebfc0b978fb857bd859342117953d
parent e6752329

packages/SystemUI/src/com/android/systemui/statusbar/phone/StatusBarNotificationActivityStarter.java

+3 −1
Original line number Diff line number Diff line
@@ -213,7 +213,9 @@ public class StatusBarNotificationActivityStarter implements NotificationActivit 
     */

    @Override

    public void onNotificationClicked(NotificationEntry entry, ExpandableNotificationRow row) {

        mLogger.logStartingActivityFromClick(entry);

        mLogger.logStartingActivityFromClick(entry, row.isHeadsUpState(),

                mKeyguardStateController.isVisible(),

                mNotificationShadeWindowController.getPanelExpanded());



        if (mRemoteInputManager.isRemoteInputActive(entry)) {

            // We have an active remote input typed and the user clicked on the notification.

packages/SystemUI/src/com/android/systemui/statusbar/phone/StatusBarNotificationActivityStarterLogger.kt

+7 −2
Original line number Diff line number Diff line
@@ -30,11 +30,16 @@ import javax.inject.Inject 
class StatusBarNotificationActivityStarterLogger @Inject constructor(

    @NotifInteractionLog private val buffer: LogBuffer

) {

    fun logStartingActivityFromClick(entry: NotificationEntry) {

    fun logStartingActivityFromClick(entry: NotificationEntry, isHeadsUpState: Boolean,

                                     isKeyguardVisible: Boolean, isPanelExpanded: Boolean) {

        buffer.log(TAG, DEBUG, {

            str1 = entry.logKey

            bool1 = isHeadsUpState

            bool2 = isKeyguardVisible

            bool3 = isPanelExpanded

        }, {

            "(1/5) onNotificationClicked: $str1"

            "(1/5) onNotificationClicked: $str1 isHeadsUpState: $bool1 " +

                    "isKeyguardVisible: $bool2 isPanelExpanded: $bool3"

        })

    }