165394 2016-12-05 11:46:32 -0800 [GStreamer][MSE] Support multiple streams per SourceBuffer and dynamic audio/video/text type changes 2022-08-05 14:43:05 -0700 1 1 1 Unclassified WebKit Platform WebKit Local Build Unspecified Unspecified RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=167108 https://bugs.webkit.org/show_bug.cgi?id=167109 P2 Enhancement --- 229072 167107 1 eocanha webkit-unassigned aboya bugs-noreply clopez eocanha hoboprimate pnormand oldest_to_newest 1256411 0 eocanha 2016-12-05 11:46:32 -0800 The current GStreamer MSE platform implementation doesn't support more than one stream per SourceBuffer. Caps changes are managed as stream quality changes in order to support adaptive streaming, but that's all. An audio/video/text stream can't be reconverted to other type. This feature is required for LayoutTests/media/media-source/media-source-seek-detach-crash.html to work properly. 1358932 1 aboya 2017-10-10 15:12:25 -0700 > An audio/video/text stream can't be reconverted to other type. This feature > is required for > LayoutTests/media/media-source/media-source-seek-detach-crash.html to work > properly. Note that MSE not only does not require support for converting streams to other formats, but even forbids it per spec: https://www.w3.org/TR/media-source/#sourcebuffer-init-segment-received 3. If the first initialization segment received flag is true, then run the following steps: 1. Verify the following properties. If any of the checks fail then run the append error algorithm and abort these steps. * The number of audio, video, and text tracks match what was in the first initialization segment. * The codecs for each track, match what was specified in the first initialization segment. * If more than one track for a single type are present (e.g., 2 audio tracks), then the Track IDs match the ones in the first initialization segment. What the test is checking is that if that any of these conditions is met the error is handled gracefully, which is not currently the case. The test expectation should be changed from [ Skip ] to [ Crash ] and the bug eventually fixed. 1698075 2 pnormand 2020-10-15 05:10:50 -0700 At least one stream in the wild hitting this issue: https://peer.tube/videos/watch/5a370009-1868-46de-a5c2-2b34405fc5c7 1698095 3 aperez 2020-10-15 07:23:24 -0700 *** Bug 217580 has been marked as a duplicate of this bug. *** 1815607 4 eocanha 2021-11-16 12:37:22 -0800 The multiple streams per SourceBuffer has been implemented in r281440. Peertube videos work now, and I ensured that they have multiple streams per SourceBuffer. Resolving this bug as fixed. 1815609 5 eocanha 2021-11-16 12:42:03 -0800 ...but the stream type reconversion (MSE changeType()[1]) is still missing. [1] https://w3c.github.io/media-source/#dom-sourcebuffer-changetype 1889231 6 aboya 2022-08-05 14:43:05 -0700 There is no reason to have this bug set as REOPENED. (In reply to Enrique Ocaña from comment #5) > ...but the stream type reconversion (MSE changeType()[1]) is still missing. > > [1] https://w3c.github.io/media-source/#dom-sourcebuffer-changetype A changeType implementation is a completely different effort. Note it wasn't even in the spec when this original bug was open. Also, as I explained "An audio/video/text stream can't be reconverted to other type" is a requirement of the spec. Even with changeType, SourceBuffers can't change from video into audio. The number and type of tracks must much, but container and codecs can differ. https://w3c.github.io/media-source/#ref-for-dfn-first-initialization-segment-received-flag-2 Feel free to open a separate bug for a changeType implementation in MSE GStreamer. That is a feature that is in the cooking pot, but to which it's hard to get to while being overwhelmed with bug fixing.