Fix Xiaomi haptic feedback crash (#374)

Add the VIBRATE permission and guard View.performHapticFeedback so Xiaomi/Android 16 devices do not crash when haptic feedback reaches the vibrator service without permission.

Reported by A**** K***.

Author: rosuH

app/src/main/AndroidManifest.xml

@@ -8,6 +8,7 @@
     <uses-permission
         android:name="android.permission.READ_EXTERNAL_STORAGE"
         android:maxSdkVersion="32" />
+    <uses-permission android:name="android.permission.VIBRATE" />
 
     <!-- Devices running Android 12L (API level 32) or lower  -->
     <uses-permission android:name="android.permission.READ_EXTERNAL_STORAGE" android:maxSdkVersion="32" />
@@ -76,4 +77,4 @@
         </provider>
     </application>
 
-</manifest>
+</manifest>

app/src/main/java/me/rosuh/easywatermark/utils/VibrateHelper.kt

@@ -1,5 +1,6 @@
 package me.rosuh.easywatermark.utils
 
+import android.os.Build
 import android.view.HapticFeedbackConstants
 import android.view.View
 
@@ -9,18 +10,30 @@ class VibrateHelper private constructor() {
     private var cd: Long = 20L
 
     fun doVibrate(view: View) {
-        if (android.os.Build.VERSION.SDK_INT <= android.os.Build.VERSION_CODES.M) {
+        if (Build.VERSION.SDK_INT <= Build.VERSION_CODES.M) {
             return
         }
-        if (System.currentTimeMillis() - latestVibration <= cd) {
+        val now = System.currentTimeMillis()
+        if (now - latestVibration <= cd) {
             return
         }
-        view.performHapticFeedback(HapticFeedbackConstants.KEYBOARD_TAP)
+        latestVibration = now
+        performHapticFeedbackSafely {
+            view.performHapticFeedback(HapticFeedbackConstants.KEYBOARD_TAP)
+        }
     }
 
     companion object {
         fun get(): VibrateHelper {
             return VibrateHelper()
         }
+
+        internal fun performHapticFeedbackSafely(performFeedback: () -> Boolean): Boolean {
+            return try {
+                performFeedback()
+            } catch (_: SecurityException) {
+                false
+            }
+        }
     }
 }

app/src/test/java/me/rosuh/easywatermark/utils/VibrateHelperTest.kt

@@ -0,0 +1,22 @@
+package me.rosuh.easywatermark.utils
+
+import org.junit.Assert.assertFalse
+import org.junit.Assert.assertTrue
+import org.junit.Test
+
+class VibrateHelperTest {
+    @Test
+    fun performHapticFeedbackSafelyReturnsFalseWhenPermissionIsDenied() {
+        val result = VibrateHelper.performHapticFeedbackSafely {
+            throw SecurityException("missing android.permission.VIBRATE")
+        }
+
+        assertFalse(result)
+    }
+
+    @Test
+    fun performHapticFeedbackSafelyReturnsDelegateResult() {
+        assertTrue(VibrateHelper.performHapticFeedbackSafely { true })
+        assertFalse(VibrateHelper.performHapticFeedbackSafely { false })
+    }
+}