Sync the documentation of seekable() with the MediaSource specification

Sync the documentation of seekable() with the MediaSource specification

https://www.w3.org/Bugs/Public/show_bug.cgi?id=27296

Also tweak an assert to match the preceding comment.

BUG=432051
R=henriks@opera.com

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

[philipj_slow, 2014-11-14 12:57:37]

PTAL. There was a misunderstanding with the assert_not_equals, this is what I
had in mind.

[Henrik Steen, 2014-11-21 07:46:12]

https://codereview.chromium.org/733453002/diff/1/LayoutTests/http/tests/media...
File LayoutTests/http/tests/media/media-source/mediasource-seekable.html
(right):

https://codereview.chromium.org/733453002/diff/1/LayoutTests/http/tests/media...
LayoutTests/http/tests/media/media-source/mediasource-seekable.html:61:
assert_not_equals(mediaElement.seekable.end(0), mediaElement.buffered.start(0));
Hmm..  I would rather (or maybe "also") see a test to assert that
seekable.start(0) isn't equal to buffered.start(0).  This was basically what we
asserted before by saying that seekable.start(0) = 0 != buffered.start(0).

[philipj_slow, 2014-11-21 08:33:23]

one more assert

[philipj_slow, 2014-11-21 08:33:38]

https://codereview.chromium.org/733453002/diff/1/LayoutTests/http/tests/media...
File LayoutTests/http/tests/media/media-source/mediasource-seekable.html
(right):

https://codereview.chromium.org/733453002/diff/1/LayoutTests/http/tests/media...
LayoutTests/http/tests/media/media-source/mediasource-seekable.html:61:
assert_not_equals(mediaElement.seekable.end(0), mediaElement.buffered.start(0));
On 2014/11/21 07:46:12, Henrik Steen wrote:
> Hmm..  I would rather (or maybe "also") see a test to assert that
> seekable.start(0) isn't equal to buffered.start(0).  This was basically what
we
> asserted before by saying that seekable.start(0) = 0 != buffered.start(0).

Done.

[Henrik Steen, 2014-11-21 11:49:08]

lgtm

[philipj_slow, 2014-11-21 11:49:39]

The CQ bit was checked by philipj@opera.com

[commit-bot: I haz the power, 2014-11-21 11:50:47]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/733453002/20001

[commit-bot: I haz the power, 2014-11-21 11:50:49]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-11-21 11:50:50]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[philipj_slow, 2014-11-21 14:48:50]

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

[philipj_slow, 2014-11-21 14:48:50]

fs, please rubberstamp :)

[fs, 2014-11-21 15:02:13]

The CQ bit was checked by fs@opera.com

[fs, 2014-11-21 15:02:14]

lgtm

[commit-bot: I haz the power, 2014-11-21 15:02:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/733453002/20001

[commit-bot: I haz the power, 2014-11-21 15:05:18]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=185799