Introduce WebMediaPlayer::seekable() to replace maxTimeSeekable()

Introduce WebMediaPlayer::seekable() to replace maxTimeSeekable()

TEST=LayoutTests/media/video-seekable.html

Update the test to check that seekable end time is between 5 and 7
seconds, as content/test.* are around 6 seconds long. This should
catch grossly incorrect implementations of seekable.

Also wait only for loadedmetadata, as that should be enough to
determine the seekable ranges, and an implementation waiting until
canplaythrough should fail the test.

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

[philipj_slow, 2014-09-25 13:10:06]

philipj@opera.com changed reviewers:
+ dalecurtis@chromium.org, fs@opera.com, scherkus@chromium.org

[philipj_slow, 2014-09-25 13:10:07]

PTAL. Let's keep discussion in this CL and I'll send out the two remaining CLs
when appropriate:
https://codereview.chromium.org/599103003/
https://codereview.chromium.org/604823002/

[DaleCurtis, 2014-09-25 17:36:00]

+100, this will make exposing better seekable() ranges in the future easier.

[DaleCurtis, 2014-09-25 17:36:05]

lgtm

[scherkus (not reviewing), 2014-09-26 01:19:27]

lgtm!

https://codereview.chromium.org/607493002/diff/1/public/platform/WebMediaPlay...
File public/platform/WebMediaPlayer.h (right):

https://codereview.chromium.org/607493002/diff/1/public/platform/WebMediaPlay...
public/platform/WebMediaPlayer.h:117: virtual WebTimeRanges seekable() const
I assume we'll be ripping this out at some point?

[philipj_slow, 2014-09-26 07:39:26]

It looks like we have agreement, so I'll send out the other two CLs as well.

https://codereview.chromium.org/607493002/diff/1/public/platform/WebMediaPlay...
File public/platform/WebMediaPlayer.h (right):

https://codereview.chromium.org/607493002/diff/1/public/platform/WebMediaPlay...
public/platform/WebMediaPlayer.h:117: virtual WebTimeRanges seekable() const
On 2014/09/26 01:19:27, scherkus wrote:
> I assume we'll be ripping this out at some point?

Yep, that's https://codereview.chromium.org/604823002/

[philipj_slow, 2014-09-26 07:41:42]

philipj@opera.com changed reviewers:
+ pfeldman@chromium.org

[philipj_slow, 2014-09-26 07:41:43]

pfeldman@, can you please review public/platform?

[pfeldman, 2014-09-29 09:03:07]

lgtm

[philipj_slow, 2014-09-29 09:18:10]

The CQ bit was checked by philipj@opera.com

[commit-bot: I haz the power, 2014-09-29 09:20:27]

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

[commit-bot: I haz the power, 2014-09-29 09:42:48]

Committed patchset #1 (id:1) as 182821