summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJeff DeCew <jeffdq@google.com>2022-05-31 15:59:00 +0000
committerJeff DeCew <jeffdq@google.com>2022-08-03 16:20:40 +0000
commit16186884ecaa1650d57fcca7d46394a80bcd3c2c (patch)
tree1162e4602a3e5b0f56349608d618d1df20bc750b
parent7cb144249f8d9284a3766c49e573fb0d52798abf (diff)
downloadbase-16186884ecaa1650d57fcca7d46394a80bcd3c2c.tar.gz
Always set the top roundness of child rows to 0.
The bug demonstrated that only the top radius was getting into a dirty state. When I saw this function was resetting the bottom roundness and never touching the top roundness, it was clear that we just needed to set the top roundness. Even though it's hard to reproduce the issue where top roundness is incorrectly huge, it's trivial to reproduce the issue where the top roundness is incorrectly small, and this fix definitively addresses that state. Test: post auto-groupable Heads Up notifications. Notice after grouping that child rows now never have any internal corner radii. Fixes: 227433308 Merged-In: I3cd1aa23ad8333f35ec837edd39bc12aeb1573b6 Change-Id: I3cd1aa23ad8333f35ec837edd39bc12aeb1573b6 (cherry picked from commit ef4782f55d7c00493fe6cd9e10c2a082c2a819e3)
-rw-r--r--packages/SystemUI/src/com/android/systemui/statusbar/notification/stack/NotificationChildrenContainer.java1
1 files changed, 1 insertions, 0 deletions
diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/notification/stack/NotificationChildrenContainer.java b/packages/SystemUI/src/com/android/systemui/statusbar/notification/stack/NotificationChildrenContainer.java
index a552f999aeb4..a76f0827fc18 100644
--- a/packages/SystemUI/src/com/android/systemui/statusbar/notification/stack/NotificationChildrenContainer.java
+++ b/packages/SystemUI/src/com/android/systemui/statusbar/notification/stack/NotificationChildrenContainer.java
@@ -1312,6 +1312,7 @@ public class NotificationChildrenContainer extends ViewGroup
}
float bottomRoundness = last ? currentBottomRoundness : 0.0f;
child.setBottomRoundness(bottomRoundness, isShown() /* animate */);
+ child.setTopRoundness(0.0f, false /* animate */);
last = false;
}
}