Collect Firefox CPU consumption (#4051)

This PR makes sure it's collectable. We don't show it in the HTML, lets make a follow up to fix that. #3944
sitespeedio · Feb 11, 2024 · 4d6fd5e · 4d6fd5e
1 parent 74c9c20
commit 4d6fd5e
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/lib/cli/cli.js b/lib/cli/cli.js
@@ -724,6 +724,14 @@ export async function parseCommandLine() {
       type: 'number',
       group: 'Firefox'
     })
+    .option('browsertime.firefox.powerConsumption', {
+      alias: 'firefox.powerConsumption',
+      type: 'boolean',
+      default: false,
+      describe:
+        'Enable power consumption collection (in Wh). To get the consumption you also need to set firefox.geckoProfilerParams.features to include power.',
+      group: 'Firefox'
+    })
     .option('browsertime.firefox.geckodriverArgs', {
       alias: 'firefox.geckodriverArgs',
       describe:

diff --git a/lib/plugins/browsertime/index.js b/lib/plugins/browsertime/index.js
@@ -360,6 +360,11 @@ export default class BrowsertimePlugin extends SitespeedioPlugin {
                 run.memory = result[resultIndex].memory[runIndex];
               }
 
+              if (result[resultIndex].powerConsumption) {
+                run.powerConsumption =
+                  result[resultIndex].powerConsumption[runIndex];
+              }
+
               if (result[resultIndex].extras) {
                 run.extras = result[resultIndex].extras[runIndex];
               }