Skip to content

Commit

Permalink
chore: revert "don't do sequence mode workaround unless there's a tex…
Browse files Browse the repository at this point in the history
…t stream" (#5108)

Reverts #5079
  • Loading branch information
avelad authored Mar 22, 2023
1 parent 71affe7 commit 66219bc
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 8 deletions.
7 changes: 0 additions & 7 deletions lib/media/media_source_engine.js
Original file line number Diff line number Diff line change
Expand Up @@ -408,13 +408,6 @@ shaka.media.MediaSourceEngine = class {
'expected \'open\'');
}

if (sequenceMode && !streamsByType.has(ContentType.TEXT)) {
// There's no text stream, so we can set sequence mode early instead
// of setting it after the first segment is appended in appendBuffer_.
sourceBuffer.mode =
shaka.media.MediaSourceEngine.SourceBufferMode_.SEQUENCE;
}

this.eventManager_.listen(
sourceBuffer, 'error',
() => this.onError_(contentType));
Expand Down
1 change: 0 additions & 1 deletion test/media/media_source_engine_unit.js
Original file line number Diff line number Diff line change
Expand Up @@ -639,7 +639,6 @@ describe('MediaSourceEngine', () => {
};
const initObject = new Map();
initObject.set(ContentType.VIDEO, fakeVideoStream);
initObject.set(ContentType.TEXT, fakeTextStream);

await mediaSourceEngine.init(initObject, /* sequenceMode= */ true);

Expand Down

0 comments on commit 66219bc

Please sign in to comment.