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
|
@Override
|
||||||
public void applyRoundnessAndInvalidate() {
|
public void applyRoundnessAndInvalidate() {
|
||||||
boolean last = true;
|
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--) {
|
for (int i = mAttachedChildren.size() - 1; i >= 0; i--) {
|
||||||
ExpandableNotificationRow child = mAttachedChildren.get(i);
|
ExpandableNotificationRow child = mAttachedChildren.get(i);
|
||||||
if (child.getVisibility() == View.GONE) {
|
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.SourceType;
|
||||||
import com.android.systemui.statusbar.notification.row.ExpandableNotificationRow;
|
import com.android.systemui.statusbar.notification.row.ExpandableNotificationRow;
|
||||||
import com.android.systemui.statusbar.notification.row.NotificationTestHelper;
|
import com.android.systemui.statusbar.notification.row.NotificationTestHelper;
|
||||||
|
import com.android.systemui.statusbar.notification.row.wrapper.NotificationHeaderViewWrapper;
|
||||||
|
|
||||||
import org.junit.Assert;
|
import org.junit.Assert;
|
||||||
import org.junit.Before;
|
import org.junit.Before;
|
||||||
@@ -216,4 +217,29 @@ public class NotificationChildrenContainerTest extends SysuiTestCase {
|
|||||||
Assert.assertEquals(1f, mChildrenContainer.getBottomRoundness(), 0.001f);
|
Assert.assertEquals(1f, mChildrenContainer.getBottomRoundness(), 0.001f);
|
||||||
Assert.assertEquals(1f, notificationRow.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