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

Commit 626f043a authored by Will Brockman's avatar Will Brockman
Browse files

Notification statsd logs: Notification panel. 

Experimenting with moving this logging to the SysUI side to be able to
log the complete panel, including any lifetime-extended items.

Test: atest NotificiationLoggerTest SystemUITests
Test: statsd_testdrive 245
Test: adb shell cmd stats print-logs && adb logcat -s statsd:I | grep ' (245)'

Bug: 146488473
Change-Id: I0a1a1ddd8838c002807e7137477175f0acceba98
parent 881b688f

core/java/com/android/internal/logging/InstanceId.java

+11 −0
Original line number Diff line number Diff line
@@ -48,6 +48,17 @@ public final class InstanceId implements Parcelable { 
        return mId;

    }



    /**

     * Create a fake instance ID for testing purposes.  Not for production use. See also

     * InstanceIdSequenceFake, which is a testing replacement for InstanceIdSequence.

     * @param id The ID you want to assign.

     * @return new InstanceId.

     */

    @VisibleForTesting

    public static InstanceId fakeInstanceId(int id) {

        return new InstanceId(id);

    }



    @Override

    public int hashCode() {

        return mId;

packages/SystemUI/src/com/android/systemui/statusbar/notification/dagger/NotificationsModule.java

+13 −2
Original line number Diff line number Diff line
@@ -43,6 +43,8 @@ import com.android.systemui.statusbar.notification.init.NotificationsController; 
import com.android.systemui.statusbar.notification.init.NotificationsControllerImpl;

import com.android.systemui.statusbar.notification.init.NotificationsControllerStub;

import com.android.systemui.statusbar.notification.logging.NotificationLogger;

import com.android.systemui.statusbar.notification.logging.NotificationPanelLogger;

import com.android.systemui.statusbar.notification.logging.NotificationPanelLoggerImpl;

import com.android.systemui.statusbar.notification.row.NotificationBlockingHelperManager;

import com.android.systemui.statusbar.notification.row.NotificationGutsManager;

import com.android.systemui.statusbar.phone.NotificationGroupManager;
@@ -144,13 +146,22 @@ public interface NotificationsModule { 
            @UiBackground Executor uiBgExecutor,

            NotificationEntryManager entryManager,

            StatusBarStateController statusBarStateController,

            NotificationLogger.ExpansionStateLogger expansionStateLogger) {

            NotificationLogger.ExpansionStateLogger expansionStateLogger,

            NotificationPanelLogger notificationPanelLogger) {

        return new NotificationLogger(

                notificationListener,

                uiBgExecutor,

                entryManager,

                statusBarStateController,

                expansionStateLogger);

                expansionStateLogger,

                notificationPanelLogger);

    }



    /** Provides an instance of {@link NotificationPanelLogger} */

    @Singleton

    @Provides

    static NotificationPanelLogger provideNotificationPanelLogger() {

        return new NotificationPanelLoggerImpl();

    }



    /** Provides an instance of {@link NotificationBlockingHelperManager} */

packages/SystemUI/src/com/android/systemui/statusbar/notification/logging/NotificationLogger.java

