Karl Tomlinson (:karlt) Assignee
	Description • 10 years ago

Assertion failure: mUpdating, at content/media/mediasource/SourceBuffer.cpp:566
in StopUpdating running web-platform/tests/media-source/SourceBuffer-abort-updating.html for example.

abort() sets updating to false synchronously, per the spec, but the buffer
append algorithm is not aborted, and so update is dispatched and updateend
again.

	Karl Tomlinson (:karlt) Assignee
	Comment 1 • 10 years ago

Logically, one would cancel the runnable on abort(), but it doesn't matter
exactly when the "update" event is queued, as long as it is async, and so
merely testing mUpdating gives acceptable results with less code.

With this change, the running web platform tests give the same results with
XPCOM_DEBUG_BREAK=abort debug and opt builds.  i.e. no assertion failures.
But only opt tests are currently running on tinderboxen.

	Karl Tomlinson (:karlt) Assignee
	Comment 2 • 10 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/c9bd08318422

	Carsten Book [:Tomcat]
	Comment 3 • 10 years ago

https://hg.mozilla.org/mozilla-central/rev/c9bd08318422