Closed Bug 1651165 Opened 1 year ago Closed 1 year ago

Rename and improve documentation around _scheduleArbitrarilyLateIdleTasks

Categories

(Firefox :: General, task)

task

Tracking

()

RESOLVED FIXED
Firefox 80
Tracking Status
firefox80 --- fixed

People

(Reporter: Gijs, Assigned: dthayer)

References

(Blocks 1 open bug)

Details

(Whiteboard: [fxperf:p1])

Attachments

(2 files)

This helper only runs things when the user is not using the machine (not just Firefox). Due to the multiple meanings of "idle" used in various API names (viz. ChromeUtils.idleDispatch() referring to whether gecko/firefox has stuff to do before the next paint / rendering tick), this is not obvious. We should rename the method so it's more obvious, and in bug 1651161 evaluate which of the tasks that we run this way actually need/want to be run when the user is not using the machine, and/or if it's acceptable if they never end up getting run (which may currently happen).

I'm just going to go ahead and take this, in lieu of giving it a priority.

Assignee: nobody → dothayer
Whiteboard: [fxperf] → [fxperf:p1]
Pushed by dothayer@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2a646a7936e7
Rename idle service r=Gijs,geckoview-reviewers,snorp
https://hg.mozilla.org/integration/autoland/rev/d67baa908131
Rename and redocument _scheduleArbitrarilyLateIdleTasks r=Gijs
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 80
Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/ab1568fa6f2b
Rename nsIIdleService to nsIUserIdleService in Thunderbird code. rs=bustage-fix
You need to log in before you can comment on or make changes to this bug.