1683031 - Refactoring patches for IdleSchedulerParent

Status: RESOLVED FIXED

(Core :: DOM: Content Processes, task)

Description

[Paul Bone [:pbone]]

I'm moving some patches from Bug 1629064 since there's more refactoring than I thought.

Comment 1

[Paul Bone [:pbone]]

Attachment: Bug 1683031 - pt 1. Fix some comments in IdleSchedulerParent.cpp r=smaug

Comment 2

[Paul Bone [:pbone]]

Attachment: Bug 1683031 - pt 2. Remove unused variable r=smaug

Comment 3

[Paul Bone [:pbone]]

Attachment: Bug 1683031 - pt 3. Use a single linked list for idle task queues r=smaug

This refactoring will make the following patches make more sense.

Comment 4

[Paul Bone [:pbone]]

Attachment: Bug 1683031 - pt 4. IdleScheduler may now schedule multiple items from a loop r=smaug

Comment 5

[Paul Bone [:pbone]]

Attachment: Bug 1683031 - pt 5. Add an assertion and document idle request IDs r=smaug

Add assertions and comments to explain the behaviour/use of the idle request
IDs.

Depends on D99330

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/66a0f5832ec2
pt 1. Fix some comments in IdleSchedulerParent.cpp r=smaug
https://hg.mozilla.org/integration/autoland/rev/070b6982780c
pt 2. Remove unused variable r=smaug
https://hg.mozilla.org/integration/autoland/rev/8622ff028777
pt 3. Use a single linked list for idle task queues r=smaug
https://hg.mozilla.org/integration/autoland/rev/42ba809d0936
pt 4. IdleScheduler may now schedule multiple items from a loop r=smaug
https://hg.mozilla.org/integration/autoland/rev/8006d28a1efa
pt 5. Add an assertion and document idle request IDs r=smaug

Comment 7

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/66a0f5832ec2
https://hg.mozilla.org/mozilla-central/rev/070b6982780c
https://hg.mozilla.org/mozilla-central/rev/8622ff028777
https://hg.mozilla.org/mozilla-central/rev/42ba809d0936
https://hg.mozilla.org/mozilla-central/rev/8006d28a1efa