From fe81470f368ce24a147cfa5baab5b4bc1b8a292f Mon Sep 17 00:00:00 2001
From: Daniel James Smith <2670567+djsmith85@users.noreply.github.com>
Date: Mon, 20 May 2024 15:49:47 +0200
Subject: [PATCH] [PM-6195] Refactor injection of services in browser services
 module (Pt2) (#8849)

* Remove UsernameGenerationService from popup/services.module

* Remove PasswordGenerationService from popup/services.module

---------

Co-authored-by: Daniel James Smith <djsmith85@users.noreply.github.com>
---
 apps/browser/src/popup/services/services.module.ts | 12 ------------
 1 file changed, 12 deletions(-)

diff --git a/apps/browser/src/popup/services/services.module.ts b/apps/browser/src/popup/services/services.module.ts
index 32d4adae4a1..a4f5c8a4c67 100644
--- a/apps/browser/src/popup/services/services.module.ts
+++ b/apps/browser/src/popup/services/services.module.ts
@@ -81,8 +81,6 @@ import {
 } from "@bitwarden/common/platform/state";
 // eslint-disable-next-line import/no-restricted-paths -- Used for dependency injection
 import { InlineDerivedStateProvider } from "@bitwarden/common/platform/state/implementations/inline-derived-state";
-import { PasswordGenerationServiceAbstraction } from "@bitwarden/common/tools/generator/password";
-import { UsernameGenerationServiceAbstraction } from "@bitwarden/common/tools/generator/username";
 import { VaultTimeoutStringType } from "@bitwarden/common/types/vault-timeout.type";
 import { CipherService } from "@bitwarden/common/vault/abstractions/cipher.service";
 import { CollectionService } from "@bitwarden/common/vault/abstractions/collection.service";
@@ -317,11 +315,6 @@ const safeProviders: SafeProvider[] = [
     },
     deps: [ToastService, OffscreenDocumentService],
   }),
-  safeProvider({
-    provide: PasswordGenerationServiceAbstraction,
-    useFactory: getBgService<PasswordGenerationServiceAbstraction>("passwordGenerationService"),
-    deps: [],
-  }),
   safeProvider({
     provide: SyncService,
     useFactory: getBgService<SyncService>("syncService"),
@@ -482,11 +475,6 @@ const safeProviders: SafeProvider[] = [
       MigrationRunner,
     ],
   }),
-  safeProvider({
-    provide: UsernameGenerationServiceAbstraction,
-    useFactory: getBgService<UsernameGenerationServiceAbstraction>("usernameGenerationService"),
-    deps: [],
-  }),
   safeProvider({
     provide: BaseStateServiceAbstraction,
     useExisting: StateServiceAbstraction,