1723181 - "HasImageRequests" could be clearer in RefreshDriverTick profiler markers

Status: RESOLVED FIXED

(Core :: Web Painting, task)

Description

[Florian Quèze [:florian]]

From looking around at the code and trying to understand the behavior, it seems we have "HasImageRequests" as the tick reason when the refresh driver is ticking because of animated images.

An example test case to reproduce this is loading about:preferences.

I'm proposing that in the profiler marker we rename "HasImageRequests" into the more self explanatory "HasImageAnimations", and that we add a new "Image Animation" marker (that name fits with the existing "Image Load" and "Image Paint" markers).

Example profile with the changes: https://share.firefox.dev/3ic5Hdq

Comment 1

[Florian Quèze [:florian]]

Attachment: Bug 1723181 - Add an 'Image Animation' profiler marker, and rename the tick reason 'HasImageRequests' into 'HasImageAnimations' for clarity, r=mstange.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4fc8a8759035
Add an 'Image Animation' profiler marker, and rename the tick reason 'HasImageRequests' into 'HasImageAnimations' for clarity, r=mstange.

Comment 3

[Marian-Vasile Laza]

https://hg.mozilla.org/mozilla-central/rev/4fc8a8759035