Heavy CPU usage with tray.setImage

[johannesjo]

Issue Details

• Electron Version:
  • 10.1.5
• Operating System:
  • several different Linux OS

Several of my users seem to experience heavy CPU usage with the tray icon:
johannesjo/super-productivity#675

[ckerr]

Thanks for reporting this and helping to make Electron better!

Would it be possible for you to make a standalone testcase with only the code necessary to reproduce the issue? For example, Electron Fiddle is a great tool for making small test cases and makes it easy to publish your test case to a gist that Electron maintainers can use.

Stand-alone test cases make fixing issues go more smoothly: it ensure everyone's looking at the same issue, it removes all unnecessary variables from the equation, and it can also provide the basis for automated regression tests.

I'm adding the blocked/needs-repro label for this reason. After you make a test case, please link to it in a followup comment.

Thanks in advance! Your help is appreciated.

[johannesjo]

Would checking out the repository provided above work as well?

[johannesjo]

Might also be an issue of the appindicator extension:
https://bugs.launchpad.net/ubuntu/+source/gnome-shell-extension-appindicator/+bug/1849142
ubuntu/gnome-shell-extension-appindicator#236

[github-actions]

This issue has been automatically marked as stale. If this issue is still affecting you, please leave any comment (for example, "bump"), and we'll keep it open. If you have any new additional information—in particular, if this is still reproducible in the latest version of Electron or in the beta—please include it with your comment!

[johannesjo]

bump

[zcbenz]

Checking out the pasted repo would require us to debug into the repo's code, which our team does not have time to do. If you could reproduce the problem with some simple code like below it would be much more practical for us to look into it, otherwise we have to close this issue as most of other Electron apps do not have this issue.

const {app, Tray} = require('electron')
let tray
app.once('ready', () => {
  tray = new Tray('/path/to-icon')
})

[johannesjo]

This should reproduce the issue.

const {app, Tray} = require('electron')
let tray

app.once('ready', () => {
  tray = new Tray('/path/to-icon')

 setInterval(()=>{
        tr.setImage('path/to-icon);
 }, 500);
})

[github-actions]

This issue has been automatically marked as stale. If this issue is still affecting you, please leave any comment (for example, "bump"), and we'll keep it open. If you have any new additional information—in particular, if this is still reproducible in the latest version of Electron or in the beta—please include it with your comment!

[github-actions]

This issue has been closed due to inactivity, and will not be monitored. If this is a bug and you can reproduce this issue on a supported version of Electron please open a new issue and include instructions for reproducing the issue.

[ckerr]

I ran the testcase @ #26603 (comment) with Electron v23.2.0 on Ubuntu 22.04.02 and saw only minimal CPU use, so apparently this was fixed sometime in the interim between Electron 10 and 23 💫

If anyone is still experiencing this issue on a supported version of Electron, please file a new issue for that version of Electron. Thanks!