[media-source] Editorial? Prepare append algorithm's caller needs to abort if the algorithm aborted (#288)

wolenetz has just created a new issue for https://github.com/w3c/media-source:

== Editorial? Prepare append algorithm's caller needs to abort if the algorithm aborted ==
Currently, the last step in the "Prepare Append" algorithm is (bold for emphasis in this copy of it):

> If the buffer full flag equals true, then throw a QuotaExceededError exception **and abort these steps.**

However, the caller (SourceBuffer's appendBuffer() method) does not acknowledge that the Prepare Append algorithm might have thrown and aborted.

If the Prepare Append algorithm had aborted, the remainder of the steps in appendBuffer() should **not* be run. e.g., |data| should not be appended to the input buffer, updating shouldn't become true, updatestart event shouldn't be enqueued, and no asynchronous run of the buffer append algorithm should happen.

I believe this is likely editorial, but will see if I can find behavior to the contrary in either Mozilla or Safari MSE implementations (assuming I can get them to decide that an append is too much and give QuotaExceededError).

If truly editorial, the next step after "Run the prepare append algorithm" in appendBuffer() needs to abort if the prepare append algorithm aborted. 

Please view or discuss this issue at https://github.com/w3c/media-source/issues/288 using your GitHub account


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config

Received on Wednesday, 4 August 2021 01:32:41 UTC