Update Handler constructors from deprecated versions (#1726)

* Update Handler constructors from deprecated

* Change to mainLooper

* Add null checks for myLooper to prep looper

* Cleanup where looper was already prepared

* Replace mainLooper with myLooper for non UI tasks

Co-authored-by: Henigan <rheniga1@MGC12Z921DLVCG.fbpld77.ford.com>

Author: Robert Henigan

android/sdl_android/src/main/java/com/smartdevicelink/encoder/VirtualDisplayEncoder.java

@@ -337,7 +337,7 @@ public void run() {
             Looper.prepare();
 
             // create a Handler for this thread
-            mHandler = new Handler() {
+            mHandler = new Handler(Looper.myLooper()) {
                 public void handleMessage(Message msg) {
                     switch (msg.what) {
                         case MSG_TICK: {

android/sdl_android/src/main/java/com/smartdevicelink/managers/lockscreen/LockScreenManager.java

@@ -38,6 +38,7 @@
 import android.content.IntentFilter;
 import android.graphics.Bitmap;
 import android.os.Handler;
+import android.os.Looper;
 
 import androidx.annotation.RestrictTo;
 
@@ -227,7 +228,7 @@ public void onNotified(RPCNotification notification) {
                                         mLockScreenShouldBeAutoDismissed = false;
                                     }
                                     if (!receivedFirstDDNotification) {
-                                        new Handler().postDelayed(new Runnable() {
+                                        new Handler(Looper.getMainLooper()).postDelayed(new Runnable() {
                                             @Override
                                             public void run() {
                                                 launchLockScreenActivity();

android/sdl_android/src/main/java/com/smartdevicelink/protocol/heartbeat/HeartbeatMonitor.java

@@ -155,7 +155,7 @@ public void run() {
                         Looper.prepare();
                         mHeartbeatThreadLooper = Looper.myLooper();
 
-                        mHeartbeatThreadHandler = new Handler();
+                        mHeartbeatThreadHandler = new Handler(mHeartbeatThreadLooper);
                         mIsAckReceived = true;
                         isHeartbeatReceived = true;
 

android/sdl_android/src/main/java/com/smartdevicelink/transport/MultiplexBluetoothTransport.java

@@ -348,7 +348,10 @@ private void connectionLost() {
     }
 
     private void timerDelayRemoveDialog(final BluetoothSocket sock) {
-        timeOutHandler = new Handler();
+        if (Looper.myLooper() == null) {
+            Looper.prepare();
+        }
+        timeOutHandler = new Handler(Looper.myLooper());
         socketRunnable = new Runnable() {
             public void run() {
                 //Log.e(TAG, "BLUETOOTH SOCKET CONNECT TIMEOUT - ATTEMPT TO CLOSE SOCKET");

android/sdl_android/src/main/java/com/smartdevicelink/transport/SdlRouterService.java

@@ -1442,7 +1442,10 @@ private int getNotificationTimeout(String address, boolean confirmedDevice) {
     public void resetForegroundTimeOut(long delay) {
         synchronized (FOREGROUND_NOTIFICATION_LOCK) {
             if (foregroundTimeoutHandler == null) {
-                foregroundTimeoutHandler = new Handler();
+                if (Looper.myLooper() == null) {
+                    Looper.prepare();
+                }
+                foregroundTimeoutHandler = new Handler(Looper.myLooper());
             }
             if (foregroundTimeoutRunnable == null) {
                 foregroundTimeoutRunnable = new Runnable() {
@@ -2536,7 +2539,10 @@ protected static LocalRouterService getLocalRouterService(Intent launchIntent, C
      * This method is used to check for the newest version of this class to make sure the latest and greatest is up and running.
      */
     private void startAltTransportTimer() {
-        altTransportTimerHandler = new Handler();
+        if (Looper.myLooper() == null) {
+            Looper.prepare();
+        }
+        altTransportTimerHandler = new Handler(Looper.myLooper());
         altTransportTimerRunnable = new Runnable() {
             public void run() {
                 altTransportTimerHandler = null;
@@ -3095,7 +3101,10 @@ public RegisteredApp(String appId, int routerMessagingVersion, Messenger messeng
             this.messenger = messenger;
             this.sessionIds = new Vector<Long>();
             this.queues = new ConcurrentHashMap<>();
-            queueWaitHandler = new Handler();
+            if (Looper.myLooper() == null) {
+                Looper.prepare();
+            }
+            queueWaitHandler = new Handler(Looper.myLooper());
             registeredTransports = new SparseArray<ArrayList<TransportType>>();
             awaitingSession = new Vector<>();
             setDeathNote(); //messaging Version

android/sdl_android/src/main/java/com/smartdevicelink/transport/TransportManager.java

@@ -402,10 +402,10 @@ synchronized void enterLegacyMode(final String info) {
             return; //Already in legacy mode
         }
 
+        if (Looper.myLooper() == null) {
+            Looper.prepare();
+        }
         if (transportListener.onLegacyModeEnabled(info)) {
-            if (Looper.myLooper() == null) {
-                Looper.prepare();
-            }
             legacyBluetoothHandler = new LegacyBluetoothHandler(this);
             legacyBluetoothTransport = new MultiplexBluetoothTransport(legacyBluetoothHandler);
             if (contextWeakReference.get() != null) {
@@ -415,7 +415,7 @@ synchronized void enterLegacyMode(final String info) {
                 contextWeakReference.get().registerReceiver(legacyDisconnectReceiver, intentFilter);
             }
         } else {
-            new Handler().postDelayed(new Runnable() {
+            new Handler(Looper.myLooper()).postDelayed(new Runnable() {
                 @Override
                 public void run() {
                     transportListener.onError(info + " - Legacy mode unacceptable; shutting down.");