Limit the number of concurrent notifs appear on screen

To avoid an ANR in notification history

Test: Manual; with an app with 10k recent notifications
Bug: 183712803
Change-Id: Ia1340accadf43620c7b5a2c94ca1032cfe49ea59
This commit is contained in:
Julia Reynolds
2021-08-02 14:17:00 -04:00
parent ef23c240bb
commit abe9246caa
3 changed files with 14 additions and 9 deletions

View File

@@ -81,9 +81,10 @@
<com.android.settings.notification.history.NotificationHistoryRecyclerView
android:id="@+id/notification_list"
android:layout_width="match_parent"
android:layout_height="wrap_content"
android:layout_height="500dp"
android:clipChildren="true"
android:clipToPadding="true"
android:clipToOutline="true"
android:importantForAccessibility="yes"
android:scrollbars="none"/>

View File

@@ -71,6 +71,11 @@ public class NotificationHistoryAdapter extends
return new NotificationHistoryViewHolder(view);
}
@Override
public long getItemId(int position) {
return mValues.get(position).hashCode();
}
@Override
public void onBindViewHolder(final @NonNull NotificationHistoryViewHolder holder,
int position) {

View File

@@ -40,7 +40,6 @@ public class NotificationHistoryRecyclerView extends RecyclerView {
ItemTouchHelper touchHelper = new ItemTouchHelper(
new DismissTouchHelper(0, ItemTouchHelper.START | ItemTouchHelper.END));
touchHelper.attachToRecyclerView(this);
setNestedScrollingEnabled(false);
}
public void setOnItemSwipeDeleteListener(OnItemSwipeDeleteListener listener) {