Re: [html-tests] Test media element load algorithm sequence (#571)

Sorry I didn't see your comment two weeks ago, I've taken a look at this again now. Try to avoid using an arbitrary timeout like 100 ms, it's usually possible to avoid by e.g. waiting for an event you know comes later (loadstart) than the one your testing for (emptied). In this case, that timeout caused networkState to sometimes be NETWORK_LOADING and sometimes NETWORK_IDLE when I alerted the value, which was OK for the test, but failed to try all three non-empty networkStates.

So... I took some time to test all the networkStates and to include the abort event, as you can see above. Can you see if this catches the IE bug, and if I missed to steal anything from your test?

View on GitHub: https://github.com/w3c/web-platform-tests/pull/571#issuecomment-34773447

Received on Tuesday, 11 February 2014 16:44:02 UTC