+7 −1
Original line number Diff line number Diff line
@@ -70,6 +70,7 @@ public class NotificationLogger implements StateListener { 
    private final NotificationListenerService mNotificationListener;

    private final Executor mUiBgExecutor;

    private final NotificationEntryManager mEntryManager;

    private final NotificationPanelLogger mNotificationPanelLogger;

    private HeadsUpManager mHeadsUpManager;

    private final ExpansionStateLogger mExpansionStateLogger;
@@ -198,13 +199,15 @@ public class NotificationLogger implements StateListener { 
            @UiBackground Executor uiBgExecutor,

            NotificationEntryManager entryManager,

            StatusBarStateController statusBarStateController,

            ExpansionStateLogger expansionStateLogger) {

            ExpansionStateLogger expansionStateLogger,

            NotificationPanelLogger notificationPanelLogger) {

        mNotificationListener = notificationListener;

        mUiBgExecutor = uiBgExecutor;

        mEntryManager = entryManager;

        mBarService = IStatusBarService.Stub.asInterface(

                ServiceManager.getService(Context.STATUS_BAR_SERVICE));

        mExpansionStateLogger = expansionStateLogger;

        mNotificationPanelLogger = notificationPanelLogger;

        // Not expected to be destroyed, don't need to unsubscribe

        statusBarStateController.addCallback(this);
@@ -264,6 +267,9 @@ public class NotificationLogger implements StateListener { 
        // (Note that in cases where the scroller does emit events, this

        // additional event doesn't break anything.)

        mNotificationLocationsChangedListener.onChildLocationsChanged();

        // TODO - determine if this work needs to be put on mUiBgExecutor

        mNotificationPanelLogger.logPanelShown(mListContainer.hasPulsingNotifications(),

                mEntryManager.getVisibleNotifications());

    }



    private void setDozing(boolean dozing) {

packages/SystemUI/src/com/android/systemui/statusbar/notification/logging/NotificationPanelLogger.java

0 → 100644
+91 −0
Original line number Diff line number Diff line 
/*

 * Copyright (C) 2020 The Android Open Source Project

 *

 * Licensed under the Apache License, Version 2.0 (the "License");

 * you may not use this file except in compliance with the License.

 * You may obtain a copy of the License at

 *

 *      http://www.apache.org/licenses/LICENSE-2.0

 *

 * Unless required by applicable law or agreed to in writing, software

 * distributed under the License is distributed on an "AS IS" BASIS,

 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.

 * See the License for the specific language governing permissions and

 * limitations under the License.

 */



package com.android.systemui.statusbar.notification.logging;



import android.annotation.Nullable;

import android.service.notification.StatusBarNotification;



import com.android.internal.logging.UiEvent;

import com.android.internal.logging.UiEventLogger;

import com.android.systemui.statusbar.notification.collection.NotificationEntry;



import java.util.List;



import nano.Notifications;



/**

 * Statsd logging for notification panel.

 */

public interface NotificationPanelLogger {



    /**

     * Log a NOTIFICATION_PANEL_REPORTED statsd event.

     * @param visibleNotifications as provided by NotificationEntryManager.getVisibleNotifications()

     */

    void logPanelShown(boolean isLockscreen,

            @Nullable List<NotificationEntry> visibleNotifications);



    enum NotificationPanelEvent implements UiEventLogger.UiEventEnum {

        @UiEvent(doc = "Notification panel shown from status bar.")

        NOTIFICATION_PANEL_OPEN_STATUS_BAR(200),

        @UiEvent(doc = "Notification panel shown from lockscreen.")

        NOTIFICATION_PANEL_OPEN_LOCKSCREEN(201);



        private final int mId;

        NotificationPanelEvent(int id) {

            mId = id;

        }

        @Override public int getId() {

            return mId;

        }



        public static NotificationPanelEvent fromLockscreen(boolean isLockscreen) {

            return isLockscreen ? NOTIFICATION_PANEL_OPEN_LOCKSCREEN :

                    NOTIFICATION_PANEL_OPEN_STATUS_BAR;

        }

    }



    /**

     * Composes a NotificationsList proto from the list of visible notifications.

     * @param visibleNotifications as provided by NotificationEntryManager.getVisibleNotifications()

     * @return NotificationList proto suitable for SysUiStatsLog.write(NOTIFICATION_PANEL_REPORTED)

     */

    static Notifications.NotificationList toNotificationProto(

            @Nullable List<NotificationEntry> visibleNotifications) {

        Notifications.NotificationList notificationList = new Notifications.NotificationList();

        if (visibleNotifications == null) {

            return notificationList;

        }

        final Notifications.Notification[] nn =

                new Notifications.Notification[visibleNotifications.size()];

        int i = 0;

        for (NotificationEntry ne : visibleNotifications) {

            StatusBarNotification n = ne.getSbn();

            Notifications.Notification np = new Notifications.Notification();

            np.uid = n.getUid();

            np.packageName = n.getPackageName();

            np.instanceId = n.getInstanceId().getId();

            // TODO set np.groupInstanceId

            np.isGroupSummary = n.getNotification().isGroupSummary();

            np.section = 1 + ne.getBucket();  // We want 0 to mean not set / unknown

            nn[i] = np;

            ++i;

        }

        notificationList.notifications = nn;

        return notificationList;

    }

}

packages/SystemUI/src/com/android/systemui/statusbar/notification/logging/NotificationPanelLoggerImpl.java

0 → 100644
+42 −0
Original line number Diff line number Diff line 
/*

 * Copyright (C) 2020 The Android Open Source Project

 *

 * Licensed under the Apache License, Version 2.0 (the "License");

 * you may not use this file except in compliance with the License.

 * You may obtain a copy of the License at

 *

 *      http://www.apache.org/licenses/LICENSE-2.0

 *

 * Unless required by applicable law or agreed to in writing, software

 * distributed under the License is distributed on an "AS IS" BASIS,

 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.

 * See the License for the specific language governing permissions and

 * limitations under the License.

 */



package com.android.systemui.statusbar.notification.logging;



import com.android.systemui.shared.system.SysUiStatsLog;

import com.android.systemui.statusbar.notification.collection.NotificationEntry;



import com.google.protobuf.nano.MessageNano;



import java.util.List;



import nano.Notifications;



/**

 * Normal implementation of NotificationPanelLogger.

 */

public class NotificationPanelLoggerImpl implements NotificationPanelLogger {

    @Override

    public void logPanelShown(boolean isLockscreen,

            List<NotificationEntry> visibleNotifications) {

        final Notifications.NotificationList proto = NotificationPanelLogger.toNotificationProto(

                visibleNotifications);

        SysUiStatsLog.write(SysUiStatsLog.NOTIFICATION_PANEL_REPORTED,

                /* int event_id */ NotificationPanelEvent.fromLockscreen(isLockscreen).getId(),

                /* int num_notifications*/ proto.notifications.length,

                /* byte[] notifications*/ MessageNano.toByteArray(proto));

    }

}