Stability issues with seeking and buffering in Microsoft IE11

[torerikal]

This problem can be reproduced with the "Sintel" 1080p MP4" manifest referred in the Shaka player app, or a custom example stream:

1. Open Microsoft Edge or IE11 and the player test page, for instance http://shaka-player-demo.appspot.com/.
2. Select the mentioned test manifest from the dropdown and press Load stream.
3. Open F12 Developer tools, and select the Console tab.
4. Do a random seek by clicking at a new position in the timeline control.
5. Notice a red "undefined" error log entry in the console. Where could this originate from?
6. Repeat the seek procedure a few more times. This multiplies the "undefined" log entries.
7. Perhaps in 1 of 10 repetitions, an additional player error (custom event) shows up, with the description "Cannot fetch (audio/mp4): previous operation not complete". These might be followed by "Unexpected call to resync_()".
8. More randomly, further repeated seeking makes the stream stall or freeze in a buffering state, and it is not possible to get the playback recovered from this. This appears to be more frequent on custom streams or in IE11. In this case, there is no traces of other types of errors than the ones mentioned in step 5 and 7.

With our specific streams, the unrecoverable buffer state is unfortunately occurring a bit too often, and we are looking for a solution to cover Edge/IE 11 with DASH.

At this time, it is hard to conclude that the unrecoverable state is related to the observations in the console. Still, it would be interesting if you could look into and sort out the mentioned errors. The "undefined" entries could be a bit hard to trace down, though...

[joeyparrish]

"undefined" is caused by failing assertions which have no messages associated with them. In other browsers, this is not a huge issue because the JS console will still show the line number and file on which the assertion occurred. It would seem that for some reason, IE11 only gives you the message string and nothing else. (Because of this, we are now requiring messages for all assertions in Shaka v2.)

There's nothing I know of that can be done about "undefined" short of adding messages to all of them and re-running the test to see which assertions are failing.

I think I may have a handle of the seek problems, though. It may be an IE-specific race condition.

[joeyparrish]

Okay, not a race after all. Here's the setup:

1. Shaka assumes only one buffered range because it only buffers in order. It has no logic to handle edge cases that could arise with multiple ranges.
2. Seeking to an unbuffered position causes Shaka to clear the buffer (sbm.clear()) to maintain the above assumption.
3. If multiple buffered ranges are detected, Shaka assumes something has gone horribly wrong and clears the buffer (sbm.clear()) to try to re-establish the above assumption.

sbm.clear() is not working correctly on IE11. It calls SourceBuffer.remove(0, Number.POSITIVE_INFINITY), and although IE fires an 'updateend' event to say it has done so, it hasn't. So, what we end with is:

1. Seek to an unbuffered position causes Shaka to perform a clear() which is actually a no-op on IE11.
2. onUpdate_ is called, but only one range (the old one) is still buffered. It does not notice that currentTime is outside of this range, so it grabs a new segment and buffers it.
3. onUpdate_ is called again, notices that there are now multiple buffered ranges (old range, new segment) and tries to clear(), which is still a no-op on IE11.
4. Go to 3.

We should be able to fix this. Here are the results of my preliminary testing:

• remove(0, Number.POSITIVE_INFINITY) does nothing on IE11
• remove(0, Number.MAX_VALUE) also does nothing on IE11
• remove(0, mediaSource.duration) seems to work on IE11

[wolenetz]

That's quite ugly. Does the same situation occur with the Edge browser?

[joeyparrish]

I quickly smoke-tested the app without my patch in Edge and had no issues.

[joeyparrish]

@torerikal: Please try the latest sources from master, or this hosted preview which includes the fix: http://ie11test.shaka-player-demo.appspot.com/

[joeyparrish]

To document for future generations, on IE11:

• remove(0, Math.pow(2, 39)) works.
• remove(0, Math.pow(2, 40)) is a no-op.

[torerikal]

Thanks a lot for your investigations and fixes.

Unfortunately, I still get similar errors both in Edge and IE11 (see screenshot), but less frequently than earlier. I have earlier mentioned the error message "Cannot fetch (audio/mp4): previous operation not complete". Now I only observe "Cannot clear (video/mp4): previous operation not complete". I don't know if this distinction is of interest.

With the fixes, I haven't been able to stall the playback in your test manifest or our custom streams, which do look like an improvement.

[joeyparrish]

Okay, I'll take another pass at it. Perhaps I did not stress test it enough. I won't be able to get to this until the end of the month, though.