Remove MediaResource::IsTransportSeekable()

RESOLVED FIXED in Firefox 57

Status

()

Core
Audio/Video: Playback
P3
normal
RESOLVED FIXED
6 months ago
6 months ago

People

(Reporter: jwwang, Assigned: jwwang)

Tracking

unspecified
mozilla57
Points:
---

Firefox Tracking Flags

(firefox57 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(2 attachments)

Comment hidden (empty)
(Assignee)

Updated

6 months ago
Assignee: nobody → jwwang
Blocks: 1373160
Priority: -- → P3
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
(Assignee)

Updated

6 months ago
Attachment #8900999 - Flags: review?(gsquelart)
Attachment #8901000 - Flags: review?(gsquelart)

Comment 3

6 months ago
mozreview-review
Comment on attachment 8900999 [details]
Bug 1393346. P1 - virtualize IsTransportSeekable().

https://reviewboard.mozilla.org/r/172462/#review177712
Attachment #8900999 - Flags: review?(gsquelart) → review+

Comment 4

6 months ago
mozreview-review
Comment on attachment 8901000 [details]
Bug 1393346. P2 - move IsTransportSeekable() from MediaResource to BaseMediaResource.

https://reviewboard.mozilla.org/r/172464/#review177714
Attachment #8901000 - Flags: review?(gsquelart) → review+
(Assignee)

Comment 5

6 months ago
Thanks for the review!

Comment 6

6 months ago
hg error in cmd: hg update --clean -r dac8634cb381: abort: unknown revision 'dac8634cb381'!

Comment 7

6 months ago
Pushed by jwwang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/47258c1620dc
P1 - virtualize IsTransportSeekable(). r=gerald
https://hg.mozilla.org/integration/autoland/rev/4c5be1a3ee2c
P2 - move IsTransportSeekable() from MediaResource to BaseMediaResource. r=gerald

Comment 8

6 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/47258c1620dc
https://hg.mozilla.org/mozilla-central/rev/4c5be1a3ee2c
Status: NEW → RESOLVED
Last Resolved: 6 months ago
status-firefox57: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
You need to log in before you can comment on or make changes to this bug.