Bug 1721178 - Add targeting trigger for successful captive portal login r=dmose

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

browser/components/newtab/lib/ASRouterTriggerListeners.jsm

@@ -585,6 +585,44 @@ this.ASRouterTriggerListeners = new Map([
       },
     },
   ],
+
+  [
+    "captivePortalLogin",
+    {
+      id: "captivePortalLogin",
+      _initialized: false,
+      _triggerHandler: null,
+
+      init(triggerHandler) {
+        if (!this._initialized) {
+          Services.obs.addObserver(this, "captive-portal-login-success");
+          this._initialized = true;
+        }
+        this._triggerHandler = triggerHandler;
+      },
+
+      observe(aSubject, aTopic, aData) {
+        switch (aTopic) {
+          case "captive-portal-login-success":
+            const browser = Services.wm.getMostRecentBrowserWindow();
+            if (browser) {
+              this._triggerHandler(browser.gBrowser.selectedBrowser, {
+                id: this.id,
+              });
+            }
+            break;
+        }
+      },
+
+      uninit() {
+        if (this._initialized) {
+          this._triggerHandler = null;
+          this._initialized = false;
+          Services.obs.removeObserver(this, "captive-portal-login-success");
+        }
+      },
+    },
+  ],
 ]);
 
 const EXPORTED_SYMBOLS = ["ASRouterTriggerListeners"];

browser/components/newtab/test/browser/browser_trigger_listeners.js

@@ -63,3 +63,22 @@ add_task(async function test_openURL_visit_counter_withPattern() {
     "Third call should have count 2 for http://example.com"
   );
 });
+
+add_task(async function test_captivePortalLogin() {
+  const stub = sinon.stub();
+  const captivePortalTrigger = ASRouterTriggerListeners.get(
+    "captivePortalLogin"
+  );
+
+  captivePortalTrigger.init(stub);
+
+  Services.obs.notifyObservers(this, "captive-portal-login-success", {});
+
+  Assert.ok(stub.called, "Called after login event");
+
+  captivePortalTrigger.uninit();
+
+  Services.obs.notifyObservers(this, "captive-portal-login-success", {});
+
+  Assert.equal(stub.callCount, 1, "Not called after uninit");
+});

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

@@ -185,6 +185,18 @@
           "additionalProperties": false,
           "required": ["id"],
           "description": "Happens when starting the browser or navigating to about:home/newtab"
+        },
+        {
+          "type": "object",
+          "properties": {
+            "id": {
+              "type": "string",
+              "enum": ["captivePortalLogin"]
+            }
+          },
+          "additionalProperties": false,
+          "required": ["id"],
+          "description": "Happens when the user successfully goes through a captive portal authentication flow."
         }
       ]
     }

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

@@ -119,3 +119,8 @@ On the newtab/homepage it reports the `source` as `newtab`.
 let source = "newtab" | undefined;
 let willShowDefaultPrompt = boolean;
 ```
+
+### `captivePortalLogin`
+
+Happens when the user successfully goes through a captive portal authentication flow.
+