fix multiple unresponsive and unclosable overlays can be opened

X-SLAYER · Feb 6, 2024 · 7fff772 · 7fff772
1 parent 5897ec4
commit 7fff772
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 11 deletions.
diff --git a/...c/main/java/flutter/overlay/window/flutter_overlay_window/FlutterOverlayWindowPlugin.java b/...c/main/java/flutter/overlay/window/flutter_overlay_window/FlutterOverlayWindowPlugin.java
@@ -6,6 +6,7 @@
 import android.net.Uri;
 import android.os.Build;
 import android.provider.Settings;
+
 import androidx.annotation.NonNull;
 import androidx.annotation.Nullable;
 
@@ -110,7 +111,6 @@ public void onMethodCall(@NonNull MethodCall call, @NonNull Result result) {
         } else {
             result.notImplemented();
         }
-
     }
 
     @Override

diff --git a/android/src/main/java/flutter/overlay/window/flutter_overlay_window/OverlayService.java b/android/src/main/java/flutter/overlay/window/flutter_overlay_window/OverlayService.java
@@ -76,6 +76,14 @@ public IBinder onBind(Intent intent) {
     public void onDestroy() {
         Log.d("OverLay", "Destroying the overlay window service");
         isRunning = false;
+        if(windowManager != null && flutterView != null) {
+            windowManager.removeView(flutterView);
+            windowManager = null;
+        }
+        if(flutterView != null) {
+            flutterView.detachFromFlutterEngine();
+            flutterView = null;
+        }
         NotificationManager notificationManager = (NotificationManager) getApplicationContext().getSystemService(Context.NOTIFICATION_SERVICE);
         notificationManager.cancel(OverlayConstants.NOTIFICATION_ID);
         if(CachedMessageChannels.overlayMessageChannel != null){
@@ -90,20 +98,16 @@ public int onStartCommand(Intent intent, int flags, int startId) {
         mResources = getApplicationContext().getResources();
         boolean isCloseWindow = intent.getBooleanExtra(INTENT_EXTRA_IS_CLOSE_WINDOW, false);
         if (isCloseWindow) {
-            if (windowManager != null) {
-                windowManager.removeView(flutterView);
-                windowManager = null;
-                flutterView.detachFromFlutterEngine();
-                stopSelf();
-            }
-            isRunning = false;
+            stopSelf();
             return START_STICKY;
         }
-        if (windowManager != null) {
+        if(windowManager != null && flutterView != null) {
             windowManager.removeView(flutterView);
             windowManager = null;
+        }
+        if(flutterView != null) {
             flutterView.detachFromFlutterEngine();
-            stopSelf();
+            flutterView = null;
         }
         isRunning = true;
         Log.d("onStartCommand", "Service started");
@@ -128,7 +132,7 @@ public int onStartCommand(Intent intent, int flags, int startId) {
 
         BasicMessageChannel<Object> overlayMessageChannel = new BasicMessageChannel<>(FlutterEngineCache.getInstance().get(OverlayConstants.CACHED_TAG).getDartExecutor().getBinaryMessenger(), OverlayConstants.MESSENGER_TAG, JSONMessageCodec.INSTANCE);
         overlayMessageChannel.setMessageHandler((message, reply) -> {
-            if(CachedMessageChannels.mainAppMessageChannel == null) {
+            if (CachedMessageChannels.mainAppMessageChannel == null) {
                 reply.reply(false);
                 return;
             }