fix: remove removeAccountsBySnapId

Accounts shouldn't be removed by directly calling the keyring because
they also need to be removed from other controllers, which is done by
the `removeAccount` method from `metamask-controller.js`.

src/SnapKeyring.ts

@@ -624,20 +624,6 @@ export class SnapKeyring extends EventEmitter {
       : undefined;
   }
 
-  /**
-   * Remove all accounts associated with a given Snap ID.
-   *
-   * @param snapId - The Snap ID to remove accounts for.
-   * @returns A Promise that resolves when all accounts have been removed.
-   */
-  async removeAccountsBySnapId(snapId: string): Promise<void> {
-    for (const entry of this.#accounts.values()) {
-      if (entry.snapId === snapId) {
-        await this.removeAccount(entry.account.address.toLowerCase());
-      }
-    }
-  }
-
   /**
    * List all snap keyring accounts.
    *