Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix captions disappear on unbuffered seeks when user activates, deactivates, and reactivates captions #2674

Merged
merged 12 commits into from
Jun 24, 2020
Merged

Fix captions disappear on unbuffered seeks when user activates, deactivates, and reactivates captions #2674

Changes from 5 commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
ffaed64
Created MP4 box parsers to parse data for common box types (#2648)
muhammadharis Jun 11, 2020
e17d2f1
merged in master
muhammadharis Jun 21, 2020
2fb10c1
fix: captions disappear during unbuffered seeks when activating, deac…
muhammadharis Jun 22, 2020
e9d299b
Merge branch 'master' of https://github.com/google/shaka-player into …
muhammadharis Jun 22, 2020
c22d766
removed extra file from unrebased local master
muhammadharis Jun 22, 2020
9c0db06
fixed captions disappearing or showing double when turned off then on
muhammadharis Jun 23, 2020
57ef3bb
removed extraneous null test
muhammadharis Jun 23, 2020
b677f11
minor semantic fix in test description
muhammadharis Jun 23, 2020
037011a
added descriptive comment and tightened tests as per review
muhammadharis Jun 24, 2020
1629ee7
modified test message
muhammadharis Jun 24, 2020
a1f117a
added unit test in streaming_engine_unit.js
muhammadharis Jun 24, 2020
a8808cd
removed extra setup from test
muhammadharis Jun 24, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 4 additions & 1 deletion lib/player.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3507,8 +3507,11 @@ shaka.Player = class extends shaka.util.FakeEventTarget {
// can override this and configure us to always stream captions.
if (!this.config_.streaming.alwaysStreamText) {
if (newVisibility) {
if (this.streamingEngine_.getCurrentTextStream()) {
const currentTextStream = this.streamingEngine_
.getCurrentTextStream();
if (currentTextStream) {
// We already have a selected text stream.
this.streamingEngine_.switchTextStream(currentTextStream);
} else {
// Find the text stream that best matches the user's preferences.
const streams =
Expand Down