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

Commit d93a22ea authored by Steve Elliott's avatar Steve Elliott Committed by Android (Google) Code Review
Browse files

Merge "Deprecate Expandablenotificationrow#setNeedsRedaction"

parents 0947f40a 88c6fe27

packages/SystemUI/src/com/android/systemui/statusbar/notification/collection/inflation/NotificationRowBinderImpl.java

+5 −3
Original line number Diff line number Diff line
@@ -268,9 +268,11 @@ public class NotificationRowBinderImpl implements NotificationRowBinder { 
        params.setUseIncreasedCollapsedHeight(useIncreasedCollapsedHeight);

        params.setUseLowPriority(isLowPriority);



        // TODO: Replace this API with RowContentBindParams directly. Also move to a separate

        // redaction controller.

        row.setNeedsRedaction(mNotificationLockscreenUserManager.needsRedaction(entry));

        if (mNotificationLockscreenUserManager.needsRedaction(entry)) {

            params.requireContentViews(FLAG_CONTENT_VIEW_PUBLIC);

        } else {

            params.markContentViewsFreeable(FLAG_CONTENT_VIEW_PUBLIC);

        }



        params.rebindAllContentViews();

        mRowContentBindStage.requestRebind(entry, en -> {

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

+3 −4
Original line number Diff line number Diff line
@@ -42,7 +42,6 @@ import android.graphics.drawable.AnimatedVectorDrawable; 
import android.graphics.drawable.AnimationDrawable;

import android.graphics.drawable.ColorDrawable;

import android.graphics.drawable.Drawable;

import android.os.AsyncTask;

import android.os.Build;

import android.os.Bundle;

import android.os.RemoteException;
@@ -56,7 +55,6 @@ import android.util.IndentingPrintWriter; 
import android.util.Log;

import android.util.MathUtils;

import android.util.Property;

import android.util.Slog;

import android.view.KeyEvent;

import android.view.LayoutInflater;

import android.view.MotionEvent;
@@ -90,8 +88,8 @@ import com.android.systemui.statusbar.RemoteInputController; 
import com.android.systemui.statusbar.SmartReplyController;

import com.android.systemui.statusbar.StatusBarIconView;

import com.android.systemui.statusbar.notification.AboveShelfChangedListener;

import com.android.systemui.statusbar.notification.LaunchAnimationParameters;

import com.android.systemui.statusbar.notification.FeedbackIcon;

import com.android.systemui.statusbar.notification.LaunchAnimationParameters;

import com.android.systemui.statusbar.notification.NotificationFadeAware;

import com.android.systemui.statusbar.notification.NotificationLaunchAnimatorController;

import com.android.systemui.statusbar.notification.NotificationUtils;
@@ -1505,8 +1503,9 @@ public class ExpandableNotificationRow extends ActivatableNotificationView 
        mUseIncreasedHeadsUpHeight = use;

    }



    /** @deprecated TODO: Remove this when the old pipeline code is removed. */

    @Deprecated

    public void setNeedsRedaction(boolean needsRedaction) {

        // TODO: Move inflation logic out of this call and remove this method

        if (mNeedsRedaction != needsRedaction) {

            mNeedsRedaction = needsRedaction;

            if (!isRemoved()) {

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

+2 −1
Original line number Diff line number Diff line
@@ -102,8 +102,9 @@ public final class RowContentBindParams { 
     * @see InflationFlag

     */

    public void markContentViewsFreeable(@InflationFlag int contentViews) {

        @InflationFlag int existingContentViews = contentViews &= mContentViews;

        mContentViews &= ~contentViews;

        mDirtyContentViews &= ~contentViews;

        mDirtyContentViews |= existingContentViews;

    }



    public @InflationFlag int getContentViews() {