Merge "Rename NotificationInflater to NotificationContentInflater"
This commit is contained in:
committed by
Android (Google) Code Review
commit
5db74fbe16
@@ -52,7 +52,7 @@ import com.android.systemui.statusbar.notification.NotificationEntryListener;
|
||||
import com.android.systemui.statusbar.notification.NotificationEntryManager;
|
||||
import com.android.systemui.statusbar.notification.NotificationInterruptionStateProvider;
|
||||
import com.android.systemui.statusbar.notification.collection.NotificationEntry;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationInflater;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationContentInflater.InflationFlag;
|
||||
import com.android.systemui.statusbar.phone.StatusBarWindowController;
|
||||
|
||||
import java.util.List;
|
||||
@@ -315,8 +315,7 @@ public class BubbleController implements BubbleExpandedView.OnBubbleBlockedListe
|
||||
}
|
||||
|
||||
@Override
|
||||
public void onEntryInflated(NotificationEntry entry,
|
||||
@NotificationInflater.InflationFlag int inflatedFlags) {
|
||||
public void onEntryInflated(NotificationEntry entry, @InflationFlag int inflatedFlags) {
|
||||
if (!areBubblesEnabled(mContext)) {
|
||||
return;
|
||||
}
|
||||
|
||||
@@ -28,7 +28,7 @@ import android.view.accessibility.AccessibilityEvent;
|
||||
|
||||
import com.android.internal.annotations.VisibleForTesting;
|
||||
import com.android.systemui.statusbar.notification.collection.NotificationEntry;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationInflater.InflationFlag;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationContentInflater.InflationFlag;
|
||||
|
||||
import java.util.stream.Stream;
|
||||
|
||||
|
||||
@@ -16,7 +16,7 @@
|
||||
|
||||
package com.android.systemui.statusbar;
|
||||
|
||||
import static com.android.systemui.statusbar.notification.row.NotificationInflater.FLAG_CONTENT_VIEW_AMBIENT;
|
||||
import static com.android.systemui.statusbar.notification.row.NotificationContentInflater.FLAG_CONTENT_VIEW_AMBIENT;
|
||||
|
||||
import android.annotation.NonNull;
|
||||
import android.content.Context;
|
||||
@@ -26,7 +26,7 @@ import android.util.ArraySet;
|
||||
import com.android.internal.annotations.VisibleForTesting;
|
||||
import com.android.systemui.R;
|
||||
import com.android.systemui.statusbar.notification.collection.NotificationEntry;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationInflater.InflationFlag;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationContentInflater.InflationFlag;
|
||||
|
||||
import javax.inject.Inject;
|
||||
import javax.inject.Singleton;
|
||||
|
||||
@@ -17,8 +17,8 @@
|
||||
package com.android.systemui.statusbar.notification;
|
||||
|
||||
import static com.android.systemui.statusbar.NotificationRemoteInputManager.FORCE_REMOTE_INPUT_HISTORY;
|
||||
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 static com.android.systemui.statusbar.notification.row.NotificationContentInflater.FLAG_CONTENT_VIEW_AMBIENT;
|
||||
import static com.android.systemui.statusbar.notification.row.NotificationContentInflater.FLAG_CONTENT_VIEW_HEADS_UP;
|
||||
|
||||
import android.app.Notification;
|
||||
import android.service.notification.StatusBarNotification;
|
||||
@@ -30,7 +30,7 @@ import com.android.systemui.statusbar.AmbientPulseManager;
|
||||
import com.android.systemui.statusbar.NotificationListener;
|
||||
import com.android.systemui.statusbar.NotificationRemoteInputManager;
|
||||
import com.android.systemui.statusbar.notification.collection.NotificationEntry;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationInflater;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationContentInflater.InflationFlag;
|
||||
import com.android.systemui.statusbar.phone.ShadeController;
|
||||
import com.android.systemui.statusbar.policy.HeadsUpManager;
|
||||
|
||||
@@ -102,8 +102,7 @@ public class NotificationAlertingManager {
|
||||
* @param entry entry to add
|
||||
* @param inflatedFlags flags representing content views that were inflated
|
||||
*/
|
||||
private void showAlertingView(NotificationEntry entry,
|
||||
@NotificationInflater.InflationFlag int inflatedFlags) {
|
||||
private void showAlertingView(NotificationEntry entry, @InflationFlag int inflatedFlags) {
|
||||
if ((inflatedFlags & FLAG_CONTENT_VIEW_HEADS_UP) != 0) {
|
||||
// Possible for shouldHeadsUp to change between the inflation starting and ending.
|
||||
// If it does and we no longer need to heads up, we should free the view.
|
||||
|
||||
@@ -20,7 +20,7 @@ import android.service.notification.StatusBarNotification;
|
||||
|
||||
import com.android.internal.statusbar.NotificationVisibility;
|
||||
import com.android.systemui.statusbar.notification.collection.NotificationEntry;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationInflater;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationContentInflater.InflationFlag;
|
||||
|
||||
/**
|
||||
* Listener interface for changes sent by NotificationEntryManager.
|
||||
@@ -65,8 +65,7 @@ public interface NotificationEntryListener {
|
||||
/**
|
||||
* Called when a notification's views are inflated for the first time.
|
||||
*/
|
||||
default void onEntryInflated(NotificationEntry entry,
|
||||
@NotificationInflater.InflationFlag int inflatedFlags) {
|
||||
default void onEntryInflated(NotificationEntry entry, @InflationFlag int inflatedFlags) {
|
||||
}
|
||||
|
||||
/**
|
||||
|
||||
@@ -36,8 +36,8 @@ import com.android.systemui.statusbar.notification.collection.NotificationData;
|
||||
import com.android.systemui.statusbar.notification.collection.NotificationData.KeyguardEnvironment;
|
||||
import com.android.systemui.statusbar.notification.collection.NotificationEntry;
|
||||
import com.android.systemui.statusbar.notification.logging.NotificationLogger;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationInflater;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationInflater.InflationFlag;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationContentInflater;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationContentInflater.InflationFlag;
|
||||
import com.android.systemui.statusbar.notification.stack.NotificationListContainer;
|
||||
import com.android.systemui.statusbar.policy.HeadsUpManager;
|
||||
import com.android.systemui.util.leak.LeakDetector;
|
||||
@@ -56,7 +56,7 @@ import java.util.Map;
|
||||
*/
|
||||
public class NotificationEntryManager implements
|
||||
Dumpable,
|
||||
NotificationInflater.InflationCallback,
|
||||
NotificationContentInflater.InflationCallback,
|
||||
NotificationUpdateHandler,
|
||||
VisualStabilityManager.Callback {
|
||||
private static final String TAG = "NotificationEntryMgr";
|
||||
|
||||
@@ -19,8 +19,8 @@ package com.android.systemui.statusbar.notification;
|
||||
import static com.android.internal.util.Preconditions.checkNotNull;
|
||||
import static com.android.systemui.Dependency.ALLOW_NOTIFICATION_LONG_PRESS_NAME;
|
||||
import static com.android.systemui.statusbar.NotificationRemoteInputManager.ENABLE_REMOTE_INPUT;
|
||||
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 static com.android.systemui.statusbar.notification.row.NotificationContentInflater.FLAG_CONTENT_VIEW_AMBIENT;
|
||||
import static com.android.systemui.statusbar.notification.row.NotificationContentInflater.FLAG_CONTENT_VIEW_HEADS_UP;
|
||||
|
||||
import android.annotation.Nullable;
|
||||
import android.content.Context;
|
||||
@@ -42,8 +42,8 @@ import com.android.systemui.statusbar.NotificationUiAdjustment;
|
||||
import com.android.systemui.statusbar.notification.collection.NotificationEntry;
|
||||
import com.android.systemui.statusbar.notification.logging.NotificationLogger;
|
||||
import com.android.systemui.statusbar.notification.row.ExpandableNotificationRow;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationContentInflater;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationGutsManager;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationInflater;
|
||||
import com.android.systemui.statusbar.notification.row.RowInflaterTask;
|
||||
import com.android.systemui.statusbar.notification.stack.NotificationListContainer;
|
||||
import com.android.systemui.statusbar.phone.NotificationGroupManager;
|
||||
@@ -78,7 +78,7 @@ public class NotificationRowBinder {
|
||||
private NotificationPresenter mPresenter;
|
||||
private NotificationListContainer mListContainer;
|
||||
private HeadsUpManager mHeadsUpManager;
|
||||
private NotificationInflater.InflationCallback mInflationCallback;
|
||||
private NotificationContentInflater.InflationCallback mInflationCallback;
|
||||
private ExpandableNotificationRow.OnAppOpsClickListener mOnAppOpsClickListener;
|
||||
private BindRowCallback mBindRowCallback;
|
||||
private NotificationClicker mNotificationClicker;
|
||||
@@ -105,7 +105,7 @@ public class NotificationRowBinder {
|
||||
public void setUpWithPresenter(NotificationPresenter presenter,
|
||||
NotificationListContainer listContainer,
|
||||
HeadsUpManager headsUpManager,
|
||||
NotificationInflater.InflationCallback inflationCallback,
|
||||
NotificationContentInflater.InflationCallback inflationCallback,
|
||||
BindRowCallback bindRowCallback) {
|
||||
mPresenter = presenter;
|
||||
mListContainer = listContainer;
|
||||
|
||||
@@ -54,8 +54,8 @@ import com.android.systemui.statusbar.InflationTask;
|
||||
import com.android.systemui.statusbar.StatusBarIconView;
|
||||
import com.android.systemui.statusbar.notification.InflationException;
|
||||
import com.android.systemui.statusbar.notification.row.ExpandableNotificationRow;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationContentInflater.InflationFlag;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationGuts;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationInflater;
|
||||
|
||||
import java.util.ArrayList;
|
||||
import java.util.Collections;
|
||||
@@ -515,7 +515,7 @@ public final class NotificationEntry {
|
||||
if (row != null) row.resetUserExpansion();
|
||||
}
|
||||
|
||||
public void freeContentViewWhenSafe(@NotificationInflater.InflationFlag int inflationFlag) {
|
||||
public void freeContentViewWhenSafe(@InflationFlag int inflationFlag) {
|
||||
if (row != null) row.freeContentViewWhenSafe(inflationFlag);
|
||||
}
|
||||
|
||||
|
||||
@@ -17,13 +17,13 @@
|
||||
package com.android.systemui.statusbar.notification.row;
|
||||
|
||||
import static com.android.systemui.statusbar.notification.ActivityLaunchAnimator.ExpandAnimationParameters;
|
||||
import static com.android.systemui.statusbar.notification.row.NotificationContentInflater.FLAG_CONTENT_VIEW_AMBIENT;
|
||||
import static com.android.systemui.statusbar.notification.row.NotificationContentInflater.FLAG_CONTENT_VIEW_HEADS_UP;
|
||||
import static com.android.systemui.statusbar.notification.row.NotificationContentInflater.FLAG_CONTENT_VIEW_PUBLIC;
|
||||
import static com.android.systemui.statusbar.notification.row.NotificationContentInflater.InflationCallback;
|
||||
import static com.android.systemui.statusbar.notification.row.NotificationContentView.VISIBLE_TYPE_AMBIENT;
|
||||
import static com.android.systemui.statusbar.notification.row.NotificationContentView.VISIBLE_TYPE_CONTRACTED;
|
||||
import static com.android.systemui.statusbar.notification.row.NotificationContentView.VISIBLE_TYPE_HEADSUP;
|
||||
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 static com.android.systemui.statusbar.notification.row.NotificationInflater.FLAG_CONTENT_VIEW_PUBLIC;
|
||||
import static com.android.systemui.statusbar.notification.row.NotificationInflater.InflationCallback;
|
||||
|
||||
import android.animation.Animator;
|
||||
import android.animation.AnimatorListenerAdapter;
|
||||
@@ -89,7 +89,7 @@ import com.android.systemui.statusbar.notification.NotificationUtils;
|
||||
import com.android.systemui.statusbar.notification.VisualStabilityManager;
|
||||
import com.android.systemui.statusbar.notification.collection.NotificationEntry;
|
||||
import com.android.systemui.statusbar.notification.logging.NotificationCounters;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationInflater.InflationFlag;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationContentInflater.InflationFlag;
|
||||
import com.android.systemui.statusbar.notification.row.wrapper.NotificationMediaTemplateViewWrapper;
|
||||
import com.android.systemui.statusbar.notification.row.wrapper.NotificationViewWrapper;
|
||||
import com.android.systemui.statusbar.notification.stack.AmbientState;
|
||||
@@ -134,7 +134,7 @@ public class ExpandableNotificationRow extends ActivatableNotificationView
|
||||
}
|
||||
|
||||
private LayoutListener mLayoutListener;
|
||||
private final NotificationInflater mNotificationInflater;
|
||||
private final NotificationContentInflater mNotificationInflater;
|
||||
private int mIconTransformContentShift;
|
||||
private int mIconTransformContentShiftNoIcon;
|
||||
private int mMaxHeadsUpHeightBeforeN;
|
||||
@@ -1616,7 +1616,7 @@ public class ExpandableNotificationRow extends ActivatableNotificationView
|
||||
}
|
||||
|
||||
@VisibleForTesting
|
||||
public NotificationInflater getNotificationInflater() {
|
||||
public NotificationContentInflater getNotificationInflater() {
|
||||
return mNotificationInflater;
|
||||
}
|
||||
|
||||
@@ -1631,7 +1631,7 @@ public class ExpandableNotificationRow extends ActivatableNotificationView
|
||||
public ExpandableNotificationRow(Context context, AttributeSet attrs) {
|
||||
super(context, attrs);
|
||||
mFalsingManager = FalsingManager.getInstance(context);
|
||||
mNotificationInflater = new NotificationInflater(this);
|
||||
mNotificationInflater = new NotificationContentInflater(this);
|
||||
mMenuRow = new NotificationMenuRow(mContext);
|
||||
mImageResolver = new NotificationInlineImageResolver(context,
|
||||
new NotificationInlineImageCache());
|
||||
|
||||
@@ -55,9 +55,9 @@ import java.util.concurrent.atomic.AtomicInteger;
|
||||
/**
|
||||
* A utility that inflates the right kind of contentView based on the state
|
||||
*/
|
||||
public class NotificationInflater {
|
||||
public class NotificationContentInflater {
|
||||
|
||||
public static final String TAG = "NotificationInflater";
|
||||
public static final String TAG = "NotifContentInflater";
|
||||
|
||||
@Retention(RetentionPolicy.SOURCE)
|
||||
@IntDef(flag = true,
|
||||
@@ -127,7 +127,7 @@ public class NotificationInflater {
|
||||
private boolean mRedactAmbient;
|
||||
private final ArrayMap<Integer, RemoteViews> mCachedContentViews = new ArrayMap<>();
|
||||
|
||||
public NotificationInflater(ExpandableNotificationRow row) {
|
||||
public NotificationContentInflater(ExpandableNotificationRow row) {
|
||||
mRow = row;
|
||||
}
|
||||
|
||||
@@ -232,8 +232,7 @@ public class NotificationInflater {
|
||||
* will reinflate it.
|
||||
*
|
||||
* @param reInflateFlags flags which views should be inflated. Should be a subset of
|
||||
* {@link NotificationInflater#mInflationFlags} as only those will be
|
||||
* inflated/reinflated.
|
||||
* {@link #mInflationFlags} as only those will be inflated/reinflated.
|
||||
*/
|
||||
private void inflateNotificationViews(@InflationFlag int reInflateFlags) {
|
||||
if (mRow.isRemoved()) {
|
||||
@@ -52,7 +52,7 @@ public class NotificationInlineImageCache implements NotificationInlineImageReso
|
||||
@Override
|
||||
public void preload(Uri uri) {
|
||||
PreloadImageTask newTask = new PreloadImageTask(mResolver);
|
||||
newTask.executeOnExecutor(NotificationInflater.EXECUTOR, uri);
|
||||
newTask.executeOnExecutor(NotificationContentInflater.EXECUTOR, uri);
|
||||
mCache.put(uri, newTask);
|
||||
}
|
||||
|
||||
|
||||
@@ -34,8 +34,8 @@ import com.android.systemui.statusbar.InflationTask;
|
||||
import com.android.systemui.statusbar.notification.NotificationEntryListener;
|
||||
import com.android.systemui.statusbar.notification.NotificationEntryManager;
|
||||
import com.android.systemui.statusbar.notification.collection.NotificationEntry;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationInflater.AsyncInflationTask;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationInflater.InflationFlag;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationContentInflater.AsyncInflationTask;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationContentInflater.InflationFlag;
|
||||
import com.android.systemui.statusbar.phone.NotificationGroupManager.NotificationGroup;
|
||||
import com.android.systemui.statusbar.phone.NotificationGroupManager.OnGroupChangeListener;
|
||||
import com.android.systemui.statusbar.policy.HeadsUpManager;
|
||||
|
||||
@@ -16,7 +16,7 @@
|
||||
|
||||
package com.android.systemui.statusbar.policy;
|
||||
|
||||
import static com.android.systemui.statusbar.notification.row.NotificationInflater.FLAG_CONTENT_VIEW_HEADS_UP;
|
||||
import static com.android.systemui.statusbar.notification.row.NotificationContentInflater.FLAG_CONTENT_VIEW_HEADS_UP;
|
||||
|
||||
import android.annotation.NonNull;
|
||||
import android.annotation.Nullable;
|
||||
@@ -31,7 +31,7 @@ import com.android.internal.logging.MetricsLogger;
|
||||
import com.android.systemui.R;
|
||||
import com.android.systemui.statusbar.AlertingNotificationManager;
|
||||
import com.android.systemui.statusbar.notification.collection.NotificationEntry;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationInflater.InflationFlag;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationContentInflater.InflationFlag;
|
||||
|
||||
import java.io.FileDescriptor;
|
||||
import java.io.PrintWriter;
|
||||
|
||||
@@ -17,7 +17,7 @@
|
||||
|
||||
package com.android.systemui.statusbar;
|
||||
|
||||
import static com.android.systemui.statusbar.notification.row.NotificationInflater.FLAG_CONTENT_VIEW_CONTRACTED;
|
||||
import static com.android.systemui.statusbar.notification.row.NotificationContentInflater.FLAG_CONTENT_VIEW_CONTRACTED;
|
||||
|
||||
import static junit.framework.Assert.assertFalse;
|
||||
import static junit.framework.Assert.assertTrue;
|
||||
|
||||
@@ -39,8 +39,8 @@ import com.android.systemui.R;
|
||||
import com.android.systemui.bubbles.BubblesTestActivity;
|
||||
import com.android.systemui.statusbar.notification.collection.NotificationEntry;
|
||||
import com.android.systemui.statusbar.notification.row.ExpandableNotificationRow;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationInflater.InflationFlag;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationInflaterTest;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationContentInflater.InflationFlag;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationContentInflaterTest;
|
||||
import com.android.systemui.statusbar.phone.HeadsUpManagerPhone;
|
||||
import com.android.systemui.statusbar.phone.NotificationGroupManager;
|
||||
import com.android.systemui.statusbar.policy.HeadsUpManager;
|
||||
@@ -279,7 +279,7 @@ public class NotificationTestHelper {
|
||||
entry.channel.setBlockableSystem(true);
|
||||
row.setEntry(entry);
|
||||
row.getNotificationInflater().addInflationFlags(extraInflationFlags);
|
||||
NotificationInflaterTest.runThenWaitForInflation(
|
||||
NotificationContentInflaterTest.runThenWaitForInflation(
|
||||
() -> row.inflateViews(),
|
||||
row.getNotificationInflater());
|
||||
|
||||
|
||||
@@ -68,8 +68,8 @@ import com.android.systemui.statusbar.notification.collection.NotificationData;
|
||||
import com.android.systemui.statusbar.notification.collection.NotificationData.KeyguardEnvironment;
|
||||
import com.android.systemui.statusbar.notification.collection.NotificationEntry;
|
||||
import com.android.systemui.statusbar.notification.row.ExpandableNotificationRow;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationContentInflater.InflationFlag;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationGutsManager;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationInflater;
|
||||
import com.android.systemui.statusbar.notification.row.RowInflaterTask;
|
||||
import com.android.systemui.statusbar.notification.stack.NotificationListContainer;
|
||||
import com.android.systemui.statusbar.phone.NotificationGroupManager;
|
||||
@@ -143,7 +143,7 @@ public class NotificationEntryManagerTest extends SysuiTestCase {
|
||||
|
||||
@Override
|
||||
public void onAsyncInflationFinished(NotificationEntry entry,
|
||||
@NotificationInflater.InflationFlag int inflatedFlags) {
|
||||
@InflationFlag int inflatedFlags) {
|
||||
super.onAsyncInflationFinished(entry, inflatedFlags);
|
||||
|
||||
mCountDownLatch.countDown();
|
||||
|
||||
@@ -18,9 +18,9 @@ package com.android.systemui.statusbar.notification.row;
|
||||
|
||||
import static android.app.NotificationManager.IMPORTANCE_DEFAULT;
|
||||
|
||||
import static com.android.systemui.statusbar.notification.row.NotificationInflater.FLAG_CONTENT_VIEW_ALL;
|
||||
import static com.android.systemui.statusbar.notification.row.NotificationInflater.FLAG_CONTENT_VIEW_HEADS_UP;
|
||||
import static com.android.systemui.statusbar.notification.row.NotificationInflater.FLAG_CONTENT_VIEW_PUBLIC;
|
||||
import static com.android.systemui.statusbar.notification.row.NotificationContentInflater.FLAG_CONTENT_VIEW_ALL;
|
||||
import static com.android.systemui.statusbar.notification.row.NotificationContentInflater.FLAG_CONTENT_VIEW_HEADS_UP;
|
||||
import static com.android.systemui.statusbar.notification.row.NotificationContentInflater.FLAG_CONTENT_VIEW_PUBLIC;
|
||||
|
||||
import static org.junit.Assert.assertEquals;
|
||||
import static org.junit.Assert.assertFalse;
|
||||
|
||||
@@ -16,11 +16,11 @@
|
||||
|
||||
package com.android.systemui.statusbar.notification.row;
|
||||
|
||||
import static com.android.systemui.statusbar.notification.row.NotificationInflater.FLAG_CONTENT_VIEW_ALL;
|
||||
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_EXPANDED;
|
||||
import static com.android.systemui.statusbar.notification.row.NotificationInflater.FLAG_CONTENT_VIEW_HEADS_UP;
|
||||
import static com.android.systemui.statusbar.notification.row.NotificationInflater.FLAG_CONTENT_VIEW_PUBLIC;
|
||||
import static com.android.systemui.statusbar.notification.row.NotificationContentInflater.FLAG_CONTENT_VIEW_ALL;
|
||||
import static com.android.systemui.statusbar.notification.row.NotificationContentInflater.FLAG_CONTENT_VIEW_AMBIENT;
|
||||
import static com.android.systemui.statusbar.notification.row.NotificationContentInflater.FLAG_CONTENT_VIEW_EXPANDED;
|
||||
import static com.android.systemui.statusbar.notification.row.NotificationContentInflater.FLAG_CONTENT_VIEW_HEADS_UP;
|
||||
import static com.android.systemui.statusbar.notification.row.NotificationContentInflater.FLAG_CONTENT_VIEW_PUBLIC;
|
||||
|
||||
import static org.junit.Assert.assertEquals;
|
||||
import static org.junit.Assert.assertNotNull;
|
||||
@@ -49,6 +49,7 @@ import com.android.systemui.SysuiTestCase;
|
||||
import com.android.systemui.statusbar.InflationTask;
|
||||
import com.android.systemui.statusbar.NotificationTestHelper;
|
||||
import com.android.systemui.statusbar.notification.collection.NotificationEntry;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationContentInflater.InflationCallback;
|
||||
|
||||
import org.junit.Assert;
|
||||
import org.junit.Before;
|
||||
@@ -64,9 +65,9 @@ import java.util.concurrent.TimeUnit;
|
||||
@SmallTest
|
||||
@RunWith(AndroidTestingRunner.class)
|
||||
@RunWithLooper(setAsMainLooper = true)
|
||||
public class NotificationInflaterTest extends SysuiTestCase {
|
||||
public class NotificationContentInflaterTest extends SysuiTestCase {
|
||||
|
||||
private NotificationInflater mNotificationInflater;
|
||||
private NotificationContentInflater mNotificationInflater;
|
||||
private Notification.Builder mBuilder;
|
||||
private ExpandableNotificationRow mRow;
|
||||
|
||||
@@ -80,8 +81,8 @@ public class NotificationInflaterTest extends SysuiTestCase {
|
||||
ExpandableNotificationRow row = new NotificationTestHelper(mContext).createRow(
|
||||
mBuilder.build());
|
||||
mRow = spy(row);
|
||||
mNotificationInflater = new NotificationInflater(mRow);
|
||||
mNotificationInflater.setInflationCallback(new NotificationInflater.InflationCallback() {
|
||||
mNotificationInflater = new NotificationContentInflater(mRow);
|
||||
mNotificationInflater.setInflationCallback(new InflationCallback() {
|
||||
@Override
|
||||
public void handleInflationException(StatusBarNotification notification,
|
||||
Exception e) {
|
||||
@@ -89,7 +90,7 @@ public class NotificationInflaterTest extends SysuiTestCase {
|
||||
|
||||
@Override
|
||||
public void onAsyncInflationFinished(NotificationEntry entry,
|
||||
@NotificationInflater.InflationFlag int inflatedFlags) {
|
||||
@NotificationContentInflater.InflationFlag int inflatedFlags) {
|
||||
}
|
||||
});
|
||||
}
|
||||
@@ -158,14 +159,14 @@ public class NotificationInflaterTest extends SysuiTestCase {
|
||||
@Test
|
||||
@Ignore
|
||||
public void testInflationIsRetriedIfAsyncFails() throws Exception {
|
||||
NotificationInflater.InflationProgress result =
|
||||
new NotificationInflater.InflationProgress();
|
||||
NotificationContentInflater.InflationProgress result =
|
||||
new NotificationContentInflater.InflationProgress();
|
||||
result.packageContext = mContext;
|
||||
CountDownLatch countDownLatch = new CountDownLatch(1);
|
||||
NotificationInflater.applyRemoteView(result, FLAG_CONTENT_VIEW_EXPANDED, 0,
|
||||
NotificationContentInflater.applyRemoteView(result, FLAG_CONTENT_VIEW_EXPANDED, 0,
|
||||
new ArrayMap() /* cachedContentViews */, mRow, false /* redactAmbient */,
|
||||
true /* isNewView */, (v, p, r) -> true,
|
||||
new NotificationInflater.InflationCallback() {
|
||||
new InflationCallback() {
|
||||
@Override
|
||||
public void handleInflationException(StatusBarNotification notification,
|
||||
Exception e) {
|
||||
@@ -175,11 +176,11 @@ public class NotificationInflaterTest extends SysuiTestCase {
|
||||
|
||||
@Override
|
||||
public void onAsyncInflationFinished(NotificationEntry entry,
|
||||
@NotificationInflater.InflationFlag int inflatedFlags) {
|
||||
@NotificationContentInflater.InflationFlag int inflatedFlags) {
|
||||
countDownLatch.countDown();
|
||||
}
|
||||
}, mRow.getPrivateLayout(), null, null, new HashMap<>(),
|
||||
new NotificationInflater.ApplyCallback() {
|
||||
new NotificationContentInflater.ApplyCallback() {
|
||||
@Override
|
||||
public void setResultView(View v) {
|
||||
}
|
||||
@@ -199,8 +200,8 @@ public class NotificationInflaterTest extends SysuiTestCase {
|
||||
mNotificationInflater.updateInflationFlag(FLAG_CONTENT_VIEW_PUBLIC, true);
|
||||
mNotificationInflater.updateNeedsRedaction(true);
|
||||
|
||||
NotificationInflater.AsyncInflationTask asyncInflationTask =
|
||||
(NotificationInflater.AsyncInflationTask) mRow.getEntry().getRunningTask();
|
||||
NotificationContentInflater.AsyncInflationTask asyncInflationTask =
|
||||
(NotificationContentInflater.AsyncInflationTask) mRow.getEntry().getRunningTask();
|
||||
assertEquals(FLAG_CONTENT_VIEW_AMBIENT | FLAG_CONTENT_VIEW_PUBLIC,
|
||||
asyncInflationTask.getReInflateFlags());
|
||||
asyncInflationTask.abort();
|
||||
@@ -217,8 +218,8 @@ public class NotificationInflaterTest extends SysuiTestCase {
|
||||
mNotificationInflater.setIsChildInGroup(true);
|
||||
|
||||
InflationTask runningTask = mRow.getEntry().getRunningTask();
|
||||
NotificationInflater.AsyncInflationTask asyncInflationTask =
|
||||
(NotificationInflater.AsyncInflationTask) runningTask;
|
||||
NotificationContentInflater.AsyncInflationTask asyncInflationTask =
|
||||
(NotificationContentInflater.AsyncInflationTask) runningTask;
|
||||
assertEquals("Successive inflations don't inherit the previous flags!",
|
||||
FLAG_CONTENT_VIEW_ALL, asyncInflationTask.getReInflateFlags());
|
||||
runningTask.abort();
|
||||
@@ -233,19 +234,19 @@ public class NotificationInflaterTest extends SysuiTestCase {
|
||||
R.layout.custom_view_dark));
|
||||
RemoteViews decoratedMediaView = mBuilder.createContentView();
|
||||
Assert.assertFalse("The decorated media style doesn't allow a view to be reapplied!",
|
||||
NotificationInflater.canReapplyRemoteView(mediaView, decoratedMediaView));
|
||||
NotificationContentInflater.canReapplyRemoteView(mediaView, decoratedMediaView));
|
||||
}
|
||||
|
||||
public static void runThenWaitForInflation(Runnable block,
|
||||
NotificationInflater inflater) throws Exception {
|
||||
NotificationContentInflater inflater) throws Exception {
|
||||
runThenWaitForInflation(block, false /* expectingException */, inflater);
|
||||
}
|
||||
|
||||
private static void runThenWaitForInflation(Runnable block, boolean expectingException,
|
||||
NotificationInflater inflater) throws Exception {
|
||||
NotificationContentInflater inflater) throws Exception {
|
||||
CountDownLatch countDownLatch = new CountDownLatch(1);
|
||||
final ExceptionHolder exceptionHolder = new ExceptionHolder();
|
||||
inflater.setInflationCallback(new NotificationInflater.InflationCallback() {
|
||||
inflater.setInflationCallback(new InflationCallback() {
|
||||
@Override
|
||||
public void handleInflationException(StatusBarNotification notification,
|
||||
Exception e) {
|
||||
@@ -257,7 +258,7 @@ public class NotificationInflaterTest extends SysuiTestCase {
|
||||
|
||||
@Override
|
||||
public void onAsyncInflationFinished(NotificationEntry entry,
|
||||
@NotificationInflater.InflationFlag int inflatedFlags) {
|
||||
@NotificationContentInflater.InflationFlag int inflatedFlags) {
|
||||
if (expectingException) {
|
||||
exceptionHolder.setException(new RuntimeException(
|
||||
"Inflation finished even though there should be an error"));
|
||||
Reference in New Issue
Block a user