From 29aaa2cab8b873f4e923bb3dd0b352cf1a7d6799 Mon Sep 17 00:00:00 2001
From: rosariopf <rosariopf@google.com>
Date: Thu, 18 Aug 2022 17:34:48 +0100
Subject: [PATCH 1/2] refactor(auth): log an exception if the device has no
 browser installed

---
 auth/src/main/AndroidManifest.xml                 | 10 ++++++++++
 .../ui/phone/PhoneNumberVerificationHandler.java  | 15 +++++++++++++--
 2 files changed, 23 insertions(+), 2 deletions(-)

diff --git a/auth/src/main/AndroidManifest.xml b/auth/src/main/AndroidManifest.xml
index acc3e68ad..1342170ae 100644
--- a/auth/src/main/AndroidManifest.xml
+++ b/auth/src/main/AndroidManifest.xml
@@ -7,6 +7,16 @@
     <uses-permission android:name="android.permission.ACCESS_NETWORK_STATE" />
     <uses-permission android:name="android.permission.INTERNET" />
 
+    <!-- Used to check if a browser is available before launching phone auth -->
+    <!-- See data/ui/phone/PhoneNumberVerificationHandler.isBrowserAvailable() -->
+    <queries>
+        <intent>
+            <action android:name="android.intent.action.VIEW" />
+            <category android:name="android.intent.category.BROWSABLE" />
+            <data android:scheme="http" />
+        </intent>
+    </queries>
+
     <application>
 
         <meta-data
diff --git a/auth/src/main/java/com/firebase/ui/auth/ui/phone/PhoneNumberVerificationHandler.java b/auth/src/main/java/com/firebase/ui/auth/ui/phone/PhoneNumberVerificationHandler.java
index 16d032986..251581560 100644
--- a/auth/src/main/java/com/firebase/ui/auth/ui/phone/PhoneNumberVerificationHandler.java
+++ b/auth/src/main/java/com/firebase/ui/auth/ui/phone/PhoneNumberVerificationHandler.java
@@ -2,8 +2,10 @@
 
 import android.app.Activity;
 import android.app.Application;
+import android.content.ActivityNotFoundException;
+import android.content.Intent;
+import android.net.Uri;
 import android.os.Bundle;
-
 import com.firebase.ui.auth.data.model.PhoneNumberVerificationRequiredException;
 import com.firebase.ui.auth.data.model.Resource;
 import com.firebase.ui.auth.viewmodel.AuthViewModelBase;
@@ -58,7 +60,11 @@ public void onCodeSent(@NonNull String verificationId,
         if (force) {
             optionsBuilder.setForceResendingToken(mForceResendingToken);
         }
-        PhoneAuthProvider.verifyPhoneNumber(optionsBuilder.build());
+        if (isBrowserAvailable(activity)) {
+            PhoneAuthProvider.verifyPhoneNumber(optionsBuilder.build());
+        } else {
+            setResult(Resource.forFailure(new ActivityNotFoundException("No browser was found in this device")));
+        }
     }
 
     public void submitVerificationCode(String number, String code) {
@@ -77,4 +83,9 @@ public void onRestoreInstanceState(@Nullable Bundle savedInstanceState) {
             mVerificationId = savedInstanceState.getString(VERIFICATION_ID_KEY);
         }
     }
+
+    private boolean isBrowserAvailable(Activity activity) {
+        Intent browserIntent = new Intent(Intent.ACTION_VIEW, Uri.parse("http://"));
+        return browserIntent.resolveActivity(activity.getPackageManager()) != null;
+    }
 }

From 1153ff9ececd10696af732904fa0e608a805335d Mon Sep 17 00:00:00 2001
From: rosariopf <rosariopf@google.com>
Date: Thu, 18 Aug 2022 17:40:21 +0100
Subject: [PATCH 2/2] fix path in Manifest comment

---
 auth/src/main/AndroidManifest.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/auth/src/main/AndroidManifest.xml b/auth/src/main/AndroidManifest.xml
index 1342170ae..5aee51352 100644
--- a/auth/src/main/AndroidManifest.xml
+++ b/auth/src/main/AndroidManifest.xml
@@ -8,7 +8,7 @@
     <uses-permission android:name="android.permission.INTERNET" />
 
     <!-- Used to check if a browser is available before launching phone auth -->
-    <!-- See data/ui/phone/PhoneNumberVerificationHandler.isBrowserAvailable() -->
+    <!-- See ui/phone/PhoneNumberVerificationHandler.isBrowserAvailable() -->
     <queries>
         <intent>
             <action android:name="android.intent.action.VIEW" />