Simplify breakage report. Don't trigger expanded on frame

Author: jonathanKingston

injected/src/features/breakage-reporting.js

@@ -12,7 +12,10 @@ export default class BreakageReporting extends ContentFeature {
                 referrer,
             };
             if (isExpandedPerformanceMetricsEnabled) {
-                result.expandedPerformanceMetrics = getExpandedPerformanceMetrics();
+                const expandedPerformanceMetrics = getExpandedPerformanceMetrics();
+                if (expandedPerformanceMetrics.success) {
+                    result.expandedPerformanceMetrics = expandedPerformanceMetrics.metrics;
+                }
             }
             this.messaging.notify('breakageReportResult', result);
         });

injected/src/features/performance-metrics.js

@@ -1,5 +1,6 @@
 import ContentFeature from '../content-feature';
 import { getExpandedPerformanceMetrics, getJsPerformanceMetrics } from './breakage-reporting/utils.js';
+import { isBeingFramed } from '../utils.js';
 
 export default class PerformanceMetrics extends ContentFeature {
     init() {
@@ -8,6 +9,10 @@ export default class PerformanceMetrics extends ContentFeature {
             this.messaging.notify('vitalsResult', { vitals });
         });
 
+        // If the document is being framed, we don't want to collect expanded performance metrics
+        if (isBeingFramed()) return;
+
+        // If the feature is enabled, we want to collect expanded performance metrics
         if (this.getFeatureSettingEnabled('expandedPerformanceMetricsOnLoad', 'enabled')) {
             document.addEventListener('load', () => {
                 this.triggerExpandedPerformanceMetrics();