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

Commit f529ec2b authored by Ned Burns's avatar Ned Burns
Browse files

Remove unused isUpdate argument in RowBinder 

No longer necessary due to VisualStabilityManager changes.

Test: atest, manual
Change-Id: I0bd67c64b62cbe15a6e826d184fed20d9d6e64df
parent 6b19b089

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

+3 −6
Original line number Diff line number Diff line
@@ -345,8 +345,7 @@ public class NotificationEntryManager implements 


        Dependency.get(LeakDetector.class).trackInstance(entry);

        // Construct the expanded view.

        getRowBinder().inflateViews(entry, () -> performRemoveNotification(notification),

                mNotificationData.get(entry.key) != null);

        getRowBinder().inflateViews(entry, () -> performRemoveNotification(notification));



        abortExistingInflation(key);
@@ -387,8 +386,7 @@ public class NotificationEntryManager implements 
            listener.onPreEntryUpdated(entry);

        }



        getRowBinder().inflateViews(entry, () -> performRemoveNotification(notification),

                mNotificationData.get(entry.key) != null);

        getRowBinder().inflateViews(entry, () -> performRemoveNotification(notification));

        updateNotifications();



        if (DEBUG) {
@@ -446,8 +444,7 @@ public class NotificationEntryManager implements 
                    entry,

                    oldImportances.get(entry.key),

                    oldAdjustments.get(entry.key),

                    NotificationUiAdjustment.extractFromNotificationEntry(entry),

                    mNotificationData.get(entry.key) != null);

                    NotificationUiAdjustment.extractFromNotificationEntry(entry));

        }



        updateNotifications();

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

+6 −9
Original line number Diff line number Diff line
@@ -124,8 +124,7 @@ public class NotificationRowBinder { 
     */

    public void inflateViews(

            NotificationEntry entry,

            Runnable onDismissRunnable,

            boolean isUpdate)

            Runnable onDismissRunnable)

            throws InflationException {

        ViewGroup parent = mListContainer.getViewParentForNotification(entry);

        PackageManager pmUser = StatusBar.getPackageManagerForUser(mContext,
@@ -135,13 +134,13 @@ public class NotificationRowBinder { 
        if (entry.rowExists()) {

            entry.updateIcons(mContext, sbn);

            entry.reset();

            updateNotification(entry, pmUser, sbn, entry.getRow(), isUpdate);

            updateNotification(entry, pmUser, sbn, entry.getRow());

        } else {

            entry.createIcons(mContext, sbn);

            new RowInflaterTask().inflate(mContext, parent, entry,

                    row -> {

                        bindRow(entry, pmUser, sbn, row, onDismissRunnable);

                        updateNotification(entry, pmUser, sbn, row, isUpdate);

                        updateNotification(entry, pmUser, sbn, row);

                    });

        }

    }
@@ -197,15 +196,14 @@ public class NotificationRowBinder { 
            NotificationEntry entry,

            @Nullable Integer oldImportance,

            NotificationUiAdjustment oldAdjustment,

            NotificationUiAdjustment newAdjustment,

            boolean isUpdate) {

            NotificationUiAdjustment newAdjustment) {

        if (NotificationUiAdjustment.needReinflate(oldAdjustment, newAdjustment)) {

            if (entry.rowExists()) {

                entry.reset();

                PackageManager pmUser = StatusBar.getPackageManagerForUser(

                        mContext,

                        entry.notification.getUser().getIdentifier());

                updateNotification(entry, pmUser, entry.notification, entry.getRow(), isUpdate);

                updateNotification(entry, pmUser, entry.notification, entry.getRow());

            } else {

                // Once the RowInflaterTask is done, it will pick up the updated entry, so

                // no-op here.
@@ -224,8 +222,7 @@ public class NotificationRowBinder { 
            NotificationEntry entry,

            PackageManager pmUser,

            StatusBarNotification sbn,

            ExpandableNotificationRow row,

            boolean isUpdate) {

            ExpandableNotificationRow row) {

        row.setIsLowPriority(entry.ambient);

        // bind the click event to the content area

        checkNotNull(mNotificationClicker).register(row, sbn);