Merge "Remove ZigZagClassifer from lock-icon longpress falsing algo" into sc-dev
This commit is contained in:
committed by
Android (Google) Code Review
commit
7326de8cc3
@@ -497,8 +497,10 @@ public class LockIconViewController extends ViewController<LockIconView> impleme
|
||||
if (!wasClickableOnDownEvent()) {
|
||||
return;
|
||||
}
|
||||
mDetectedLongPress = true;
|
||||
|
||||
if (mVibrator != null) {
|
||||
if (onAffordanceClick() && mVibrator != null) {
|
||||
// only vibrate if the click went through and wasn't intercepted by falsing
|
||||
mVibrator.vibrate(
|
||||
Process.myUid(),
|
||||
getContext().getOpPackageName(),
|
||||
@@ -506,8 +508,6 @@ public class LockIconViewController extends ViewController<LockIconView> impleme
|
||||
"lockIcon-onLongPress",
|
||||
VIBRATION_SONIFICATION_ATTRIBUTES);
|
||||
}
|
||||
mDetectedLongPress = true;
|
||||
onAffordanceClick();
|
||||
}
|
||||
|
||||
public boolean onSingleTapUp(MotionEvent e) {
|
||||
@@ -531,15 +531,21 @@ public class LockIconViewController extends ViewController<LockIconView> impleme
|
||||
return mDownDetected;
|
||||
}
|
||||
|
||||
private void onAffordanceClick() {
|
||||
/**
|
||||
* Whether we tried to launch the affordance.
|
||||
*
|
||||
* If falsing intercepts the click, returns false.
|
||||
*/
|
||||
private boolean onAffordanceClick() {
|
||||
if (mFalsingManager.isFalseTouch(LOCK_ICON)) {
|
||||
return;
|
||||
return false;
|
||||
}
|
||||
|
||||
// pre-emptively set to true to hide view
|
||||
mIsBouncerShowing = true;
|
||||
updateVisibility();
|
||||
mKeyguardViewController.showBouncer(/* scrim */ true);
|
||||
return true;
|
||||
}
|
||||
});
|
||||
|
||||
|
||||
@@ -21,6 +21,7 @@ import static com.android.internal.config.sysui.SystemUiDeviceConfigFlags.BRIGHT
|
||||
import static com.android.internal.config.sysui.SystemUiDeviceConfigFlags.BRIGHTLINE_FALSING_ZIGZAG_Y_PRIMARY_DEVIANCE;
|
||||
import static com.android.internal.config.sysui.SystemUiDeviceConfigFlags.BRIGHTLINE_FALSING_ZIGZAG_Y_SECONDARY_DEVIANCE;
|
||||
import static com.android.systemui.classifier.Classifier.BRIGHTNESS_SLIDER;
|
||||
import static com.android.systemui.classifier.Classifier.LOCK_ICON;
|
||||
import static com.android.systemui.classifier.Classifier.SHADE_DRAG;
|
||||
|
||||
import android.graphics.Point;
|
||||
@@ -89,7 +90,9 @@ class ZigZagClassifier extends FalsingClassifier {
|
||||
Result calculateFalsingResult(
|
||||
@Classifier.InteractionType int interactionType,
|
||||
double historyBelief, double historyConfidence) {
|
||||
if (interactionType == BRIGHTNESS_SLIDER || interactionType == SHADE_DRAG) {
|
||||
if (interactionType == BRIGHTNESS_SLIDER
|
||||
|| interactionType == SHADE_DRAG
|
||||
|| interactionType == LOCK_ICON) {
|
||||
return Result.passed(0);
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user