From 33dae77a4d9caab3a0ea88ec89d10efb38097a84 Mon Sep 17 00:00:00 2001
From: Justin Baur <19896123+justindbaur@users.noreply.github.com>
Date: Mon, 22 Apr 2024 17:11:30 -0400
Subject: [PATCH] Revert "Stop CryptoService from using `getBgService` (#8843)"
 (#8867)

This reverts commit e29779875797cb8a508d3857aa25ecefca5aa5c1.
---
 .../src/popup/services/services.module.ts     | 43 ++-----------------
 1 file changed, 3 insertions(+), 40 deletions(-)

diff --git a/apps/browser/src/popup/services/services.module.ts b/apps/browser/src/popup/services/services.module.ts
index b344a18492..a7da6b7612 100644
--- a/apps/browser/src/popup/services/services.module.ts
+++ b/apps/browser/src/popup/services/services.module.ts
@@ -31,7 +31,6 @@ import { AuthService as AuthServiceAbstraction } from "@bitwarden/common/auth/ab
 import { DeviceTrustCryptoServiceAbstraction } from "@bitwarden/common/auth/abstractions/device-trust-crypto.service.abstraction";
 import { DevicesServiceAbstraction } from "@bitwarden/common/auth/abstractions/devices/devices.service.abstraction";
 import { KeyConnectorService } from "@bitwarden/common/auth/abstractions/key-connector.service";
-import { InternalMasterPasswordServiceAbstraction } from "@bitwarden/common/auth/abstractions/master-password.service.abstraction";
 import { SsoLoginServiceAbstraction } from "@bitwarden/common/auth/abstractions/sso-login.service.abstraction";
 import { TokenService } from "@bitwarden/common/auth/abstractions/token.service";
 import { TwoFactorService } from "@bitwarden/common/auth/abstractions/two-factor.service";
@@ -56,7 +55,6 @@ import { EncryptService } from "@bitwarden/common/platform/abstractions/encrypt.
 import { EnvironmentService } from "@bitwarden/common/platform/abstractions/environment.service";
 import { FileDownloadService } from "@bitwarden/common/platform/abstractions/file-download/file-download.service";
 import { I18nService as I18nServiceAbstraction } from "@bitwarden/common/platform/abstractions/i18n.service";
-import { KeyGenerationService } from "@bitwarden/common/platform/abstractions/key-generation.service";
 import { LogService } from "@bitwarden/common/platform/abstractions/log.service";
 import { PlatformUtilsService } from "@bitwarden/common/platform/abstractions/platform-utils.service";
 import { StateService as BaseStateServiceAbstraction } from "@bitwarden/common/platform/abstractions/state.service";
@@ -65,7 +63,6 @@ import {
   AbstractStorageService,
   ObservableStorageService,
 } from "@bitwarden/common/platform/abstractions/storage.service";
-import { BiometricStateService } from "@bitwarden/common/platform/biometrics/biometric-state.service";
 import { StateFactory } from "@bitwarden/common/platform/factories/state-factory";
 import { Message, MessageListener, MessageSender } from "@bitwarden/common/platform/messaging";
 // eslint-disable-next-line no-restricted-imports -- Used for dependency injection
@@ -105,7 +102,6 @@ import BrowserPopupUtils from "../../platform/popup/browser-popup-utils";
 import { BrowserFileDownloadService } from "../../platform/popup/services/browser-file-download.service";
 import { BrowserStateService as StateServiceAbstraction } from "../../platform/services/abstractions/browser-state.service";
 import { ScriptInjectorService } from "../../platform/services/abstractions/script-injector.service";
-import { BrowserCryptoService } from "../../platform/services/browser-crypto.service";
 import { BrowserEnvironmentService } from "../../platform/services/browser-environment.service";
 import BrowserLocalStorageService from "../../platform/services/browser-local-storage.service";
 import { BrowserScriptInjectorService } from "../../platform/services/browser-script-injector.service";
@@ -236,45 +232,12 @@ const safeProviders: SafeProvider[] = [
   }),
   safeProvider({
     provide: CryptoService,
-    useFactory: (
-      masterPasswordService: InternalMasterPasswordServiceAbstraction,
-      keyGenerationService: KeyGenerationService,
-      cryptoFunctionService: CryptoFunctionService,
-      encryptService: EncryptService,
-      platformUtilsService: PlatformUtilsService,
-      logService: LogService,
-      stateService: StateServiceAbstraction,
-      accountService: AccountServiceAbstraction,
-      stateProvider: StateProvider,
-      biometricStateService: BiometricStateService,
-    ) => {
-      const cryptoService = new BrowserCryptoService(
-        masterPasswordService,
-        keyGenerationService,
-        cryptoFunctionService,
-        encryptService,
-        platformUtilsService,
-        logService,
-        stateService,
-        accountService,
-        stateProvider,
-        biometricStateService,
-      );
+    useFactory: (encryptService: EncryptService) => {
+      const cryptoService = getBgService<CryptoService>("cryptoService")();
       new ContainerService(cryptoService, encryptService).attachToGlobal(self);
       return cryptoService;
     },
-    deps: [
-      InternalMasterPasswordServiceAbstraction,
-      KeyGenerationService,
-      CryptoFunctionService,
-      EncryptService,
-      PlatformUtilsService,
-      LogService,
-      StateServiceAbstraction,
-      AccountServiceAbstraction,
-      StateProvider,
-      BiometricStateService,
-    ],
+    deps: [EncryptService],
   }),
   safeProvider({
     provide: TotpServiceAbstraction,