Bug 1535005 - part1 : no need to update 'mLastTimeMarchesOnCalled' in 'DidSeek()' r=jya

According to spec [1],  `mLastTimeMarchesOnCalled` is used to represent the `last time` in step3. It's used to record last time we run `TimeMarchOn()`, so there is no need to upate it on `DidSeek()`.

[1] https://html.spec.whatwg.org/multipage/media.html#time-marches-on

Differential Revision: https://phabricator.services.mozilla.com/D29094

--HG--
extra : moz-landing-system : lando
This commit is contained in:
Alastor Wu 2019-05-01 23:16:42 +00:00
Родитель 703b321635
Коммит b8c30ffe3c
1 изменённых файлов: 0 добавлений и 6 удалений

Просмотреть файл

@ -232,12 +232,6 @@ void TextTrackManager::RemoveTextTrack(TextTrack* aTextTrack,
void TextTrackManager::DidSeek() { void TextTrackManager::DidSeek() {
WEBVTT_LOG("DidSeek"); WEBVTT_LOG("DidSeek");
if (mMediaElement) {
mLastTimeMarchesOnCalled =
media::TimeUnit::FromSeconds(mMediaElement->CurrentTime());
WEBVTT_LOGV("DidSeek set mLastTimeMarchesOnCalled %lf",
mLastTimeMarchesOnCalled.ToSeconds());
}
mHasSeeked = true; mHasSeeked = true;
} }