Open Bug 1698958 Opened 3 years ago Updated 3 years ago

ReflowActor should not create permanent 300ms timers

Categories

(DevTools :: Inspector, task, P3)

task

Tracking

(Fission Milestone:Future)

Fission Milestone Future

People

(Reporter: jdescottes, Unassigned)

References

(Blocks 3 open bugs)

Details

See bug 1698338

The reflow actor relies on a timer triggered every 300ms to emit new reflows and resize events: https://searchfox.org/mozilla-central/rev/526a5089c61db85d4d43eb0e46edaf1f632e853a/devtools/server/actors/reflow.js#305,324

The intent was probably to avoid spamming the client with too many events, but we could use throttling/debouncing instead of having a loop.

With Fission, the DevTools server might be instantiated in several processes at the same time, so it makes sense to try to reduce the amount of unnecessary background work generated by DevTools.

Blocking dt-fission-perf

Component: Performance → Inspector
Product: Testing → DevTools
Version: Default → unspecified
Fission Milestone: --- → Future
You need to log in before you can comment on or make changes to this bug.