LayoutTests for the new MSE GC behavior.

LayoutTests for the new MSE GC behavior.

These unit tests verify that QuotaExceededErr is thrown when appending
data to an MSE SourceBuffer that is full. This new behavior was
ntroduced by the recent CLs:
https://codereview.chromium.org/1008463002/
https://codereview.chromium.org/1013923002/

BUG=421694
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=201032

[philipj_slow, 2015-08-21 08:21:54]

philipj@opera.com changed reviewers:
+ philipj@opera.com

[philipj_slow, 2015-08-21 08:21:54]

I gues BUG=421694?

[philipj_slow, 2015-08-21 08:25:22]

LGTM, I just checked that these are the same as in
https://codereview.chromium.org/1013923002/#ps260001

[wolenetz, 2015-08-21 20:11:14]

wolenetz@chromium.org changed reviewers:
+ wolenetz@chromium.org

[wolenetz, 2015-08-21 20:11:15]

lgtm % please update CL description to:
Refer to the previous chain of Blink and Chromium CL prerequisites, and to refer
to the bug number.

https://codereview.chromium.org/1304953002/diff/1/LayoutTests/http/tests/medi...
File
LayoutTests/http/tests/media/media-source/mediasource-appendbuffer-quota-exceeded.html
(right):

https://codereview.chromium.org/1304953002/diff/1/LayoutTests/http/tests/medi...
LayoutTests/http/tests/media/media-source/mediasource-appendbuffer-quota-exceeded.html:42:
}, 'Appending data repeatedly should fill up the buffer and throw a
QuotaExceededError when buffer is full.');
ditto my (aside) comment in the appendStream eviction layout test

https://codereview.chromium.org/1304953002/diff/1/LayoutTests/http/tests/medi...
File
LayoutTests/http/tests/media/media-source/mediasource-appendstream-quota-exceeded.html
(right):

https://codereview.chromium.org/1304953002/diff/1/LayoutTests/http/tests/medi...
LayoutTests/http/tests/media/media-source/mediasource-appendstream-quota-exceeded.html:49:
}, 'Calling appendStream repeatedly should fill up the buffer and throw a
QuotaExceededError when buffer is full.');
aside: Looks good for the Chromium implementation.
However, if used in cross-platform web-platform-tests, this test assumes
interoperable implementations will not evict from the current buffered range
beyond the current playback position, right? This may not be usable for spec
compliance testing of other implementations, since it makes assumption about
which (or if any) removal ranges are chosen during prepare append algorithma and
during each iteration of the stream append loop.

[wolenetz, 2015-08-22 00:24:20]

Currently pending servolk@'s update from chat, that will add another
appendStream test that looks for SourceBuffer error event (from append stream
loop), and doesn't expect quotaexceeded exception, because it will not specify
maxsize on the append. The existing appendStream test needs to verify also that
no SourceBuffer error event occurs (since maxSize should have already been
evicted -- or quota exceeded err thrown).

[servolk, 2015-08-22 00:50:43]

The CQ bit was checked by servolk@chromium.org

[servolk, 2015-08-22 00:50:44]

The patchset sent to the CQ was uploaded after l-g-t-m from philipj@opera.com,
wolenetz@chromium.org
Link to the patchset: https://codereview.chromium.org/1304953002/#ps40001
(title: "New test case for error event sent from async appendStream")

[commit-bot: I haz the power, 2015-08-22 00:50:55]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1304953002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1304953002/40001

[commit-bot: I haz the power, 2015-08-22 01:36:32]

Committed patchset #3 (id:40001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=201032