Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Firefox on Android always runs power usage tests #2073

Open
soulgalore opened this issue Feb 1, 2024 · 2 comments
Open

Firefox on Android always runs power usage tests #2073

soulgalore opened this issue Feb 1, 2024 · 2 comments

Comments

@soulgalore
Copy link
Member

When I run tests on Firefox on Android I always get a log line like:
CPU / Power usage: 9180

The power usage should be turned off by default, so I think there's a bug.

@gmierz
Copy link
Collaborator

gmierz commented Mar 26, 2024

@soulgalore I think this is expected since we always gather the CPU data:

let powerusage = await runner.runPrivilegedScript(`
return new Promise(async function(resolve) {
Services.fog.initializeFOG(); // prevents timeout when collecting CPU metrics
await Services.fog.testFlushAllChildren(); // force data from child processes to be sent to the parent.
resolve(Glean.power.totalCpuTimeMs.testGetValue());
});
`);
if (powerusage) {
log.info('CPU / Power usage: ' + powerusage);
}
result.cpu = powerusage;

Maybe the log output needs to be reworded to only mention that it's CPU usage?

@soulgalore
Copy link
Member Author

Let me check that, I think I reworded issue wrong. I think we always use root on Android when we test using Firefox and I was thinking we needed root to get that metric but I was wrong. Let me test again next week see what's going on.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants