Adjusting autoplay to properly wait for loadstart event.

[jnwng]

Since moving the loadstart into the player.ready function here, there
exists a race condition whereby the player emits a play event
before the loadstart event, but since onLoadStart was waiting
for that play event, the play button never goes away.

alternatively, we could adjust https://github.com/videojs/video.js/blob/master/src/js/player.js#L412-L413 to instead wait for the playing event instead of the play event, since we know that the playing event will always be after the loadstart, whereas play is not guaranteed to be so.

[mmcc]

This is great, thank you for addressing this. This should address #1245, as well as #1237 (and others).

[jnwng]

Actually, after reviewing the other issues, I think the proper solution
would be to have onLoadStart wait for the 'playing' event and not the
'play' event, since it looks like many people will call 'play' in the ready
block, not just for the case I have here.

[Mewte]

I'm trying to call .play() in the ready block and it never removes the big play button or shows the controls. If I call play inside this.one('loadstart', function(){}); inside the ready event it does work, however. Is this issue related to that bug? I've been searching for hours to see if anyone has the same problem and this seems to be the closest issue I've seen.

[heff]

@jnwng Working on the onLoadStart fix you mention here in #1271

[heff]

@jnwng, 4.6.2 is out with a patch. Can you check to make sure that fixes the issue you saw?

[jnwng]

yep, this fixes it. thanks @heff!