Simplify screenshot trigger example and move to its own activity (#4252)

Author: lfkellogg

firebase-appdistribution/test-app/src/main/AndroidManifest.xml

@@ -18,6 +18,7 @@
       </intent-filter>
     </activity>
     <activity android:name="com.googletest.firebase.appdistribution.testapp.SecondActivity" />
+    <activity android:name="com.googletest.firebase.appdistribution.testapp.ScreenshotDetectionActivity" />
     <activity android:name="com.googletest.firebase.appdistribution.testapp.TakeScreenshotAndTriggerFeedbackActivity" />
   </application>
 </manifest>

firebase-appdistribution/test-app/src/main/java/com/googletest/firebase/appdistribution/testapp/AppDistroTestApplication.kt

@@ -7,7 +7,6 @@ class AppDistroTestApplication : Application() {
         super.onCreate()
 
         // Perform any required trigger initialization here
-        ScreenshotDetectionFeedbackTrigger.initialize(this, R.string.termsAndConditions)
         CustomNotificationFeedbackTrigger.initialize(this);
 
         // Default feedback triggers can optionally be enabled application-wide here

firebase-appdistribution/test-app/src/main/java/com/googletest/firebase/appdistribution/testapp/CustomNotificationFeedbackTrigger.kt

@@ -209,7 +209,7 @@ class TakeScreenshotAndTriggerFeedbackActivity : Activity() {
   override fun onResume() {
     super.onResume()
     val screenshotUri = Uri.fromFile(getFileStreamPath(SCREENSHOT_FILE_NAME))
-    Firebase.appDistribution.startFeedback(R.string.termsAndConditions, screenshotUri)
+    Firebase.appDistribution.startFeedback(R.string.feedbackInfoText, screenshotUri)
     finish()
   }
 

firebase-appdistribution/test-app/src/main/java/com/googletest/firebase/appdistribution/testapp/MainActivity.kt

@@ -93,7 +93,7 @@ class MainActivity : AppCompatActivity() {
                     disableAllFeedbackTriggers()
                     Log.i(TAG, "Enabling notification trigger (SDK)")
                     firebaseAppDistribution.showFeedbackNotification(
-                        R.string.termsAndConditions, InterruptionLevel.HIGH)
+                        R.string.feedbackInfoText, InterruptionLevel.HIGH)
                 }
                 FeedbackTrigger.CUSTOM_NOTIFICATION.label -> {
                     disableAllFeedbackTriggers()
@@ -107,8 +107,10 @@ class MainActivity : AppCompatActivity() {
                 }
                 FeedbackTrigger.SCREENSHOT.label -> {
                     disableAllFeedbackTriggers()
-                    Log.i(TAG, "Enabling screenshot detection trigger")
-                    ScreenshotDetectionFeedbackTrigger.enable()
+                    startActivity(Intent(this, ScreenshotDetectionActivity::class.java))
+                    // Set the selection back to None since once we're back to this activity the
+                    // trigger will be disabled
+                    autoCompleteTextView.setText(FeedbackTrigger.NONE.label, false)
                 }
             }
         }
@@ -124,7 +126,6 @@ class MainActivity : AppCompatActivity() {
         firebaseAppDistribution.cancelFeedbackNotification()
         CustomNotificationFeedbackTrigger.disable()
         ShakeForFeedback.disable(application)
-        ScreenshotDetectionFeedbackTrigger.disable()
     }
 
     override fun onCreateOptionsMenu(menu: Menu): Boolean {
@@ -136,7 +137,7 @@ class MainActivity : AppCompatActivity() {
     override fun onOptionsItemSelected(item: MenuItem): Boolean {
         return when (item.itemId) {
             R.id.startFeedbackMenuItem -> {
-                Firebase.appDistribution.startFeedback(R.string.termsAndConditions)
+                Firebase.appDistribution.startFeedback(R.string.feedbackInfoText)
                 true
             }
             else -> super.onOptionsItemSelected(item)

firebase-appdistribution/test-app/src/main/java/com/googletest/firebase/appdistribution/testapp/ScreenshotDetectionActivity.kt

@@ -0,0 +1,67 @@
+package com.googletest.firebase.appdistribution.testapp
+
+import android.app.AlertDialog
+import android.os.Bundle
+import android.os.Handler
+import android.os.HandlerThread
+import android.util.Log
+import androidx.activity.result.contract.ActivityResultContracts
+import androidx.appcompat.app.AppCompatActivity
+import com.googletest.firebase.appdistribution.testapp.ScreenshotDetectionFeedbackTrigger.Companion.screenshotReadPermission
+import java.util.*
+
+class ScreenshotDetectionActivity : AppCompatActivity() {
+  private val handlerThread = HandlerThread("AppDistroFeedbackTrigger").also { it.start() }
+  private var screenshotDetectionFeedbackTrigger =
+    ScreenshotDetectionFeedbackTrigger(
+      this,
+      R.string.feedbackInfoText,
+      Handler(handlerThread.looper)
+    )
+
+  override fun onCreate(savedInstanceState: Bundle?) {
+    super.onCreate(savedInstanceState)
+    setContentView(R.layout.activity_screenshot_detection)
+
+    val launcher =
+      registerForActivityResult(ActivityResultContracts.RequestPermission()) { isGranted: Boolean ->
+        if (!isGranted) {
+          Log.i(TAG, "Permission not granted")
+          AlertDialog.Builder(this)
+            .setMessage(
+              "Because storage permission has been denied, screenshots taken will not trigger feedback to the developer."
+            )
+            .setPositiveButton("OK") { _, _ -> }
+            .show()
+        }
+      }
+
+    if (shouldShowRequestPermissionRationale(screenshotReadPermission)) {
+      Log.i(TAG, "Showing customer rationale for requesting permission.")
+      AlertDialog.Builder(this)
+        .setMessage(
+          "Taking a screenshot of the app can initiate feedback to the developer. To enable this feature, allow the app access to device storage."
+        )
+        .setPositiveButton("OK") { _, _ ->
+          Log.i(TAG, "Launching request for permission.")
+          launcher.launch(screenshotReadPermission)
+        }
+        .setNegativeButton("No thanks") { _, _ -> Log.i(TAG, "User denied permission request.") }
+        .show()
+    } else {
+      Log.i(TAG, "Launching request for permission without rationale.")
+      launcher.launch(screenshotReadPermission)
+    }
+
+    screenshotDetectionFeedbackTrigger.listenForScreenshots()
+  }
+
+  override fun onDestroy() {
+    screenshotDetectionFeedbackTrigger.stopListeningForScreenshots()
+    super.onDestroy()
+  }
+
+  companion object {
+    private const val TAG = "ScreenshotDetectionActivity"
+  }
+}