Merge change 21901 into eclair

* changes:
  Check for a timeout <= 0 before adding the vibration.
This commit is contained in:
Android (Google) Code Review
2009-08-19 11:50:17 -07:00

View File

@@ -141,8 +141,11 @@ public class HardwareService extends IHardwareService.Stub {
!= PackageManager.PERMISSION_GRANTED) {
throw new SecurityException("Requires VIBRATE permission");
}
if (mCurrentVibration != null
&& mCurrentVibration.hasLongerTimeout(milliseconds)) {
// We're running in the system server so we cannot crash. Check for a
// timeout of 0 or negative. This will ensure that a vibration has
// either a timeout of > 0 or a non-null pattern.
if (milliseconds <= 0 || (mCurrentVibration != null
&& mCurrentVibration.hasLongerTimeout(milliseconds))) {
// Ignore this vibration since the current vibration will play for
// longer than milliseconds.
return;