Doze: Workaround vibration-related pickup gestures.

Ensure the pickup is registered a safe period of time after the
last notification before resetting the notification pulse schedule.

Bug:17496795
Change-Id: I0edc9bcbdf078cbf55df5935744a3eee7e902b5d
This commit is contained in:
John Spurlock
2014-09-16 09:12:03 -04:00
parent 41f766394d
commit 50a8ea6ec2
3 changed files with 17 additions and 1 deletions

View File

@@ -203,6 +203,9 @@
<!-- Doze: maximum number of times the notification pulse schedule can be reset -->
<integer name="doze_pulse_schedule_resets">3</integer>
<!-- Doze: duration to avoid false pickup gestures triggered by notification vibrations -->
<integer name="doze_pickup_vibration_threshold">2000</integer>
<!-- Doze: pulse parameter - how long does it take to fade in? -->
<integer name="doze_pulse_duration_in">1000</integer>

View File

@@ -411,7 +411,15 @@ public class DozeService extends DreamService {
}
requestPulse();
setListening(true); // reregister, this sensor only fires once
resetNotificationResets();
// reset the notification pulse schedule, but only if we think we were not triggered
// by a notification-related vibration
final long timeSinceNotification = System.currentTimeMillis() - mNotificationPulseTime;
if (timeSinceNotification < mDozeParameters.getPickupVibrationThreshold()) {
if (DEBUG) Log.d(mTag, "Not resetting schedule, recent notification");
} else {
resetNotificationResets();
}
}
}
}

View File

@@ -57,6 +57,7 @@ public class DozeParameters {
pw.print(" getPulseOnNotifications(): "); pw.println(getPulseOnNotifications());
pw.print(" getPulseSchedule(): "); pw.println(getPulseSchedule());
pw.print(" getPulseScheduleResets(): "); pw.println(getPulseScheduleResets());
pw.print(" getPickupVibrationThreshold(): "); pw.println(getPickupVibrationThreshold());
}
public boolean getDisplayStateSupported() {
@@ -111,6 +112,10 @@ public class DozeParameters {
return getInt("doze.pulse.schedule.resets", R.integer.doze_pulse_schedule_resets);
}
public int getPickupVibrationThreshold() {
return getInt("doze.pickup.vibration.threshold", R.integer.doze_pickup_vibration_threshold);
}
private boolean getBoolean(String propName, int resId) {
return SystemProperties.getBoolean(propName, mContext.getResources().getBoolean(resId));
}