Fixed a bug in MSIE8 where HTML5 and Flash are not available

[sugendran]

the active user checker was throwing errors because there was no tech available for playback

[heff]

Thanks for this! I think this would be better fixed by adding the check for tech here, where the tech actually referenced:
https://github.com/sugendran/video.js/blob/IE8BugFix/src/js/player.js#L1278

Would you mind updating it and testing if that works?

[sugendran]

Can look at that later today/tomorrow.

Should the inactive classes still get applied if there is no tech to play? At that point we'd just see the not supported message. So is there any reason to add more events to the DOM to find out if the user is active or not?

[heff]

Well there's a small issue where if we were some how able to recover from the error, e.g. setting a new source, the userActive listeners would no longer work in that case as is. With that in mind, I don't see any specific problem with still adding the class, but if you see something let me know.

[heff]

Also #631 has a comment that says there's a new error after fixing this one. Do you see the same?

[heff]

This has been fixed now. Thanks!
https://github.com/videojs/video.js/blob/master/src/js/player.js#L1283