Bug 1660818 - Add telemetry to track the number of times the scrollab…

…le badge is clicked. r=gl Differential Revision: https://phabricator.services.mozilla.com/D88049
iaxat · Aug 31, 2020 · 99887bf · 99887bf
1 parent 6b98014
commit 99887bf
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 0 deletions.
diff --git a/devtools/client/inspector/markup/test/browser_markup_scrollable_badge_click.js b/devtools/client/inspector/markup/test/browser_markup_scrollable_badge_click.js
@@ -59,6 +59,8 @@ add_task(async function() {
     "Scrollable badge is active"
   );
 
+  checkTelemetry("devtools.markup.scrollable.badge.clicked", "", 1, "scalar");
+
   info(
     "Changing CSS so elements update their overflow highlights accordingly."
   );
@@ -90,6 +92,8 @@ add_task(async function() {
     !container.editor._scrollableBadge.classList.contains("active"),
     "Scrollable badge is not active"
   );
+
+  checkTelemetry("devtools.markup.scrollable.badge.clicked", "", 2, "scalar");
 });
 
 async function getContainerForSelector(selector, inspector) {

diff --git a/devtools/client/inspector/markup/views/element-editor.js b/devtools/client/inspector/markup/views/element-editor.js
@@ -1041,6 +1041,11 @@ ElementEditor.prototype = {
         );
       }
     }
+
+    this.markup.telemetry.scalarAdd(
+      "devtools.markup.scrollable.badge.clicked",
+      1
+    );
   },
 
   /**

diff --git a/toolkit/components/telemetry/Scalars.yaml b/toolkit/components/telemetry/Scalars.yaml
@@ -2906,6 +2906,23 @@ devtools.changesview:
     record_in_processes:
       - 'main'
 
+devtools.markup.scrollable.badge:
+  clicked:
+    bug_numbers:
+      - 1660818
+    description: >
+      Number of times the scrollable inspector badge has been clicked.
+    expires: never
+    kind: uint
+    notification_emails:
+      - [email protected]
+    release_channel_collection: opt-out
+    products:
+      - 'firefox'
+      - 'fennec'
+    record_in_processes:
+      - 'main'
+
 dom.event:
   confluence_load_count:
     bug_numbers:
-Original file line number
+Diff line change
@@ Expand Up / @@ -1041,6 +1041,11 @@ ElementEditor.prototype = { @@
             );
           }
         }
+        this.markup.telemetry.scalarAdd(
+          "devtools.markup.scrollable.badge.clicked",
+        );
       },
       /**
@@ Expand Down @@