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

Commit 4c3b329a authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Android (Google) Code Review
Browse files

Merge "Mark entry in onEntryRemoved as @Nullable"

parents 252e8d04 1dd6b40b

packages/SystemUI/src/com/android/systemui/statusbar/NotificationMediaManager.java

+1 −1
Original line number Diff line number Diff line
@@ -157,7 +157,7 @@ public class NotificationMediaManager implements Dumpable { 
        notificationEntryManager.addNotificationEntryListener(new NotificationEntryListener() {

            @Override

            public void onEntryRemoved(

                    Entry entry,

                    @Nullable Entry entry,

                    String key,

                    StatusBarNotification old,

                    NotificationVisibility visibility,

packages/SystemUI/src/com/android/systemui/statusbar/NotificationRemoteInputManager.java

+3 −2
Original line number Diff line number Diff line
@@ -20,6 +20,7 @@ import static android.app.WindowConfiguration.WINDOWING_MODE_FULLSCREEN_OR_SPLIT 
import static com.android.systemui.Dependency.MAIN_HANDLER_NAME;



import android.annotation.NonNull;

import android.annotation.Nullable;

import android.app.ActivityManager;

import android.app.ActivityOptions;

import android.app.KeyguardManager;
@@ -252,13 +253,13 @@ public class NotificationRemoteInputManager implements Dumpable { 
        notificationEntryManager.addNotificationEntryListener(new NotificationEntryListener() {

            @Override

            public void onEntryRemoved(

                    NotificationData.Entry entry,

                    @Nullable NotificationData.Entry entry,

                    String key,

                    StatusBarNotification old,

                    NotificationVisibility visibility,

                    boolean lifetimeExtended,

                    boolean removedByUser) {

                if (removedByUser) {

                if (removedByUser && entry != null) {

                    onPerformRemoveNotification(entry, key);

                }

            }

packages/SystemUI/src/com/android/systemui/statusbar/notification/NotificationAlertingManager.java

+2 −1
Original line number Diff line number Diff line
@@ -20,6 +20,7 @@ import static com.android.systemui.statusbar.NotificationRemoteInputManager.FORC 
import static com.android.systemui.statusbar.notification.row.NotificationInflater.FLAG_CONTENT_VIEW_AMBIENT;

import static com.android.systemui.statusbar.notification.row.NotificationInflater.FLAG_CONTENT_VIEW_HEADS_UP;



import android.annotation.Nullable;

import android.app.Notification;

import android.service.notification.StatusBarNotification;

import android.util.Log;
@@ -82,7 +83,7 @@ public class NotificationAlertingManager { 


            @Override

            public void onEntryRemoved(

                    NotificationData.Entry entry,

                    @Nullable NotificationData.Entry entry,

                    String key,

                    StatusBarNotification old,

                    NotificationVisibility visibility,

packages/SystemUI/src/com/android/systemui/statusbar/notification/NotificationEntryListener.java

+1 −1
Original line number Diff line number Diff line
@@ -80,7 +80,7 @@ public interface NotificationEntryListener { 
     * @param removedByUser true if the notification was removed by a user action

     */

    default void onEntryRemoved(

            NotificationData.Entry entry,

            @Nullable NotificationData.Entry entry,

            String key,

            StatusBarNotification old,

            @Nullable NotificationVisibility visibility,

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

+3 −2
Original line number Diff line number Diff line
@@ -15,6 +15,7 @@ 
 */

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



import android.annotation.Nullable;

import android.content.Context;

import android.os.Handler;

import android.os.RemoteException;
@@ -167,13 +168,13 @@ public class NotificationLogger implements StateListener { 
        entryManager.addNotificationEntryListener(new NotificationEntryListener() {

            @Override

            public void onEntryRemoved(

                    NotificationData.Entry entry,

                    @Nullable NotificationData.Entry entry,

                    String key,

                    StatusBarNotification old,

                    NotificationVisibility visibility,

                    boolean lifetimeExtended,

                    boolean removedByUser) {

                if (removedByUser && visibility != null) {

                if (removedByUser && visibility != null && entry.notification != null) {

                    logNotificationClear(key, entry.notification, visibility);

                }

            }