fix(tracing): Remove connection.downlink measurement (#5794)

AbhiPrasad · web-flow · commit d6bcb476cd16 · 2022-09-22T14:09:39.000+02:00
As per RFC: https://github.com/getsentry/rfcs/blob/main/text/0003-browser-js-built-in-metrics.md we have decided to remove `connection.downlink` from as a recorded performance metrics in the `BrowserTracing` integration.
diff --git a/packages/tracing/src/browser/metrics/index.ts b/packages/tracing/src/browser/metrics/index.ts
@@ -395,10 +395,6 @@ function _trackNavigator(transaction: Transaction): void {
     if (isMeasurementValue(connection.rtt)) {
       _measurements['connection.rtt'] = { value: connection.rtt, unit: 'millisecond' };
     }
-
-    if (isMeasurementValue(connection.downlink)) {
-      _measurements['connection.downlink'] = { value: connection.downlink, unit: '' }; // unit is empty string for now, while relay doesn't support download speed units
-    }
   }
 
   if (isMeasurementValue(navigator.deviceMemory)) {

Original file line number	Diff line number	Diff line change
`@@ -395,10 +395,6 @@ function _trackNavigator(transaction: Transaction): void {`
`395`	`395`	`if (isMeasurementValue(connection.rtt)) {`
`396`	`396`	`_measurements['connection.rtt'] = { value: connection.rtt, unit: 'millisecond' };`
`397`	`397`	`}`
`398`		`-`
`399`		`- if (isMeasurementValue(connection.downlink)) {`
`400`		`- _measurements['connection.downlink'] = { value: connection.downlink, unit: '' }; // unit is empty string for now, while relay doesn't support download speed units`
`401`		`- }`
`402`	`398`	`}`
`403`	`399`
`404`	`400`	`if (isMeasurementValue(navigator.deviceMemory)) {`