Bug 1827971 - Remove OPEN_FIREFOX_VIEW_AND_COLORWAYS_MODAL...

Bug 1827971 - Remove OPEN_FIREFOX_VIEW_AND_COLORWAYS_MODAL SpecialMessageAction r=omc-reviewers,aminomancer

Turns out both `OPEN_FIREFOX_VIEW` and `OPEN_FIREFOX_VIEW_AND_COLORWAYS_MODAL` just call `window.FirefoxViewHandler.openTab();`, so not only is this action kinda misleading now that the Colorways closet is gone, it's a duplicate action.

Differential Revision: https://phabricator.services.mozilla.com/D175792

toolkit/components/messaging-system/lib/SpecialMessageActions.sys.mjs

@@ -404,9 +404,6 @@ export const SpecialMessageActions = {
         );
         clickElement?.click();
         break;
-      case "OPEN_FIREFOX_VIEW_AND_COLORWAYS_MODAL":
-        window.FirefoxViewHandler.openTab();
-        break;
       case "RELOAD_BROWSER":
         browser.reload();
         break;

toolkit/components/messaging-system/schemas/SpecialMessageActionSchemas/SpecialMessageActionSchemas.json

@@ -562,18 +562,6 @@
           "additionalProperties": false,
           "description": "Selects an element in the current Window's document and triggers a click action"
         },
-        {
-          "type": "object",
-          "properties": {
-            "type": {
-              "type": "string",
-              "enum": ["OPEN_FIREFOX_VIEW_AND_COLORWAYS_MODAL"]
-            }
-          },
-          "required": ["type"],
-          "additionalProperties": false,
-          "description": "Message action that opens about:firefoxview and renders the colorways modal"
-        },
         {
           "type": "object",
           "properties": {

toolkit/components/messaging-system/schemas/SpecialMessageActionSchemas/index.md

@@ -335,13 +335,6 @@ Action for running multiple actions. Actions should be included in an array of a
 Selects an element in the current Window's document and triggers a click action
 
 
-### `OPEN_FIREFOX_VIEW_AND_COLORWAYS_MODAL`
-
-* args: (none)
-
-Action for opening about:firefoxview and the colorways modal
-
-
 ### `RELOAD_BROWSER`
 
 * args: (none)

toolkit/components/messaging-system/schemas/SpecialMessageActionSchemas/test/browser/browser_sma_open_firefoxview.js

@@ -9,7 +9,7 @@ add_task(async function test_open_firefoxview() {
     "about:firefoxview"
   );
   await SMATestUtils.executeAndValidateAction({
-    type: "OPEN_FIREFOX_VIEW_AND_COLORWAYS_MODAL",
+    type: "OPEN_FIREFOX_VIEW",
   });
 
   const tab = await tabPromise;