Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Initial udfps vibration should always play

Browse Source 
Even if battery saver mode is enabled or
user manually disables vibration & haptics
in settings.

Test: manual, atest UdfpsControllerTest
Fixes: 197751697
Change-Id: I7f8bfcdd77b38dcd15b7641b0b851c3cb5c1beed
This commit is contained in:
Beverly
2021-08-25 13:09:32 -04:00
committed by Beverly Tai
parent e454d516ec
commit f8a240e525
2 changed files with 9 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
packages/SystemUI/src/com/android/systemui/biometrics/UdfpsController.java
View File

@@ -163,7 +163,8 @@ public class UdfpsController implements DozeReceiver {
public static final AudioAttributes VIBRATION_SONIFICATION_ATTRIBUTES =
new AudioAttributes.Builder()
.setContentType(AudioAttributes.CONTENT_TYPE_SONIFICATION)
.setUsage(AudioAttributes.USAGE_ASSISTANCE_SONIFICATION)
// vibration will bypass battery saver mode:
.setUsage(AudioAttributes.USAGE_ASSISTANCE_ACCESSIBILITY)
.build();
public static final VibrationEffect EFFECT_CLICK =

7
packages/SystemUI/tests/src/com/android/systemui/biometrics/UdfpsControllerTest.java
View File

@@ -16,6 +16,8 @@
package com.android.systemui.biometrics;
import static android.media.AudioAttributes.USAGE_ASSISTANCE_ACCESSIBILITY;
import static junit.framework.Assert.assertEquals;
import static org.mockito.ArgumentMatchers.any;
@@ -563,5 +565,10 @@ public class UdfpsControllerTest extends SysuiTestCase {
eq(mUdfpsController.EFFECT_CLICK),
eq("udfps-onStart"),
eq(UdfpsController.VIBRATION_SONIFICATION_ATTRIBUTES));
// THEN make sure vibration attributes has so that it always will play the haptic,
// even in battery saver mode
assertEquals(USAGE_ASSISTANCE_ACCESSIBILITY,
UdfpsController.VIBRATION_SONIFICATION_ATTRIBUTES.getUsage());
}
}