corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 1509446 - part6 : update cues display without checking whether cue is active or not. r=jya

Browse Source 
We should update cue display everytime when the cues list changed.

In addition, we shouldn't check whether cue is active when we update display, because it's always inactive when the cue has been removed from `TextTrack::RemoveCue()`.

Differential Revision: https://phabricator.services.mozilla.com/D21143
This commit is contained in:
Alastor Wu
2019-03-05 19:48:31 +00:00
parent fe7579fb18
commit 7c3833ba6a
1 changed files with 1 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
dom/html/TextTrackManager.cpp
View File

@@ -295,10 +295,7 @@ void TextTrackManager::NotifyCueRemoved(TextTrackCue& aCue) {
mNewCues->RemoveCue(aCue);
}
TimeMarchesOn();
if (aCue.GetActive()) {
// We remove an active cue, need to update the display.
DispatchUpdateCueDisplay();
}
DispatchUpdateCueDisplay();
}
void TextTrackManager::PopulatePendingList() {