Closed Bug 1212701 Opened 4 years ago Closed 4 years ago

Remove AbstractMediaDecoder::OnDecodeTaskQueue()

Categories

(Core :: Audio/Video: Playback, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla44
Tracking Status
firefox44 --- fixed

People

(Reporter: jwwang, Assigned: jwwang)

Details

Attachments

(2 files)

https://hg.mozilla.org/mozilla-central/file/a45d680f073f/dom/media/webm/WebMReader.cpp#l285

Since bug 1148234 is completed, we can assert OnTaskQueue() and remove OnDecodeTaskQueue() functions.
Assignee: nobody → jwwang
Bug 1212701. Part 1 - remove AbstractMediaDecoder::OnDecodeTaskQueue().
Attachment #8672435 - Flags: review?(jyavenard)
Bug 1212701. Part 2 - remove MediaDecoderStateMachine::OnDecodeTaskQueue() which is unused.
Attachment #8672436 - Flags: review?(jyavenard)
Attachment #8672435 - Flags: review?(jyavenard) → review+
Comment on attachment 8672435 [details]
MozReview Request: Bug 1212701. Part 1 - remove AbstractMediaDecoder::OnDecodeTaskQueue().

https://reviewboard.mozilla.org/r/21711/#review19507
Comment on attachment 8672436 [details]
MozReview Request: Bug 1212701. Part 2 - remove MediaDecoderStateMachine::OnDecodeTaskQueue() which is unused.

https://reviewboard.mozilla.org/r/21713/#review19509
Attachment #8672436 - Flags: review?(jyavenard) → review+
Thanks for the review!
https://hg.mozilla.org/mozilla-central/rev/2f532f8439be
https://hg.mozilla.org/mozilla-central/rev/7d1a08a0ba44
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
You need to log in before you can comment on or make changes to this bug.