Merge "NotificationChildrenContainer apply the roundness on NotificationHeaderViewWrapper too" into tm-qpr-dev
This commit is contained in:
committed by
Android (Google) Code Review
commit
ac576e7ee4
@@ -1431,6 +1431,22 @@ public class NotificationChildrenContainer extends ViewGroup
|
||||
@Override
|
||||
public void applyRoundnessAndInvalidate() {
|
||||
boolean last = true;
|
||||
if (mUseRoundnessSourceTypes) {
|
||||
if (mNotificationHeaderWrapper != null) {
|
||||
mNotificationHeaderWrapper.requestTopRoundness(
|
||||
/* value = */ getTopRoundness(),
|
||||
/* sourceType = */ FROM_PARENT,
|
||||
/* animate = */ false
|
||||
);
|
||||
}
|
||||
if (mNotificationHeaderWrapperLowPriority != null) {
|
||||
mNotificationHeaderWrapperLowPriority.requestTopRoundness(
|
||||
/* value = */ getTopRoundness(),
|
||||
/* sourceType = */ FROM_PARENT,
|
||||
/* animate = */ false
|
||||
);
|
||||
}
|
||||
}
|
||||
for (int i = mAttachedChildren.size() - 1; i >= 0; i--) {
|
||||
ExpandableNotificationRow child = mAttachedChildren.get(i);
|
||||
if (child.getVisibility() == View.GONE) {
|
||||
|
||||
@@ -29,6 +29,7 @@ import com.android.systemui.statusbar.notification.LegacySourceType;
|
||||
import com.android.systemui.statusbar.notification.SourceType;
|
||||
import com.android.systemui.statusbar.notification.row.ExpandableNotificationRow;
|
||||
import com.android.systemui.statusbar.notification.row.NotificationTestHelper;
|
||||
import com.android.systemui.statusbar.notification.row.wrapper.NotificationHeaderViewWrapper;
|
||||
|
||||
import org.junit.Assert;
|
||||
import org.junit.Before;
|
||||
@@ -216,4 +217,29 @@ public class NotificationChildrenContainerTest extends SysuiTestCase {
|
||||
Assert.assertEquals(1f, mChildrenContainer.getBottomRoundness(), 0.001f);
|
||||
Assert.assertEquals(1f, notificationRow.getBottomRoundness(), 0.001f);
|
||||
}
|
||||
|
||||
@Test
|
||||
public void applyRoundnessAndInvalidate_should_be_immediately_applied_on_header() {
|
||||
mChildrenContainer.useRoundnessSourceTypes(true);
|
||||
|
||||
NotificationHeaderViewWrapper header = mChildrenContainer.getNotificationHeaderWrapper();
|
||||
Assert.assertEquals(0f, header.getTopRoundness(), 0.001f);
|
||||
|
||||
mChildrenContainer.requestTopRoundness(1f, SourceType.from(""), false);
|
||||
|
||||
Assert.assertEquals(1f, header.getTopRoundness(), 0.001f);
|
||||
}
|
||||
|
||||
@Test
|
||||
public void applyRoundnessAndInvalidate_should_be_immediately_applied_on_headerLowPriority() {
|
||||
mChildrenContainer.useRoundnessSourceTypes(true);
|
||||
mChildrenContainer.setIsLowPriority(true);
|
||||
|
||||
NotificationHeaderViewWrapper header = mChildrenContainer.getNotificationHeaderWrapper();
|
||||
Assert.assertEquals(0f, header.getTopRoundness(), 0.001f);
|
||||
|
||||
mChildrenContainer.requestTopRoundness(1f, SourceType.from(""), false);
|
||||
|
||||
Assert.assertEquals(1f, header.getTopRoundness(), 0.001f);
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user