Closed Bug 1448133 Opened 6 years ago Closed 6 years ago

Add a Nightly-only run-time pref to run tab throbber animation at 30fps

Categories

(Firefox :: Theme, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 61
Tracking Status
firefox61 --- fixed

People

(Reporter: mconley, Assigned: mconley)

References

Details

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #1447153 +++

This is to make it easier to diagnose things like bug 1406414, bug 1439511 and bug 1428704.

This is Nightly only because this is only for testing. This configuration should never reach release.
Comment on attachment 8961526 [details]
Bug 1448133 - Add a Nightly-only preference to set the tab throbber to animate at 30fps.

https://reviewboard.mozilla.org/r/230314/#review236302
Attachment #8961526 - Flags: review?(jaws) → review+
Pushed by mconley@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7bc1183fb32e
Add a Nightly-only preference to set the tab throbber to animate at 30fps. r=jaws
https://hg.mozilla.org/mozilla-central/rev/7bc1183fb32e
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 61
Assignee: nobody → mconley
Setting step function is pretty useless for compositor animations.  We do skip restyling for such animations on the main-thread[1], but we don't do it on the compositor.

[1] https://hg.mozilla.org/mozilla-central/file/7ff499dfcd51/dom/animation/KeyframeEffectReadOnly.cpp#l132
(In reply to Hiroyuki Ikezoe (:hiro) from comment #5)
> Setting step function is pretty useless for compositor animations.  We do
> skip restyling for such animations on the main-thread[1], but we don't do it
> on the compositor.

That's super disappointing. Is there a way we could make the compositor smarter here?
Flags: needinfo?(hikezoe)
Filed bug 1454324 for that.
Flags: needinfo?(hikezoe)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: