commit 409386ac477042177283639c4d0d01b3c557328b
parent ac665134d1e82061489fadb438a56aadd565b0ab
Author: Andrej Glavic <aglavic@mozilla.com>
Date: Thu, 13 Nov 2025 16:58:57 +0000
Bug 1993763 - Make zygote data for chrome-m optional in resource usage tests. r=perftest-reviewers,sparky
This now makes it so all variables in base_measures must exist before we enter the metrics.extend section, thereby removing all risk of a keyerror which has been intermittently causing issues
Differential Revision: https://phabricator.services.mozilla.com/D272063
Diffstat:
1 file changed, 2 insertions(+), 0 deletions(-)
diff --git a/testing/performance/android-resource/parse_resource_usage.py b/testing/performance/android-resource/parse_resource_usage.py
@@ -26,6 +26,7 @@ def make_differential_metrics(
}
for mem_type, mem_info in mem_measures.items()
for category, mem_usage in mem_info.items()
+ if category in base_measures["mem"].get(mem_type, {})
]
)
metrics.extend(
@@ -54,6 +55,7 @@ def make_differential_metrics(
"values": [cpu_time - base_measures["cpu"][category]],
}
for category, cpu_time in cpu_measures.items()
+ if category in base_measures.get("cpu", {})
]
)
metrics.append(