Refactor exitForeground logic

Author: joeygrover

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

@@ -1517,26 +1517,17 @@ private void safeStartForeground(int id, Notification notification){
 	private void exitForeground(){
 		synchronized (NOTIFICATION_LOCK) {
 			if (isForeground && !isPrimaryTransportConnected()) {	//Ensure that the service is in the foreground and no longer connected to a transport
-				this.stopForeground(true);
+				DebugTool.logInfo("SdlRouterService to exit foreground");
+				this.stopForeground(false); //false is used because some phones have issues when trying to clear the notification at the same time
 				NotificationManager notificationManager = (NotificationManager)getSystemService(Context.NOTIFICATION_SERVICE);
-				if (notificationManager!= null && Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
+				if (notificationManager!= null){
 					try {
-						boolean notificationHasDisplayed = false;
-						StatusBarNotification[] notifications = notificationManager.getActiveNotifications();
-						for (StatusBarNotification notification : notifications) {
-							if(notification != null && FOREGROUND_SERVICE_ID == notification.getId()){
-								DebugTool.logInfo("Service notification is being displayed");
-								notificationHasDisplayed = true;
-								break;
-							}
-						}
-						if (notificationHasDisplayed) {
+						notificationManager.cancel(FOREGROUND_SERVICE_ID);
+						if( Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
 							notificationManager.deleteNotificationChannel(SDL_NOTIFICATION_CHANNEL_ID);
 						}
-						//else leave the notification channel alone to avoid deleting it before the
-						//foreground service notification has a chance to be displayed.
-					} catch (Exception e){
-						DebugTool.logError("Issue when deleting notification channel", e);
+					} catch (Exception e) {
+						DebugTool.logError("Issue when removing notification and channel", e);
 					}
 				}
 				isForeground = false;