From 89b4f0377fdcc3d26db0ea71b801f4ce6d2f04a5 Mon Sep 17 00:00:00 2001
From: Ashwin R C <ashwin2001achu@gmail.com>
Date: Wed, 27 May 2020 15:17:10 +0000
Subject: [PATCH] sm8250-common: KeyHandler: Use VibrationEffect

* Also update haptic feedback logic

Change-Id: I722022d014d9e88ef267a5563b43fee87ef02b6b
---
 KeyHandler/AndroidManifest.xml                            | 1 +
 .../src/org/lineageos/settings/device/KeyHandler.java     | 8 ++++----
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/KeyHandler/AndroidManifest.xml b/KeyHandler/AndroidManifest.xml
index 4b74ee3..ea8e344 100644
--- a/KeyHandler/AndroidManifest.xml
+++ b/KeyHandler/AndroidManifest.xml
@@ -17,4 +17,5 @@
 <manifest xmlns:android="http://schemas.android.com/apk/res/android"
           android:sharedUserId="android.uid.system"
           package="org.lineageos.settings.device">
+    <uses-permission android:name="android.permission.VIBRATE" />
 </manifest>
diff --git a/KeyHandler/src/org/lineageos/settings/device/KeyHandler.java b/KeyHandler/src/org/lineageos/settings/device/KeyHandler.java
index 49312a9..8cf4cf1 100644
--- a/KeyHandler/src/org/lineageos/settings/device/KeyHandler.java
+++ b/KeyHandler/src/org/lineageos/settings/device/KeyHandler.java
@@ -18,6 +18,7 @@ package org.lineageos.settings.device;
 
 import android.content.Context;
 import android.media.AudioManager;
+import android.os.VibrationEffect;
 import android.os.Vibrator;
 import android.view.KeyEvent;
 
@@ -64,10 +65,9 @@ public class KeyHandler implements DeviceKeyHandler {
     }
 
     private void doHapticFeedback() {
-        if (mVibrator == null || !mVibrator.hasVibrator()) {
-            return;
+        if (mVibrator != null && mVibrator.hasVibrator()) {
+            mVibrator.vibrate(VibrationEffect.createOneShot(50,
+                    VibrationEffect.DEFAULT_AMPLITUDE));
         }
-
-        mVibrator.vibrate(50);
     }
 }