ANR Events not correct when cover the screen

[l3ve]

Is there an existing issue for this?

• I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
• I have reviewed the documentation https://docs.sentry.io/
• I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases

How do you use Sentry?

Sentry Saas (sentry.io)

Electron SDK Version

5.8.0

Electron Version

31.2.1

What platform are you using?

MacOS

Link to Sentry event

No response

Steps to Reproduce

1. Test in Electron Fiddle

//  main.js
const { app, BrowserWindow, powerMonitor } = require('electron')
const path = require('node:path')

function createWindow() {
  const mainWindow = new BrowserWindow({
    width: 800,
    height: 600,
    webPreferences: {
      preload: path.join(__dirname, 'preload.js')
    }
  })
  mainWindow.loadFile('index.html')
  mainWindow.webContents.openDevTools()
}
app.whenReady().then(() => {
  createWindow()
  powerMonitor.on('lock-screen', () => {
    console.log('lock-screen', new Date().toLocaleString("chinese", {
      hour12: false,
      timeZone: "Asia/Shanghai"
    }))
  })

  powerMonitor.on('unlock-screen', () => {
    console.log('unlock-screen', new Date().toLocaleString("chinese", {
      hour12: false,
      timeZone: "Asia/Shanghai"
    }))
  })
  app.on('activate', function () {
    if (BrowserWindow.getAllWindows().length === 0) createWindow()
  })
})
app.on('window-all-closed', function () {
  if (process.platform !== 'darwin') app.quit()
})

// renderer.js
document.addEventListener('visibilitychange', () => {
    console.log('document.visibilityState:', document.visibilityState, new Date().toLocaleString("chinese", {
      hour12: false,
      timeZone: "Asia/Shanghai"
    }))
});

2. cover the screen, over 20s
3. open the screen
4. it will emit the anr event, becouse the visibilitychange event isn't emit in time
5. the powerMonitor event is correct

• visibilitychange(hidden) emit when open the screen(lock)
• visibilitychange(visible) emit when open the screen(unlock)

Expected Result

ignore anr events when cover the screen

Actual Result

ANR Events emited when cover the screen

[Lms24]

Hey, thanks for writing in! @timfish when you get a chance, would you mind taking a look? thanks!

[timfish]

Yep just looking at this. We already disable ANR between suspend and resume events and it looks like we need to do the same for lock-screen and unlock-screen too.

[l3ve]

After upgrading to the new version, another ANR was triggered, which seems to be the timing of the visibilitychange event is not convenient.

"@sentry/electron": "5.11.0"

[l3ve]

@timfish @Lms24 Please take a look. thx~