Bug 1699039 Comment 8 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
Every frame, we refresh at least 2 react components: the CurrentTimeScrubber and the CurrentTimeLabel. 

Commenting out their current-time-timer listener ([1](https://searchfox.org/mozilla-central/rev/74d0efd1107a26f178b108b6a18a179e9b06547c/devtools/client/inspector/animation/components/CurrentTimeLabel.js#30) & [2](https://searchfox.org/mozilla-central/source/devtools/client/inspector/animation/components/CurrentTimeScrubber.js#45)) helps to keep the CPU at regular levels.
Revision 1 by
on 
Every frame, we refresh at least 2 react components: the CurrentTimeScrubber and the CurrentTimeLabel. 

Commenting out their current-time-timer listener ([CurrentTimeScrubber](https://searchfox.org/mozilla-central/rev/74d0efd1107a26f178b108b6a18a179e9b06547c/devtools/client/inspector/animation/components/CurrentTimeLabel.js#30) & [CurrentTimeLabel](https://searchfox.org/mozilla-central/source/devtools/client/inspector/animation/components/CurrentTimeScrubber.js#45)) helps to keep the CPU at regular levels.
Revision 2 by
on 
Every frame, we refresh at least 2 react components: the CurrentTimeScrubber and the CurrentTimeLabel. 

Commenting out their current-time-timer listener ([CurrentTimeScrubber](https://searchfox.org/mozilla-central/rev/74d0efd1107a26f178b108b6a18a179e9b06547c/devtools/client/inspector/animation/components/CurrentTimeLabel.js#30) & [CurrentTimeLabel](https://searchfox.org/mozilla-central/source/devtools/client/inspector/animation/components/CurrentTimeScrubber.js#45)) helps to keep the CPU at regular levels.

Both components are really basic, but I guess involving React components in a requestAnimationFrame loop is not working out.
Revision 3 by
on 
Every frame, we refresh at least 2 react components: the CurrentTimeScrubber and the CurrentTimeLabel. 

Commenting out their current-time-timer listener ([CurrentTimeScrubber](https://searchfox.org/mozilla-central/rev/74d0efd1107a26f178b108b6a18a179e9b06547c/devtools/client/inspector/animation/components/CurrentTimeLabel.js#30) & [CurrentTimeLabel](https://searchfox.org/mozilla-central/source/devtools/client/inspector/animation/components/CurrentTimeScrubber.js#45)) helps to keep the CPU at regular levels.

Both components are really basic, but I guess involving React components in a requestAnimationFrame loop is not working out.
In theory this also means that this affects any page which has animations continuously playing, regardless of the number of animations.

Back to Bug 1699039 Comment 8