for #117: add edit-containers event

index.js

@@ -109,6 +109,7 @@ const ContainerService = {
       "removeIdentity",
       "updateIdentity",
       "getPreference",
+      "sendTelemetryPayload"
     ];
 
     // Map of identities.
@@ -180,15 +181,6 @@ const ContainerService = {
       version: self.version
     }).sendEvent;
 
-    this._sendTelemetryPayload = function(params = {}) {
-      let payload = { // eslint-disable-line prefer-const
-        "uuid": this._metricsUUID
-      };
-      Object.assign(payload, params);
-
-      this._sendEvent(payload);
-    };
-
   },
 
   // utility methods
@@ -357,6 +349,17 @@ const ContainerService = {
     ss.storage.identitiesData = this._identitiesState;
   },
 
+  sendTelemetryPayload(params = {}) {
+    // when pings come from popup, delete "method" prop
+    delete params.method;
+    let payload = { // eslint-disable-line prefer-const
+      "uuid": this._metricsUUID
+    };
+    Object.assign(payload, params);
+
+    this._sendEvent(payload);
+  },
+
   // Tabs management
 
   hideTabs(args) {
@@ -417,7 +420,7 @@ const ContainerService = {
   },
 
   sortTabs() {
-    this._sendTelemetryPayload({
+    this.sendTelemetryPayload({
       "event": "sort-tabs",
       "shownContainersCount": this._shownContainersCount(),
       "totalContainerTabsCount": this._totalContainerTabsCount(),
@@ -577,7 +580,7 @@ const ContainerService = {
         userContextId = args.userContextId;
       }
 
-      this._sendTelemetryPayload({
+      this.sendTelemetryPayload({
         "event": "open-tab",
         "eventSource": args.source,
         "userContextId": userContextId,

webextension/js/popup.js

@@ -203,6 +203,10 @@ Logic.registerPanel(P_CONTAINERS_LIST, {
     });
 
     document.querySelector("#edit-containers-link").addEventListener("click", () => {
+      browser.runtime.sendMessage({
+        method: "sendTelemetryPayload",
+        event: "edit-containers"
+      });
       Logic.showPanel(P_CONTAINERS_EDIT);
     });