[PM-25884] Move Phishing Detection Safari check to PhishingDetectionSettingsService (#18042)

* Move safari check to phishing detection settings to expose to all places using phishing detection

* Remove duplicate comment

apps/browser/src/background/main.background.ts

@@ -1490,6 +1490,7 @@ export default class MainBackground {
       this.billingAccountProfileStateService,
       this.configService,
       this.organizationService,
+      this.platformUtilsService,
       this.stateProvider,
     );
 

apps/browser/src/dirt/phishing-detection/services/phishing-detection.service.ts

@@ -5,7 +5,6 @@ import {
   filter,
   map,
   merge,
-  of,
   Subject,
   switchMap,
   tap,
@@ -112,17 +111,7 @@ export class PhishingDetectionService {
       .messages$(PHISHING_DETECTION_CANCEL_COMMAND)
       .pipe(switchMap((message) => BrowserApi.closeTab(message.tabId)));
 
-    // Phishing detection is unavailable on Safari due to platform limitations
-    if (BrowserApi.isSafariApi) {
-      logService.debug(
-        "[PhishingDetectionService] Disabling phishing detection service for Safari.",
-      );
-    }
-
-    // Watching for settings changes to enable/disable phishing detection
-    const phishingDetectionActive$ = BrowserApi.isSafariApi
-      ? of(false)
-      : phishingDetectionSettingsService.on$;
+    const phishingDetectionActive$ = phishingDetectionSettingsService.on$;
 
     const initSub = phishingDetectionActive$
       .pipe(

apps/browser/src/popup/services/services.module.ts

@@ -523,6 +523,7 @@ const safeProviders: SafeProvider[] = [
       BillingAccountProfileStateService,
       ConfigService,
       OrganizationService,
+      PlatformUtilsService,
       StateProvider,
     ],
   }),