Fix final UDFPS enrollment haptic.
Test: Verified both accessibility/non-accessibility flows have correct haptic feedback. Fixes: 229804277 Change-Id: I8623771016b9819d4211d07a19b50291a8b06cd9 Merged-In: I8623771016b9819d4211d07a19b50291a8b06cd9
This commit is contained in:
committed by
Ilya Matyukhin
parent
9a8f44227e
commit
b7f3001cea
@@ -112,12 +112,6 @@ public class FingerprintEnrollEnrolling extends BiometricsEnrollEnrolling {
|
||||
private static final VibrationAttributes FINGERPRINT_ENROLLING_SONFICATION_ATTRIBUTES =
|
||||
VibrationAttributes.createForUsage(VibrationAttributes.USAGE_ACCESSIBILITY);
|
||||
|
||||
private static final VibrationAttributes HARDWARE_FEEDBACK_VIBRATION_ATTRIBUTES =
|
||||
VibrationAttributes.createForUsage(VibrationAttributes.USAGE_HARDWARE_FEEDBACK);
|
||||
|
||||
private static final VibrationEffect SUCCESS_VIBRATION_EFFECT =
|
||||
VibrationEffect.get(VibrationEffect.EFFECT_CLICK);
|
||||
|
||||
private FingerprintManager mFingerprintManager;
|
||||
private boolean mCanAssumeUdfps;
|
||||
@Nullable private ProgressBar mProgressBar;
|
||||
@@ -515,14 +509,6 @@ public class FingerprintEnrollEnrolling extends BiometricsEnrollEnrolling {
|
||||
mErrorText.removeCallbacks(mTouchAgainRunnable);
|
||||
mErrorText.postDelayed(mTouchAgainRunnable, HINT_TIMEOUT_DURATION);
|
||||
} else {
|
||||
if (mVibrator != null) {
|
||||
mVibrator.vibrate(Process.myUid(),
|
||||
getApplicationContext().getOpPackageName(),
|
||||
SUCCESS_VIBRATION_EFFECT,
|
||||
getClass().getSimpleName() + "::OnEnrollmentProgressChanged",
|
||||
HARDWARE_FEEDBACK_VIBRATION_ATTRIBUTES);
|
||||
}
|
||||
|
||||
if (mIsAccessibilityEnabled) {
|
||||
final int percent = (int) (((float)(steps - remaining) / (float) steps) * 100);
|
||||
CharSequence cs = getString(
|
||||
@@ -594,7 +580,8 @@ public class FingerprintEnrollEnrolling extends BiometricsEnrollEnrolling {
|
||||
mErrorText.setTranslationY(0f);
|
||||
}
|
||||
}
|
||||
if (isResumed() && (mIsAccessibilityEnabled || !mCanAssumeUdfps)) {
|
||||
|
||||
if (isResumed() && mIsAccessibilityEnabled && !mCanAssumeUdfps) {
|
||||
mVibrator.vibrate(Process.myUid(), getApplicationContext().getOpPackageName(),
|
||||
VIBRATE_EFFECT_ERROR, getClass().getSimpleName() + "::showError",
|
||||
FINGERPRINT_ENROLLING_SONFICATION_ATTRIBUTES);
|
||||
|
@@ -101,4 +101,4 @@ public class FingerprintEnrollEnrollingTest {
|
||||
|
||||
return callbackCaptor.getValue();
|
||||
}
|
||||
}
|
||||
}
|
Reference in New Issue
Block a user