Removing TODO since bug is unfeasible.
The QSLongPressEffect is tailored to each tile's own state when it comes
to touch handling and the long-press animation. Making the class a
singleton could introduce more difficulties (complex touch handling,
and state management) than benefits.
Test: presubmit
Flag: com.android.systemui.quick_settings_visual_haptics_longpress
Bug: 332902869
Change-Id: I6c1e30b05aee27f530ec2c4f0ab3dd2a686feb4b
diff --git a/packages/SystemUI/src/com/android/systemui/haptics/qs/QSLongPressEffect.kt b/packages/SystemUI/src/com/android/systemui/haptics/qs/QSLongPressEffect.kt
index 8966209..e09e198 100644
--- a/packages/SystemUI/src/com/android/systemui/haptics/qs/QSLongPressEffect.kt
+++ b/packages/SystemUI/src/com/android/systemui/haptics/qs/QSLongPressEffect.kt
@@ -45,7 +45,6 @@
* @property[vibratorHelper] The [VibratorHelper] to deliver haptic effects.
* @property[effectDuration] The duration of the effect in ms.
*/
-// TODO(b/332902869): In addition from being injectable, we can consider making it a singleton
class QSLongPressEffect
@Inject
constructor(