Merge "Fix regression of pressing power key cancel sfps enrollment" into tm-qpr-dev
This commit is contained in:
@@ -203,7 +203,9 @@ public class FingerprintEnrollEnrolling extends BiometricsEnrollEnrolling {
|
|||||||
// Do NOT cancel enrollment progress after rotating, adding mIsOrientationChanged
|
// Do NOT cancel enrollment progress after rotating, adding mIsOrientationChanged
|
||||||
// to judge if the focus changed was triggered by rotation, current WMS has triple callbacks
|
// to judge if the focus changed was triggered by rotation, current WMS has triple callbacks
|
||||||
// (true > false > true), we need to reset mIsOrientationChanged when !hasFocus callback.
|
// (true > false > true), we need to reset mIsOrientationChanged when !hasFocus callback.
|
||||||
if (!mIsOrientationChanged) {
|
// Side fps do not have to synchronize udfpsController overlay state, we should bypass sfps
|
||||||
|
// from onWindowFocusChanged() as long press sfps power key will prompt dialog to users.
|
||||||
|
if (!mIsOrientationChanged && !mCanAssumeSfps) {
|
||||||
onCancelEnrollment(FINGERPRINT_ERROR_USER_CANCELED);
|
onCancelEnrollment(FINGERPRINT_ERROR_USER_CANCELED);
|
||||||
} else {
|
} else {
|
||||||
mIsOrientationChanged = false;
|
mIsOrientationChanged = false;
|
||||||
|
@@ -222,6 +222,16 @@ public class FingerprintEnrollEnrollingTest {
|
|||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void fingerprintSfpsEnrollment_loseFocus_shouldNotCancel() {
|
||||||
|
initializeActivityFor(FingerprintSensorProperties.TYPE_POWER_BUTTON);
|
||||||
|
|
||||||
|
mActivity.onEnrollmentProgressChange(1, 1);
|
||||||
|
mActivity.onWindowFocusChanged(true);
|
||||||
|
|
||||||
|
verify(mActivity, never()).onCancelEnrollment(anyInt());
|
||||||
|
}
|
||||||
|
|
||||||
private void initializeActivityFor(int sensorType) {
|
private void initializeActivityFor(int sensorType) {
|
||||||
final List<ComponentInfoInternal> componentInfo = new ArrayList<>();
|
final List<ComponentInfoInternal> componentInfo = new ArrayList<>();
|
||||||
final FingerprintSensorPropertiesInternal prop =
|
final FingerprintSensorPropertiesInternal prop =
|
||||||
|
Reference in New Issue
Block a user