feat: Remove Request Queueing Setting

app/scripts/background.js

@@ -1057,14 +1057,10 @@ export function setupController(
 
   function getPendingApprovalCount() {
     try {
-      let pendingApprovalCount =
+      const pendingApprovalCount =
         controller.appStateController.waitingForUnlock.length +
-        controller.approvalController.getTotalApprovalCount();
-
-      if (controller.preferencesController.getUseRequestQueue()) {
-        pendingApprovalCount +=
-          controller.queuedRequestController.state.queuedRequestCount;
-      }
+        controller.approvalController.getTotalApprovalCount() +
+        controller.queuedRequestController.state.queuedRequestCount;
       return pendingApprovalCount;
     } catch (error) {
       console.error('Failed to get pending approval count:', error);

app/scripts/constants/sentry-state.ts

@@ -248,7 +248,6 @@ export const SENTRY_BACKGROUND_STATE = {
     useNonceField: true,
     usePhishDetect: true,
     useTokenDetection: true,
-    useRequestQueue: true,
     useTransactionSimulations: true,
     enableMV3TimestampSave: true,
   },

app/scripts/controllers/preferences-controller.test.ts

@@ -640,19 +640,6 @@ describe('preferences controller', () => {
     });
   });
 
-  describe('useRequestQueue', () => {
-    it('defaults useRequestQueue to true', () => {
-      const { controller } = setupController({});
-      expect(controller.state.useRequestQueue).toStrictEqual(true);
-    });
-
-    it('setUseRequestQueue to false', () => {
-      const { controller } = setupController({});
-      controller.setUseRequestQueue(false);
-      expect(controller.state.useRequestQueue).toStrictEqual(false);
-    });
-  });
-
   describe('addSnapAccountEnabled', () => {
     it('defaults addSnapAccountEnabled to false', () => {
       const { controller } = setupController({});

app/scripts/controllers/preferences-controller.ts

@@ -179,8 +179,8 @@ export const getDefaultPreferencesControllerState =
     useNftDetection: true,
     use4ByteResolution: true,
     useCurrencyRateCheck: true,
-    useRequestQueue: true,
     openSeaEnabled: true,
+    useRequestQueue: true,
     securityAlertsEnabled: true,
     watchEthereumAccountEnabled: false,
     bitcoinSupportEnabled: false,
@@ -607,17 +607,6 @@ export class PreferencesController extends BaseController<
     });
   }
 
-  /**
-   * Setter for the `useRequestQueue` property
-   *
-   * @param val - Whether or not the user wants to have requests queued if network change is required.
-   */
-  setUseRequestQueue(val: boolean): void {
-    this.update((state) => {
-      state.useRequestQueue = val;
-    });
-  }
-
   /**
    * Setter for the `openSeaEnabled` property
    *
@@ -816,15 +805,6 @@ export class PreferencesController extends BaseController<
     return selectedAccount.address;
   }
 
-  /**
-   * Getter for the `useRequestQueue` property
-   *
-   * @returns whether this option is on or off.
-   */
-  getUseRequestQueue(): boolean {
-    return this.state.useRequestQueue;
-  }
-
   /**
    * Sets a custom label for an account
    *

app/scripts/fixtures/with-preferences.js

@@ -30,7 +30,6 @@ export const FIXTURES_PREFERENCES = {
   useTokenDetection: true,
   useCurrencyRateCheck: true,
   useMultiAccountBalanceChecker: true,
-  useRequestQueue: true,
   theme: 'light',
   useExternalNameSources: true,
   useTransactionSimulations: true,

app/scripts/lib/backup.test.js

@@ -174,7 +174,6 @@ const jsonData = JSON.stringify({
     theme: 'light',
     customNetworkListEnabled: false,
     textDirection: 'auto',
-    useRequestQueue: true,
   },
   internalAccounts: {
     accounts: {