Bug 1550631 - Include the entryPoint for navigations from about:protections to about:logins. r=jaws

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

--HG--
extra : moz-landing-system : lando

browser/components/about/AboutProtectionsHandler.jsm

@@ -324,7 +324,9 @@ var AboutProtectionsHandler = {
     let win = aMessage.target.browser.ownerGlobal;
     switch (aMessage.name) {
       case "OpenAboutLogins":
-        win.openTrustedLinkIn("about:logins", "tab");
+        LoginHelper.openPasswordManager(win, {
+          entryPoint: "aboutprotections",
+        });
         break;
       case "OpenContentBlockingPreferences":
         win.openPreferences("privacy-trackingprotection", {

browser/components/protections/content/monitor-card.js

@@ -43,8 +43,10 @@ export default class MonitorClass {
     });
 
     let openLockwise = this.doc.getElementById("lockwise-link");
-    openLockwise.addEventListener("click", () => {
+    openLockwise.addEventListener("click", evt => {
       this.doc.sendTelemetryEvent("click", "lw_open_breach_link");
+      RPMSendAsyncMessage("OpenAboutLogins");
+      evt.preventDefault();
     });
   }
 

browser/components/protections/content/protections.html

@@ -171,7 +171,7 @@
                     <!-- Display number of exposed stored passwords here. -->
                   </span>
                   <span id="password-warning">
-                    <a target="_blank" href="about:logins" id="lockwise-link" data-l10n-name="lockwise-link"></a>
+                    <a target="_blank" href="about:logins?entryPoint=aboutprotections" id="lockwise-link" data-l10n-name="lockwise-link"></a>
                   </span>
                 </div>
               </div>

browser/components/protections/test/browser/browser_protections_telemetry.js

@@ -136,8 +136,13 @@ function waitForTelemetryEventCount(count) {
       Ci.nsITelemetry.DATASET_PRERELEASE_CHANNELS,
       false
     ).content;
+    if (!events) {
+      return null;
+    }
+    // Ignore irrelevant events from other parts of the browser.
+    events = events.filter(e => e[1] == "security.ui.protections");
     info("got " + (events && events.length) + " events");
-    if (events && events.length == count) {
+    if (events.length == count) {
       return events;
     }
     return null;

toolkit/components/telemetry/Events.yaml

@@ -561,7 +561,7 @@ normandy:
 
 pwmgr:
   open_management:
-    objects: ["autocomplete", "capturedoorhanger", "contextmenu", "direct", "fxamenu", "mainmenu", "pageinfo", "preferences"]
+    objects: ["aboutprotections", "autocomplete", "capturedoorhanger", "contextmenu", "direct", "fxamenu", "mainmenu", "pageinfo", "preferences"]
     methods: ["open_management"]
     description: >
       Sent when opening the password management UI.