🐛 Fixes NaN aspect ratio issue

[themisir]

Fixes #2504 - please check the linked issue for more details about the issue.

Instead of relying solely on onloadedmetadata callback, it's been recommended to use onloadeddata instead. I suppose loadedmetadata gets called earlier in most cases, so it still makes sense to use it to initialize player view, however I added a flag and a onloadeddata callback to handle the case where the loadedmetadata callback is called while the element video size attributes have not been initialized (or initialized with 0 value).

This ensures the player aspect ratio never gets set to NaN and provides an alternative way to correct the default 16:9 ratio if necessary.

[moisout]

Wow, thank you very much! Code looks good.

[moisout]

@all-contributors please add @themisir for code

[allcontributors]

@moisout

I've put up a pull request to add @themisir! 🎉