The default bug view has changed. See this FAQ.

Change |seeking| to prevent seeking in WebM livestream.

RESOLVED FIXED in mozilla15

Status

()

Core
Audio/Video
RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: padenot, Assigned: padenot)

Tracking

unspecified
mozilla15
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

5 years ago
Bug 657791 prevent us from seeking in webm files that have no Cues.

Despite that limitation, |seekable| returns |buffered| while playing a livestream, allowing the user to seek in the stream via the front-end, and displaying an infinite spinning throbber.

We should return an empty TimeRanges object for |Seekable| when playing a live stream, or playing a bounded media that is not served with range requests.
(Assignee)

Comment 1

5 years ago
Created attachment 625011 [details] [diff] [review]
Patch v0 : change seekable behavior for WebM.

This patch adds an virtual method in the ns*Readers to report if that backend is capable of seeking using only buffered ranges.

This method is then called in |nsBuiltinDecoder::GetSeekable| to get the behavior wanted.
Assignee: nobody → paul
Status: NEW → ASSIGNED
Attachment #625011 - Flags: review?(kinetik)
Comment on attachment 625011 [details] [diff] [review]
Patch v0 : change seekable behavior for WebM.

-nsresult nsRawReader::Seek(PRInt64 aTime, PRInt64 aStartTime, PRInt64 aEndTime, PRInt64 aCurrentTime)
+nsresult nsRawReader::Seek(PRInt65 aTime, PRInt64 aStartTime, PRInt64 aEndTime, PRInt64 aCurrentTime)

This looks like an accidental change.  r+ with that removed.
Attachment #625011 - Flags: review?(kinetik) → review+
(Assignee)

Comment 3

5 years ago
Created attachment 625151 [details] [diff] [review]
Addressed nit.
Attachment #625011 - Attachment is obsolete: true
(Assignee)

Updated

5 years ago
Keywords: checkin-needed
https://hg.mozilla.org/integration/mozilla-inbound/rev/1e18c991b40c

Is this something that should have a test?
Flags: in-testsuite?
Keywords: checkin-needed
Target Milestone: --- → mozilla15
(Assignee)

Comment 5

5 years ago
This is likely to be a more or less temporary patch waiting for Bug 657791. Does not worth a test, I believe.
Target Milestone: mozilla15 → ---
(Assignee)

Updated

5 years ago
Target Milestone: --- → mozilla15
Flags: in-testsuite? → in-testsuite-
https://hg.mozilla.org/mozilla-central/rev/1e18c991b40c
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED

Updated

5 years ago
Depends on: 774084

Updated

5 years ago
Depends on: 795784

Updated

4 years ago
Depends on: 817124
You need to log in before you can comment on or make changes to this bug.