How to avoid instance clash?

[phloxic]

This will turn out to be embarrassing for me, but here goes:
How do I create more than one Hls instance?
This simple example fails miserably: http://flowplayer.blacktrash.org/fp6/hlsjs-pure.html

I'm missing something basic here, but what?

[mangui]

that is weird, 4 MSE_ATTACHED events are triggered, whereas you only have 2 instances (2 MediaSource created).
I am not yet clear why ? MediaSource listener is attached to its associated hls instance only.
see https://github.com/dailymotion/hls.js/blob/master/src/hls.js#L95-L102

as a consequence, loadSource() is called twice in a row per playlist, there might be something here as well, but the MSE_ATTACHED is not supposed to behave like this.

[mangui]

also the MediaSource 'sourceopen' event listener is removed after its first trigger: https://github.com/dailymotion/hls.js/blob/master/src/hls.js#L260-L261

[phloxic]

Highly speculative:

• remove the listener before triggering MSE_ATTACHED?
• it's still a video tag which maybe fires loadedmetadata and loadeddata in quick succession: intercept just one of the two?

In retrospect I now know see why I had to always destroy an existing instance to make it work with playlists: https://github.com/flowplayer/flowplayer-hlsjs/blob/master/flowplayer.hlsjs.js#L50-L53

From the outside it looks like the Hls context persists, and maybe needs an instance wrapper?

I noticed another weird thing, which may be related. On Flowplayer's unload call, normally the video tag is removed. This somehow does not happen with hls.js - if I omit https://github.com/flowplayer/flowplayer-hlsjs/blob/master/flowplayer.hlsjs.js#L196 the tag stays, albeit empty (probably only a timing issue, but maybe it helps to know).

Something is very persistent here ;-)

[mangui]

observer is global and not per instance ...
https://github.com/dailymotion/hls.js/blob/master/src/observer.js#L3
thus all events are duplicated accross all instances.
it should not be to complex to handle, will check soon

[mangui]

@blacktrash it should work now, you can verify with
https://github.com/dailymotion/hls.js/blob/master/tests/functional/multiple/index.html

[mangui]

or here, gh-pages is now up to date
http://dailymotion.github.io/hls.js/tests/functional/multiple/

[phloxic]

Great! Can also be confirmed at http://flowplayer.blacktrash.org/fp6/hlsjs-pure.html

[mangui]

fixed