chore: cp-12.16.2 Make "Dapp Viewed" event exclude metametrics id data

app/scripts/background.js

@@ -731,18 +731,23 @@ function emitDappViewedMetricEvent(origin) {
   );
   const numberOfTotalAccounts = Object.keys(preferencesState.identities).length;
 
-  controller.metaMetricsController.trackEvent({
-    event: MetaMetricsEventName.DappViewed,
-    category: MetaMetricsEventCategory.InpageProvider,
-    referrer: {
-      url: origin,
+  controller.metaMetricsController.trackEvent(
+    {
+      event: MetaMetricsEventName.DappViewed,
+      category: MetaMetricsEventCategory.InpageProvider,
+      referrer: {
+        url: origin,
+      },
+      properties: {
+        is_first_visit: false,
+        number_of_accounts: numberOfTotalAccounts,
+        number_of_accounts_connected: numberOfConnectedAccounts,
+      },
     },
-    properties: {
-      is_first_visit: false,
-      number_of_accounts: numberOfTotalAccounts,
-      number_of_accounts_connected: numberOfConnectedAccounts,
+    {
+      excludeMetaMetricsId: true,
     },
-  });
+  );
 }
 
 /**

app/scripts/lib/rpc-method-middleware/handlers/request-accounts.js

@@ -102,18 +102,23 @@ async function requestEthereumAccountsHandler(
     const isFirstVisit = !Object.keys(metamaskState.permissionHistory).includes(
       origin,
     );
-    sendMetrics({
-      event: MetaMetricsEventName.DappViewed,
-      category: MetaMetricsEventCategory.InpageProvider,
-      referrer: {
-        url: origin,
+    sendMetrics(
+      {
+        event: MetaMetricsEventName.DappViewed,
+        category: MetaMetricsEventCategory.InpageProvider,
+        referrer: {
+          url: origin,
+        },
+        properties: {
+          is_first_visit: isFirstVisit,
+          number_of_accounts: Object.keys(metamaskState.accounts).length,
+          number_of_accounts_connected: ethAccounts.length,
+        },
       },
-      properties: {
-        is_first_visit: isFirstVisit,
-        number_of_accounts: Object.keys(metamaskState.accounts).length,
-        number_of_accounts_connected: ethAccounts.length,
+      {
+        excludeMetaMetricsId: true,
       },
-    });
+    );
   }
 
   res.result = ethAccounts;

app/scripts/lib/rpc-method-middleware/handlers/request-accounts.test.ts

@@ -187,18 +187,23 @@ describe('requestEthereumAccountsHandler', () => {
       MockUtil.shouldEmitDappViewedEvent.mockReturnValue(true);
 
       await handler(baseRequest);
-      expect(sendMetrics).toHaveBeenCalledWith({
-        category: 'inpage_provider',
-        event: 'Dapp Viewed',
-        properties: {
-          is_first_visit: true,
-          number_of_accounts: 3,
-          number_of_accounts_connected: 2,
+      expect(sendMetrics).toHaveBeenCalledWith(
+        {
+          category: 'inpage_provider',
+          event: 'Dapp Viewed',
+          properties: {
+            is_first_visit: true,
+            number_of_accounts: 3,
+            number_of_accounts_connected: 2,
+          },
+          referrer: {
+            url: 'http://test.com',
+          },
         },
-        referrer: {
-          url: 'http://test.com',
+        {
+          excludeMetaMetricsId: true,
         },
-      });
+      );
     });
 
     it('does not emit the dapp viewed metrics event when shouldEmitDappViewedEvent returns false', async () => {

app/scripts/lib/rpc-method-middleware/handlers/wallet-createSession/handler.test.ts

@@ -511,18 +511,23 @@ describe('wallet_createSession', () => {
     const { handler, sendMetrics } = createMockedHandler();
     await handler(baseRequest);
 
-    expect(sendMetrics).toHaveBeenCalledWith({
-      category: 'inpage_provider',
-      event: 'Dapp Viewed',
-      properties: {
-        is_first_visit: true,
-        number_of_accounts: 3,
-        number_of_accounts_connected: 4,
+    expect(sendMetrics).toHaveBeenCalledWith(
+      {
+        category: 'inpage_provider',
+        event: 'Dapp Viewed',
+        properties: {
+          is_first_visit: true,
+          number_of_accounts: 3,
+          number_of_accounts_connected: 4,
+        },
+        referrer: {
+          url: 'http://test.com',
+        },
       },
-      referrer: {
-        url: 'http://test.com',
+      {
+        excludeMetaMetricsId: true,
       },
-    });
+    );
   });
 
   it('returns the known sessionProperties and approved session scopes', async () => {

app/scripts/lib/rpc-method-middleware/handlers/wallet-createSession/handler.ts

@@ -299,18 +299,24 @@ async function walletCreateSessionHandler(
 
       const approvedEthAccounts = getEthAccounts(approvedCaip25CaveatValue);
 
-      hooks.sendMetrics({
-        event: MetaMetricsEventName.DappViewed,
-        category: MetaMetricsEventCategory.InpageProvider,
-        referrer: {
-          url: origin,
+      hooks.sendMetrics(
+        {
+          event: MetaMetricsEventName.DappViewed,
+          category: MetaMetricsEventCategory.InpageProvider,
+          referrer: {
+            url: origin,
+          },
+          properties: {
+            is_first_visit: isFirstVisit,
+            number_of_accounts: Object.keys(hooks.metamaskState.accounts)
+              .length,
+            number_of_accounts_connected: approvedEthAccounts.length,
+          },
         },
-        properties: {
-          is_first_visit: isFirstVisit,
-          number_of_accounts: Object.keys(hooks.metamaskState.accounts).length,
-          number_of_accounts_connected: approvedEthAccounts.length,
+        {
+          excludeMetaMetricsId: true,
         },
-      });
+      );
     }
 
     res.result = {