refactor the Android TV-App prompt message (#33469)

* refactor the class MatterCommissioningPrompter.java,don't need Activity no more to avoid memory leak; * Restyled by google-java-format --------- Co-authored-by: Restyled.io <[email protected]>
project-chip · May 20, 2024 · 7a470f5 · 7a470f5
1 parent 443bb9c
commit 7a470f5
Show file tree

Hide file tree

Showing 16 changed files with 959 additions and 242 deletions.
diff --git a/examples/tv-app/android/App/platform-app/build.gradle b/examples/tv-app/android/App/platform-app/build.gradle
@@ -7,7 +7,7 @@ android {
 
     defaultConfig {
         applicationId "com.matter.tv.server"
-        minSdk 24
+        minSdk 26
         targetSdk 30
         versionCode 1
         versionName "1.0"
@@ -57,6 +57,9 @@ android {
 //            ]
         }
     }
+    buildFeatures {
+        viewBinding = true
+    }
 }
 
 dependencies {

diff --git a/examples/tv-app/android/App/platform-app/src/main/AndroidManifest.xml b/examples/tv-app/android/App/platform-app/src/main/AndroidManifest.xml
@@ -24,6 +24,7 @@
     <uses-permission android:name="android.permission.FOREGROUND_SERVICE"/>
     <uses-permission android:name="android.permission.QUERY_ALL_PACKAGES"
         tools:ignore="QueryAllPackagesPermission" />
+    <uses-permission android:name="android.permission.SYSTEM_ALERT_WINDOW" />
 
     <application
         android:allowBackup="true"
@@ -36,6 +37,7 @@
         android:theme="@style/Theme.MatterTVSlave">
 
         <activity android:name="com.matter.tv.server.MainActivity"
+            android:exported="true"
             android:launchMode="singleTask">
             <intent-filter>
                 <action android:name="android.intent.action.MAIN" />

diff --git a/...ples/tv-app/android/App/platform-app/src/main/java/com/matter/tv/server/MainActivity.java b/...ples/tv-app/android/App/platform-app/src/main/java/com/matter/tv/server/MainActivity.java
@@ -1,13 +1,16 @@
 package com.matter.tv.server;
 
+import android.content.Intent;
+import android.net.Uri;
 import android.os.Bundle;
+import android.provider.Settings;
+import androidx.appcompat.app.AlertDialog;
 import androidx.appcompat.app.AppCompatActivity;
 import androidx.fragment.app.Fragment;
 import com.google.android.material.bottomnavigation.BottomNavigationView;
 import com.matter.tv.server.fragments.ContentAppFragment;
 import com.matter.tv.server.fragments.QrCodeFragment;
 import com.matter.tv.server.fragments.TerminalFragment;
-import com.matter.tv.server.service.MatterServant;
 import java.util.LinkedHashMap;
 
 public class MainActivity extends AppCompatActivity {
@@ -41,16 +44,32 @@ protected void onCreate(Bundle savedInstanceState) {
     super.onCreate(savedInstanceState);
     setContentView(R.layout.activity_main);
 
-    // MainActivity is needed to launch dialog prompt
-    // in UserPrompter
-    MatterServant.get().setActivity(this);
-
     BottomNavigationView bottomNavigationView = findViewById(R.id.bottom_navigation);
     bottomNavigationView.setOnItemSelectedListener(navListener);
 
     getSupportFragmentManager()
         .beginTransaction()
         .replace(R.id.fragment_container_view, new QrCodeFragment())
         .commit();
+    checkOverlayPermission();
+  }
+
+  private void checkOverlayPermission() {
+    if (!Settings.canDrawOverlays(this)) {
+      AlertDialog.Builder builder = new AlertDialog.Builder(this);
+      builder
+          .setMessage("Allow permission to display over other apps")
+          .setTitle("Request overlay permission")
+          .setPositiveButton(
+              "Ok",
+              (dialog, which) -> {
+                dialog.dismiss();
+                Intent intent = new Intent(Settings.ACTION_MANAGE_OVERLAY_PERMISSION);
+                intent.setData(Uri.parse("package:" + getPackageName()));
+                startActivity(intent);
+              })
+          .create()
+          .show();
+    }
   }
 }