Merge lp:~justinmcp/oxide/mediahub-seekable-status-1.10 into lp:oxide/1.10

Proposed by Justin McPherson on 2015-10-23
Status: Needs review
Proposed branch: lp:~justinmcp/oxide/mediahub-seekable-status-1.10
Merge into: lp:oxide/1.10
Diff against target: 13 lines (+2/-1)
1 file modified
shared/renderer/media/oxide_web_media_player.cc (+2/-1)
To merge this branch: bzr merge lp:~justinmcp/oxide/mediahub-seekable-status-1.10
Reviewer Review Type Date Requested Status
Chris Coulson 2015-10-23 Approve on 2015-10-27
Review via email: mp+275497@code.launchpad.net

Commit message

Fix reporting of seekable status

To post a comment you must log in.
review: Approve

Unmerged revisions

1207. By Justin McPherson on 2015-10-23

Fix reporting of seekable status

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'shared/renderer/media/oxide_web_media_player.cc'
2--- shared/renderer/media/oxide_web_media_player.cc 2015-08-25 20:42:44 +0000
3+++ shared/renderer/media/oxide_web_media_player.cc 2015-10-23 07:55:35 +0000
4@@ -325,7 +325,8 @@
5 }
6
7 WebTimeRanges WebMediaPlayer::seekable() const {
8- return buffered_;
9+ const blink::WebTimeRange seekable_range(0.0, duration());
10+ return blink::WebTimeRanges(&seekable_range, 1);
11 }
12
13 double WebMediaPlayer::maxTimeSeekable() const {

Subscribers

People subscribed via source and target branches