Jean-Yves Avenard [:jya] Assignee
	Description • 6 years ago

The test http://w3c-test.org/media-source/mediasource-changetype-play.html fails.

Upon investigation the issue is due to failing this step in the change duration algorithm

https://w3c.github.io/media-source/index.html#duration-change-algorithm
"If new duration is less than the highest presentation timestamp of any buffered coded frames for all SourceBuffer objects in sourceBuffers, then throw an InvalidStateError exception and abort these steps.
"

the issue here is that following the call to the Range Removal Algorithm, the highest presentation timestamp value isn't updated as it should.

Causing later failure.

	Jean-Yves Avenard [:jya] Assignee
	Comment 1 • 6 years ago

When removing frames from the trackbuffer we may remove frames outside the original removal interval as we must remove all frames depending on the removed frames.

	Jean-Yves Avenard [:jya] Assignee
	Comment 2 • 6 years ago

Also, enable changeType MSE tests.

Depends on D2837

	Phabricator Automation
	Comment 3 • 6 years ago

Comment on attachment 8998270 [details]
Bug 1481397 - P2. Update test expectations. r=jgraham

James Graham [:jgraham] has approved the revision.

https://phabricator.services.mozilla.com/D2870

	Phabricator Automation
	Comment 4 • 6 years ago

Comment on attachment 8998137 [details]
Bug 1481397 - Properly recalculate highest start timestamp when samples are removed. r?bryce

Bryce Seager van Dyk (:bryce) has approved the revision.

https://phabricator.services.mozilla.com/D2837

	Pulsebot
	Comment 5 • 6 years ago

Pushed by jyavenard@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/edf9688dc9ac
P1. Properly recalculate highest start timestamp when samples are removed. r=bryce
https://hg.mozilla.org/integration/mozilla-inbound/rev/4cf920a66ad4
P2. Update test expectations. r=jgraham

	Natalia Csoregi [:nataliaCs]
	Comment 6 • 6 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/edf9688dc9ac
https://hg.mozilla.org/mozilla-central/rev/4cf920a66ad4

	Liz Henry (:lizzard Please n-i to RyanVM, jcristau, or pascal)
	Comment 7 • 6 years ago

jya or nils, is this something you might want to uplift to beta 62?

	Nils Ohlmeier [:drno]
	Comment 8 • 6 years ago

This only appears to affect the change-type API, which I think is not turned on by default in 62. So I would say there isn't a need to uplift. But I'll let the expert jya speak to that.

	Jean-Yves Avenard [:jya] Assignee
	Comment 9 • 6 years ago

The bug was found in the changeType wpt, however the problem is unrelated to changeType and will affect all Firefox version: changing the duration of the element is not possible when partial data has been added under some circumstances. I don’t know how common that issue would be, but it would cause either a playback error or the inability for a video to ever reach its end, so the ended event won’t ever fire .

Uplifting this would be nice, but it’s likely too late.

	Jean-Yves Avenard [:jya] Assignee
	Comment 10 • 6 years ago

Comment on attachment 8998137 [details]
Bug 1481397 - Properly recalculate highest start timestamp when samples are removed. r?bryce

Approval Request Comment
[Feature/Bug causing the regression]: MSE
[User impact if declined]: error when setting the duration of a video element, or playback will stall as it can’t reach the end
[Is this code covered by automated tests?]: not exactly for this bug, but some wpt were failing because of this.
[Has the fix been verified in Nightly?]: yes
[Needs manual test from QE? If yes, steps to reproduce]: can craft a JS to present the problem.
[List of other uplifts needed for the feature/fix]: just p1
[Is the change risky?]: no
[Why is the change risky/not risky?]: it’s not working now and the change are very localised to the duration change algorithm. So it can only get better
[String changes made/needed]: none

	Ryan VanderMeulen [:RyanVM]
	Comment 11 • 6 years ago

Comment on attachment 8998137 [details]
Bug 1481397 - Properly recalculate highest start timestamp when samples are removed. r?bryce

Fixes a potential stall in MSE playback. Covered by web-platform-tests. Approved for 62.0b20.

	Ryan VanderMeulen [:RyanVM]
	Comment 12 • 6 years ago
bugherder uplift

https://hg.mozilla.org/releases/mozilla-beta/rev/809e1c8c76ab