Merge "Catch bubbles exception in PeopleTiles" into sc-dev
This commit is contained in:
committed by
Android (Google) Code Review
commit
2c95c7632a
@@ -240,9 +240,16 @@ public class NotificationHelper {
|
|||||||
/** Returns whether {@code entry} is suppressed from shade, meaning we should not show it. */
|
/** Returns whether {@code entry} is suppressed from shade, meaning we should not show it. */
|
||||||
public static boolean shouldFilterOut(
|
public static boolean shouldFilterOut(
|
||||||
Optional<Bubbles> bubblesOptional, NotificationEntry entry) {
|
Optional<Bubbles> bubblesOptional, NotificationEntry entry) {
|
||||||
return bubblesOptional.isPresent()
|
boolean isSuppressed = false;
|
||||||
&& bubblesOptional.get().isBubbleNotificationSuppressedFromShade(
|
//TODO(b/190822282): Investigate what is causing the NullPointerException
|
||||||
entry.getKey(), entry.getSbn().getGroupKey());
|
try {
|
||||||
|
isSuppressed = bubblesOptional.isPresent()
|
||||||
|
&& bubblesOptional.get().isBubbleNotificationSuppressedFromShade(
|
||||||
|
entry.getKey(), entry.getSbn().getGroupKey());
|
||||||
|
} catch (Exception e) {
|
||||||
|
Log.e(TAG, "Exception checking if notification is suppressed: " + e);
|
||||||
|
}
|
||||||
|
return isSuppressed;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user