RUM-2903 fix: refresh rate vital for variable refresh rate displays when over performing

[ganeshnj]

What and why?

Last time when we touched this area, we normalized the FPS relative to the expected rendering rate for variable refresh rate displays.

ie when a display is rendering at 30fps and expected rate is also 30fps, we report 60fps. This is because our backend only supports 60fps and renders useful indicators.

However, in some case, it has been observed that sometimes display renders quicker than expected.

ie when a display is rendering at 45fps and expected rate is 30fps, in this case normalization doesn't work and we go on to report 90fps which is not correct.

How?

• For variable refresh rate displays, if the refresh rate goes above the 60fps, we cap the FPS to 60fps.
• There is no change in the behavior for fixed refresh rate displays.
• There is no change in the behavior for displays rendering slower than expected rate for variable refresh rate displays.

Review checklist

• Feature or bugfix MUST have appropriate tests (unit, integration)
• Make sure each commit and the PR mention the Issue number or JIRA reference
• Add CHANGELOG entry for user facing changes

[maxep]

🎯