1419352 - Stop doing React updates while debugger is in background

Status: RESOLVED FIXED

(DevTools :: Debugger, enhancement, P5)

Description

[Alexandre Poirot [:ochameau]]

Similarely to bug 1399242, we could use VisibilityHandler React Element to prevent doing unecessary React updates while the panel is in background:
  https://hg.mozilla.org/integration/autoland/rev/3742c29706eb

Also, the new DAMP test specific to panels being in background should be tweaked to also cover the netmonitor:
  https://hg.mozilla.org/integration/autoland/rev/7eae6691b2f9

I opened this bug, mostly because of DAMP test that should be updated.
But feel free to cover that in a PR.

Comment 1

[Alexandre Poirot [:ochameau]]

Attachment: Bug 1419352 - [devtools] Stop background React updates when debugger is in background.

Comment 2

[Alexandre Poirot [:ochameau]]

Attachment: Bug 1419352 - [devtools] Cover performances of the debugger while in background.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/07197254dc7f
[devtools] Cover performances of the debugger while in background. r=perftest-reviewers,devtools-reviewers,nchevobbe,sparky
https://hg.mozilla.org/integration/autoland/rev/b5713e78b623
[devtools] Stop background React updates when debugger is in background. r=devtools-reviewers,nchevobbe

Comment 4

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/07197254dc7f
https://hg.mozilla.org/mozilla-central/rev/b5713e78b623