Merge "Add a clarifying comment" into qt-dev am: b1e4072518
am: 0d97074f05
Change-Id: Id61274b9365a04305fbb56d65fd011fda27ec432
This commit is contained in:
@@ -129,6 +129,13 @@ public abstract class VibrationPreferenceFragment extends RadioButtonPickerFragm
|
||||
Settings.System.putInt(getContext().getContentResolver(),
|
||||
getVibrationIntensitySetting(), candidate.getIntensity());
|
||||
} else {
|
||||
// We can't play preview effect here for all cases because that causes a data race
|
||||
// (VibratorService may access intensity settings before these settings are updated).
|
||||
// But we can't just play it in intensity settings update observer, because the
|
||||
// intensity settings are not changed if we turn the vibration off, then on.
|
||||
//
|
||||
// In this case we sould play the preview here.
|
||||
// To be refactored in b/132952771
|
||||
playVibrationPreview();
|
||||
}
|
||||
}
|
||||
|
Reference in New Issue
Block a user