Merge changes from topic "am-2c480354c5314718960fe4d0b2502426" into tm-mainline-prod
* changes: Merge changes from topic "presubmit-am-30d5e3b252a64882bd8232ecbecd0e04" into sc-v2-dev-plus-aosp am:f93e57c620am:4c5c66d961[automerger skipped] [automerged blank] Import translations. DO NOT MERGE ANYWHERE 2p:f58839f636am:ea8c96050c-s ours am:f72d223b73-s ours
This commit is contained in:
committed by
Android (Google) Code Review
commit
5bae0b0733
@@ -475,7 +475,7 @@ public class UdfpsKeyguardViewController extends UdfpsAnimationViewController<Ud
|
||||
public void onPanelExpansionChanged(
|
||||
float fraction, boolean expanded, boolean tracking) {
|
||||
mPanelExpansionFraction =
|
||||
mKeyguardViewManager.isBouncerInTransit() ? BouncerPanelExpansionCalculator
|
||||
mKeyguardViewManager.bouncerIsInTransit() ? BouncerPanelExpansionCalculator
|
||||
.aboutToShowBouncerProgress(fraction) : fraction;
|
||||
updateAlpha();
|
||||
}
|
||||
|
||||
@@ -612,7 +612,7 @@ public class QSFragment extends LifecycleFragment implements QS, CommandQueue.Ca
|
||||
} else if (progress > 0 && view.getVisibility() != View.VISIBLE) {
|
||||
view.setVisibility((View.VISIBLE));
|
||||
}
|
||||
float alpha = mQSPanelController.isBouncerInTransit()
|
||||
float alpha = mQSPanelController.bouncerInTransit()
|
||||
? BouncerPanelExpansionCalculator.aboutToShowBouncerProgress(progress)
|
||||
: ShadeInterpolation.getContentAlpha(progress);
|
||||
view.setAlpha(alpha);
|
||||
|
||||
@@ -254,8 +254,8 @@ public class QSPanelController extends QSPanelControllerBase<QSPanel> {
|
||||
*
|
||||
* @return if bouncer is in transit
|
||||
*/
|
||||
public boolean isBouncerInTransit() {
|
||||
return mStatusBarKeyguardViewManager.isBouncerInTransit();
|
||||
public boolean bouncerInTransit() {
|
||||
return mStatusBarKeyguardViewManager.bouncerIsInTransit();
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
@@ -31,7 +31,6 @@ import com.android.systemui.statusbar.notification.row.ExpandableNotificationRow
|
||||
import com.android.systemui.statusbar.notification.row.ExpandableView;
|
||||
import com.android.systemui.statusbar.notification.stack.StackScrollAlgorithm.BypassController;
|
||||
import com.android.systemui.statusbar.notification.stack.StackScrollAlgorithm.SectionProvider;
|
||||
import com.android.systemui.statusbar.phone.StatusBarKeyguardViewManager;
|
||||
|
||||
import javax.inject.Inject;
|
||||
|
||||
@@ -46,10 +45,6 @@ public class AmbientState {
|
||||
|
||||
private final SectionProvider mSectionProvider;
|
||||
private final BypassController mBypassController;
|
||||
/**
|
||||
* Used to read bouncer states.
|
||||
*/
|
||||
private StatusBarKeyguardViewManager mStatusBarKeyguardViewManager;
|
||||
private int mScrollY;
|
||||
private boolean mDimmed;
|
||||
private ActivatableNotificationView mActivatedChild;
|
||||
@@ -209,11 +204,9 @@ public class AmbientState {
|
||||
public AmbientState(
|
||||
Context context,
|
||||
@NonNull SectionProvider sectionProvider,
|
||||
@NonNull BypassController bypassController,
|
||||
@Nullable StatusBarKeyguardViewManager statusBarKeyguardViewManager) {
|
||||
@NonNull BypassController bypassController) {
|
||||
mSectionProvider = sectionProvider;
|
||||
mBypassController = bypassController;
|
||||
mStatusBarKeyguardViewManager = statusBarKeyguardViewManager;
|
||||
reload(context);
|
||||
}
|
||||
|
||||
@@ -668,14 +661,4 @@ public class AmbientState {
|
||||
public int getStackTopMargin() {
|
||||
return mStackTopMargin;
|
||||
}
|
||||
|
||||
/**
|
||||
* Check to see if we are about to show bouncer.
|
||||
*
|
||||
* @return if bouncer expansion is between 0 and 1.
|
||||
*/
|
||||
public boolean isBouncerInTransit() {
|
||||
return mStatusBarKeyguardViewManager != null
|
||||
&& mStatusBarKeyguardViewManager.isBouncerInTransit();
|
||||
}
|
||||
}
|
||||
|
||||
@@ -27,7 +27,6 @@ import android.view.ViewGroup;
|
||||
import androidx.annotation.VisibleForTesting;
|
||||
|
||||
import com.android.internal.policy.SystemBarUtils;
|
||||
import com.android.keyguard.BouncerPanelExpansionCalculator;
|
||||
import com.android.systemui.R;
|
||||
import com.android.systemui.animation.ShadeInterpolation;
|
||||
import com.android.systemui.statusbar.EmptyShadeView;
|
||||
@@ -432,9 +431,7 @@ public class StackScrollAlgorithm {
|
||||
} else if (ambientState.isExpansionChanging()) {
|
||||
// Adjust alpha for shade open & close.
|
||||
float expansion = ambientState.getExpansionFraction();
|
||||
viewState.alpha = ambientState.isBouncerInTransit()
|
||||
? BouncerPanelExpansionCalculator.aboutToShowBouncerProgress(expansion)
|
||||
: ShadeInterpolation.getContentAlpha(expansion);
|
||||
viewState.alpha = ShadeInterpolation.getContentAlpha(expansion);
|
||||
}
|
||||
|
||||
if (ambientState.isShadeExpanded() && view.mustStayOnScreen()
|
||||
|
||||
@@ -48,6 +48,7 @@ import android.view.animation.Interpolator;
|
||||
import com.android.internal.jank.InteractionJankMonitor;
|
||||
import com.android.internal.logging.nano.MetricsProto.MetricsEvent;
|
||||
import com.android.internal.util.LatencyTracker;
|
||||
import com.android.keyguard.BouncerPanelExpansionCalculator;
|
||||
import com.android.systemui.DejankUtils;
|
||||
import com.android.systemui.R;
|
||||
import com.android.systemui.animation.Interpolators;
|
||||
@@ -795,7 +796,9 @@ public abstract class PanelViewController {
|
||||
}
|
||||
mExpandedFraction = Math.min(1f,
|
||||
maxPanelHeight == 0 ? 0 : mExpandedHeight / maxPanelHeight);
|
||||
mAmbientState.setExpansionFraction(mExpandedFraction);
|
||||
mAmbientState.setExpansionFraction(mStatusBarKeyguardViewManager.bouncerIsInTransit()
|
||||
? BouncerPanelExpansionCalculator.aboutToShowBouncerProgress(mExpandedFraction)
|
||||
: mExpandedFraction);
|
||||
onHeightUpdated(mExpandedHeight);
|
||||
updatePanelExpansionAndVisibility();
|
||||
});
|
||||
|
||||
@@ -1074,7 +1074,7 @@ public class ScrimController implements ViewTreeObserver.OnPreDrawListener, Dump
|
||||
}
|
||||
|
||||
private float getInterpolatedFraction() {
|
||||
if (mStatusBarKeyguardViewManager.isBouncerInTransit()) {
|
||||
if (mStatusBarKeyguardViewManager.bouncerIsInTransit()) {
|
||||
return BouncerPanelExpansionCalculator
|
||||
.aboutToShowBouncerProgress(mPanelExpansionFraction);
|
||||
}
|
||||
|
||||
@@ -1376,7 +1376,7 @@ public class StatusBarKeyguardViewManager implements RemoteInputController.Callb
|
||||
/**
|
||||
* Returns if bouncer expansion is between 0 and 1 non-inclusive.
|
||||
*/
|
||||
public boolean isBouncerInTransit() {
|
||||
public boolean bouncerIsInTransit() {
|
||||
if (mBouncer == null) return false;
|
||||
|
||||
return mBouncer.inTransit();
|
||||
|
||||
@@ -190,7 +190,7 @@ public class QSFragmentTest extends SysuiBaseFragmentTest {
|
||||
QSFragment fragment = resumeAndGetFragment();
|
||||
enableSplitShade();
|
||||
setStatusBarState(StatusBarState.KEYGUARD);
|
||||
when(mQSPanelController.isBouncerInTransit()).thenReturn(false);
|
||||
when(mQSPanelController.bouncerInTransit()).thenReturn(false);
|
||||
int transitionPxAmount = 123;
|
||||
float transitionProgress = 0.5f;
|
||||
|
||||
@@ -206,7 +206,7 @@ public class QSFragmentTest extends SysuiBaseFragmentTest {
|
||||
QSFragment fragment = resumeAndGetFragment();
|
||||
enableSplitShade();
|
||||
setStatusBarState(StatusBarState.KEYGUARD);
|
||||
when(mQSPanelController.isBouncerInTransit()).thenReturn(true);
|
||||
when(mQSPanelController.bouncerInTransit()).thenReturn(true);
|
||||
int transitionPxAmount = 123;
|
||||
float transitionProgress = 0.5f;
|
||||
|
||||
|
||||
@@ -61,7 +61,7 @@ class QSPanelControllerTest : SysuiTestCase() {
|
||||
whenever(brightnessSliderFactory.create(any(), any())).thenReturn(brightnessSlider)
|
||||
whenever(brightnessControllerFactory.create(any())).thenReturn(brightnessController)
|
||||
whenever(qsPanel.resources).thenReturn(mContext.orCreateTestableResources.resources)
|
||||
whenever(statusBarKeyguardViewManager.isBouncerInTransit()).thenReturn(false)
|
||||
whenever(statusBarKeyguardViewManager.bouncerIsInTransit()).thenReturn(false)
|
||||
|
||||
controller = QSPanelController(
|
||||
qsPanel,
|
||||
@@ -109,10 +109,10 @@ class QSPanelControllerTest : SysuiTestCase() {
|
||||
}
|
||||
|
||||
@Test
|
||||
fun testIsBouncerInTransit() {
|
||||
whenever(statusBarKeyguardViewManager.isBouncerInTransit()).thenReturn(true)
|
||||
assertThat(controller.isBouncerInTransit()).isEqualTo(true)
|
||||
whenever(statusBarKeyguardViewManager.isBouncerInTransit()).thenReturn(false)
|
||||
assertThat(controller.isBouncerInTransit()).isEqualTo(false)
|
||||
fun testBouncerIsInTransit() {
|
||||
whenever(statusBarKeyguardViewManager.bouncerIsInTransit()).thenReturn(true)
|
||||
assertThat(controller.bouncerInTransit()).isEqualTo(true)
|
||||
whenever(statusBarKeyguardViewManager.bouncerIsInTransit()).thenReturn(false)
|
||||
assertThat(controller.bouncerInTransit()).isEqualTo(false)
|
||||
}
|
||||
}
|
||||
|
||||
@@ -64,7 +64,6 @@ import com.android.systemui.statusbar.notification.row.FooterView;
|
||||
import com.android.systemui.statusbar.phone.CentralSurfaces;
|
||||
import com.android.systemui.statusbar.phone.KeyguardBypassController;
|
||||
import com.android.systemui.statusbar.phone.ShadeController;
|
||||
import com.android.systemui.statusbar.phone.StatusBarKeyguardViewManager;
|
||||
import com.android.systemui.statusbar.phone.UnlockedScreenOffAnimationController;
|
||||
|
||||
import org.junit.Assert;
|
||||
@@ -105,7 +104,6 @@ public class NotificationStackScrollLayoutTest extends SysuiTestCase {
|
||||
@Mock private UnlockedScreenOffAnimationController mUnlockedScreenOffAnimationController;
|
||||
@Mock private NotificationShelf mNotificationShelf;
|
||||
@Mock private NotificationStackSizeCalculator mNotificationStackSizeCalculator;
|
||||
@Mock private StatusBarKeyguardViewManager mStatusBarKeyguardViewManager;
|
||||
|
||||
@Before
|
||||
@UiThreadTest
|
||||
@@ -113,8 +111,7 @@ public class NotificationStackScrollLayoutTest extends SysuiTestCase {
|
||||
allowTestableLooperAsMainThread();
|
||||
|
||||
// Interact with real instance of AmbientState.
|
||||
mAmbientState = new AmbientState(mContext, mNotificationSectionsManager, mBypassController,
|
||||
mStatusBarKeyguardViewManager);
|
||||
mAmbientState = new AmbientState(mContext, mNotificationSectionsManager, mBypassController);
|
||||
|
||||
// Inject dependencies before initializing the layout
|
||||
mDependency.injectTestDependency(SysuiStatusBarStateController.class, mBarState);
|
||||
|
||||
@@ -8,7 +8,6 @@ import com.android.systemui.statusbar.EmptyShadeView
|
||||
import com.android.systemui.statusbar.notification.row.ExpandableNotificationRow
|
||||
import com.android.systemui.statusbar.notification.stack.StackScrollAlgorithm.BypassController
|
||||
import com.android.systemui.statusbar.notification.stack.StackScrollAlgorithm.SectionProvider
|
||||
import com.android.systemui.statusbar.phone.StatusBarKeyguardViewManager
|
||||
import com.google.common.truth.Truth.assertThat
|
||||
import org.junit.Before
|
||||
import org.junit.Test
|
||||
@@ -22,14 +21,10 @@ class StackScrollAlgorithmTest : SysuiTestCase() {
|
||||
private val stackScrollAlgorithm = StackScrollAlgorithm(context, hostView)
|
||||
private val expandableViewState = ExpandableViewState()
|
||||
private val notificationRow = mock(ExpandableNotificationRow::class.java)
|
||||
private val mStatusBarKeyguardViewManager = mock(StatusBarKeyguardViewManager::class.java)
|
||||
|
||||
private val ambientState = AmbientState(
|
||||
context,
|
||||
SectionProvider { _, _ -> false },
|
||||
BypassController { false },
|
||||
mStatusBarKeyguardViewManager
|
||||
)
|
||||
context,
|
||||
SectionProvider { _, _ -> false },
|
||||
BypassController { false })
|
||||
|
||||
@Before
|
||||
fun setUp() {
|
||||
|
||||
@@ -1238,7 +1238,7 @@ public class ScrimControllerTest extends SysuiTestCase {
|
||||
|
||||
@Test
|
||||
public void expansionNotificationAlpha_shadeLocked_bouncerActive_usesBouncerInterpolator() {
|
||||
when(mStatusBarKeyguardViewManager.isBouncerInTransit()).thenReturn(true);
|
||||
when(mStatusBarKeyguardViewManager.bouncerIsInTransit()).thenReturn(true);
|
||||
|
||||
mScrimController.transitionTo(ScrimState.SHADE_LOCKED);
|
||||
|
||||
@@ -1254,7 +1254,7 @@ public class ScrimControllerTest extends SysuiTestCase {
|
||||
|
||||
@Test
|
||||
public void expansionNotificationAlpha_shadeLocked_bouncerNotActive_usesShadeInterpolator() {
|
||||
when(mStatusBarKeyguardViewManager.isBouncerInTransit()).thenReturn(false);
|
||||
when(mStatusBarKeyguardViewManager.bouncerIsInTransit()).thenReturn(false);
|
||||
|
||||
mScrimController.transitionTo(ScrimState.SHADE_LOCKED);
|
||||
|
||||
@@ -1269,7 +1269,7 @@ public class ScrimControllerTest extends SysuiTestCase {
|
||||
|
||||
@Test
|
||||
public void notificationAlpha_unnocclusionAnimating_bouncerActive_usesKeyguardNotifAlpha() {
|
||||
when(mStatusBarKeyguardViewManager.isBouncerInTransit()).thenReturn(true);
|
||||
when(mStatusBarKeyguardViewManager.bouncerIsInTransit()).thenReturn(true);
|
||||
|
||||
mScrimController.transitionTo(ScrimState.KEYGUARD);
|
||||
mScrimController.setUnocclusionAnimationRunning(true);
|
||||
@@ -1290,7 +1290,7 @@ public class ScrimControllerTest extends SysuiTestCase {
|
||||
|
||||
@Test
|
||||
public void notificationAlpha_unnocclusionAnimating_bouncerNotActive_usesKeyguardNotifAlpha() {
|
||||
when(mStatusBarKeyguardViewManager.isBouncerInTransit()).thenReturn(false);
|
||||
when(mStatusBarKeyguardViewManager.bouncerIsInTransit()).thenReturn(false);
|
||||
|
||||
mScrimController.transitionTo(ScrimState.KEYGUARD);
|
||||
mScrimController.setUnocclusionAnimationRunning(true);
|
||||
@@ -1311,7 +1311,7 @@ public class ScrimControllerTest extends SysuiTestCase {
|
||||
|
||||
@Test
|
||||
public void notificationAlpha_inKeyguardState_bouncerActive_usesInvertedBouncerInterpolator() {
|
||||
when(mStatusBarKeyguardViewManager.isBouncerInTransit()).thenReturn(true);
|
||||
when(mStatusBarKeyguardViewManager.bouncerIsInTransit()).thenReturn(true);
|
||||
|
||||
mScrimController.transitionTo(ScrimState.KEYGUARD);
|
||||
|
||||
@@ -1330,7 +1330,7 @@ public class ScrimControllerTest extends SysuiTestCase {
|
||||
|
||||
@Test
|
||||
public void notificationAlpha_inKeyguardState_bouncerNotActive_usesInvertedShadeInterpolator() {
|
||||
when(mStatusBarKeyguardViewManager.isBouncerInTransit()).thenReturn(false);
|
||||
when(mStatusBarKeyguardViewManager.bouncerIsInTransit()).thenReturn(false);
|
||||
|
||||
mScrimController.transitionTo(ScrimState.KEYGUARD);
|
||||
|
||||
|
||||
@@ -438,12 +438,12 @@ public class StatusBarKeyguardViewManagerTest extends SysuiTestCase {
|
||||
}
|
||||
|
||||
@Test
|
||||
public void testIsBouncerInTransit() {
|
||||
public void testBouncerIsInTransit() {
|
||||
when(mBouncer.inTransit()).thenReturn(true);
|
||||
Truth.assertThat(mStatusBarKeyguardViewManager.isBouncerInTransit()).isTrue();
|
||||
Truth.assertThat(mStatusBarKeyguardViewManager.bouncerIsInTransit()).isTrue();
|
||||
when(mBouncer.inTransit()).thenReturn(false);
|
||||
Truth.assertThat(mStatusBarKeyguardViewManager.isBouncerInTransit()).isFalse();
|
||||
Truth.assertThat(mStatusBarKeyguardViewManager.bouncerIsInTransit()).isFalse();
|
||||
mBouncer = null;
|
||||
Truth.assertThat(mStatusBarKeyguardViewManager.isBouncerInTransit()).isFalse();
|
||||
Truth.assertThat(mStatusBarKeyguardViewManager.bouncerIsInTransit()).isFalse();
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user