[Web] Frame Rate and UI fix

[cemalgnlts]

Hi,

I think a few frames are omitted because of the floating point numbers in this line.

corona/platform/emscripten/Rtt_EmscriptenContext.cpp

Lines 67 to 77 in ec49350

	static U64 s_tick = 0;
	void TimerTickShim(void *userdata)
	{
	CoronaAppContext *context = (CoronaAppContext*) userdata;
	float frameDuration = 1000.0f / (float) context->getFPS();
	U64 now = Rtt_AbsoluteToMilliseconds(Rtt_GetAbsoluteTime());
	if (now - s_tick >= frameDuration) // 60fps ==> 1000/60 = 16.66666 msec
	{
	s_tick = now;
	context->TimerTick();

I can show it with JS (you can paste it into the console in Developer Options and see the result):

function runTest(fps) {
  let last = 0;
  let actualFrame = 0;
  let s2dFrame = 0;
  
  const FPS = fps ?? 60;
  const MS_PER_SEC = 1000 / FPS;
  
  function onFrame(now) {
      actualFrame += 1;
  
      if(now - last >= MS_PER_SEC) {
          last = now;
          s2dFrame += 1;
      }
  
      if(actualFrame % (FPS * 3) === 0) { // run 3 seconds
          completed();
      } else {
          requestAnimationFrame(onFrame);
      }
  }
  
  requestAnimationFrame(onFrame);
  
  function completed() {
      console.log("Frame got: %s, S2D got: %s, Missed Frame: %s", actualFrame, s2dFrame, actualFrame - s2dFrame);
  }
}

The results I get when I run this example:

runTest() // Frame got: 180, S2D got: 92, Missed Frame: 88
runTest(30) // Frame got: 90, S2D got: 31, Missed Frame: 59

If we update it this way and try again:

- const MS_PER_SEC = 1000 / FPS;
+ const MS_PER_SEC = 1 / FPS;

runTest() // Frame got: 180, S2D got: 180, Missed Frame: 0
runTest(30) // Frame got: 90, S2D got: 90, Missed Frame: 0

I also fixed that if the status text is blank on the loading screen, the height is reset and the logo slides down.

The commit history may look a bit dirty because the code is not compiled properly in linux (maybe I couldn't do it).

[CLAassistant]

All committers have signed the CLA.

[Shchvova]

Was this fixed by other PR already?