Closed
Bug 1685215
Opened 5 years ago
Closed 5 years ago
Move `AdjustForStartTime()` out from `MediaData`
Categories
(Core :: Audio/Video: Playback, task, P3)
Core
Audio/Video: Playback
Tracking
()
RESOLVED
FIXED
86 Branch
| Tracking | Status | |
|---|---|---|
| firefox86 | --- | fixed |
People
(Reporter: alwu, Assigned: alwu)
Details
Attachments
(1 file)
AdjustForStartTime() is only used for decoded data type in order to adjust their start time to zero, such as VideoData and AudioData.
However, some other types inherited from MediaData don't need that function. Eg. MediaRawData, so it's not necessary to put it on the MediaData, we should move that function to VideoData and AudioData separately.
|
Assignee | |
Comment 1•5 years ago
|
||
AdjustForStartTime() is only used for decoded data type in order to adjust their start time to zero, such as VideoData and AudioData.
However, some other types inherited from MediaData don't need that function. Eg. MediaRawData, so it's not necessary to put it on the MediaData, we should move that function to VideoData and AudioData separately.
Pushed by alwu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c863241f7a13
'AdjustForStartTime()' should only exist on decoded data type. r=padenot
|
||
Comment 3•5 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox86:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 86 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•