915342 - also display fill ratio when showing FPS counters

Status: RESOLVED FIXED

(Core :: Graphics: Layers, defect)

Description

[Andreas Gal :gal]

This patch prints the current fill ratio in addition to the FPS counters. The fill ratio is defined as estimated-number-of-pixels-we-fill / framebuffer-pixels * 100.

Comment 1

[Andreas Gal :gal]

Attachment: patch

Comment 2

[Benoit Girard (:BenWa)]

Comment on attachment 803229 [details] [diff] [review]
patch

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

Ship it!

::: gfx/layers/opengl/CompositorOGL.cpp
@@ +218,5 @@
> +  unsigned txnFps = unsigned(mTransactionFps.GetFpsAt(aNow));
> +
> +  DrawCounter(0, fps, context, copyprog);
> +  DrawCounter(80, txnFps, context, copyprog);
> +  DrawCounter(160, aFillRatio, context, copyprog);

Nice, much cleaner!

@@ +1317,5 @@
>    }
>  
>    if (mFPS) {
> +    double fillRatio = 0;
> +    if (mPixelsFilled > 0.0f && mPixelsPerFrame > 0.0f) {

mPixelsFilled > 0 && mPixelsPerFrame > 0

@@ +1320,5 @@
> +    double fillRatio = 0;
> +    if (mPixelsFilled > 0.0f && mPixelsPerFrame > 0.0f) {
> +      fillRatio = 100.0f * double(mPixelsFilled) / double(mPixelsPerFrame);
> +      if (fillRatio > 999.0f)
> +        fillRatio = 999.0f;

We don't need double precision here since we only need the 3 most significant digits. Let's make everything here float since the code here is converting back and forth between double/float and finally to unsigned.

Comment 3

[Andreas Gal :gal]

https://hg.mozilla.org/integration/mozilla-inbound/rev/6be52ab5df74

Comment 4

[Andreas Gal :gal]

Ugh I landed the wrong version of the patch.

Comment 5

[Andreas Gal :gal]

The delta:

-    double fillRatio = 0;
-    if (mPixelsFilled > 0.0f && mPixelsPerFrame > 0.0f) {
-      fillRatio = 100.0f * double(mPixelsFilled) / double(mPixelsPerFrame);
+    float fillRatio = 0;
+    if (mPixelsFilled > 0 && mPixelsPerFrame > 0) {
+      fillRatio = 100.0f * float(mPixelsFilled) / float(mPixelsPerFrame);

Comment 6

[Andreas Gal :gal]

And the double->float fix from comment 5:

https://hg.mozilla.org/integration/mozilla-inbound/rev/45fab5569c95

Comment 7

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/6be52ab5df74
https://hg.mozilla.org/mozilla-central/rev/45fab5569c95