Clean up FPSCounter's use of time durations

NEW
Unassigned

Status

()

Core
Graphics: Layers
3 years ago
3 years ago

People

(Reporter: jrmuizel, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
Created attachment 8480599 [details] [diff] [review]
Clean up FPSCounter's use of time durations
Attachment #8480599 - Flags: review?(bgirard)
(Reporter)

Comment 1

3 years ago
FWIW this should improve performance because we're not doing any floating point math per frame.
Comment on attachment 8480599 [details] [diff] [review]
Clean up FPSCounter's use of time durations

Review of attachment 8480599 [details] [diff] [review]:
-----------------------------------------------------------------

::: gfx/layers/composite/FPSCounter.cpp
@@ +110,5 @@
>  {
>    // Order of evaluation here has to stay the same
>    // otherwise IteratedFullInterval reads from mFrameTimestamps which cannot
>    // be null
> +  if (mIteratorIndex != mWriteIndex)

==
Attachment #8480599 - Flags: review?(bgirard) → review+
You need to log in before you can comment on or make changes to this bug.