From eed288d79731393b753f7ee638dabb645d62fba0 Mon Sep 17 00:00:00 2001
From: Jordan Aasen <166539328+jaasen-livefront@users.noreply.github.com>
Date: Thu, 29 May 2025 13:01:07 -0700
Subject: [PATCH] [PM-21724]  - add safari and firefox to list of potential
 browser vendors (#14857)

* add safari and firefox to list of potential browser vendors

* use browserClientVendorExtended

* handle unknown browser client vendor
---
 .../popup/settings/autofill.component.ts        | 17 +++++++++++++++--
 1 file changed, 15 insertions(+), 2 deletions(-)

diff --git a/apps/browser/src/autofill/popup/settings/autofill.component.ts b/apps/browser/src/autofill/popup/settings/autofill.component.ts
index e1a5a2fc218..8c5c8e600a0 100644
--- a/apps/browser/src/autofill/popup/settings/autofill.component.ts
+++ b/apps/browser/src/autofill/popup/settings/autofill.component.ts
@@ -334,11 +334,24 @@ export class AutofillComponent implements OnInit {
     return null;
   }
 
+  get browserClientVendorExtended() {
+    if (this.browserClientVendor !== BrowserClientVendors.Unknown) {
+      return this.browserClientVendor;
+    }
+    if (this.platformUtilsService.isFirefox()) {
+      return "Firefox";
+    }
+    if (this.platformUtilsService.isSafari()) {
+      return "Safari";
+    }
+    return BrowserClientVendors.Unknown;
+  }
+
   get spotlightButtonText() {
-    if (this.browserClientVendor === BrowserClientVendors.Unknown) {
+    if (this.browserClientVendorExtended === BrowserClientVendors.Unknown) {
       return this.i18nService.t("turnOffAutofill");
     }
-    return this.i18nService.t("turnOffBrowserAutofill", this.browserClientVendor);
+    return this.i18nService.t("turnOffBrowserAutofill", this.browserClientVendorExtended);
   }
 
   async dismissSpotlight() {