Closed Bug 1307022 Opened 5 years ago Closed 5 years ago

Move some code from MDSM::OnAudioDecoded/OnVideoDecoded/OnNotDecoded to state objects

Categories

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

defect

Tracking

()

RESOLVED FIXED
mozilla52
Tracking Status
firefox52 --- fixed

People

(Reporter: jwwang, Assigned: jwwang)

References

Details

Attachments

(5 files)

No description provided.
Assignee: nobody → jwwang
Blocks: 1295892
Priority: -- → P3
Attachment #8797370 - Flags: review?(kaku)
Attachment #8797371 - Flags: review?(kaku)
Attachment #8797372 - Flags: review?(kaku)
Attachment #8797373 - Flags: review?(kaku)
Attachment #8797374 - Flags: review?(kaku)
Comment on attachment 8797370 [details]
Bug 1307022. Part 1 - Use raw pointers to avoid unnecessary ref-counting.

https://reviewboard.mozilla.org/r/82962/#review81694
Attachment #8797370 - Flags: review?(kaku) → review+
Comment on attachment 8797371 [details]
Bug 1307022. Part 2 - move some code from OnAudioDecoded() to various state objects.

https://reviewboard.mozilla.org/r/82964/#review81696
Attachment #8797371 - Flags: review?(kaku) → review+
Comment on attachment 8797372 [details]
Bug 1307022. Part 3 - Use raw pointers to avoid unnecessary ref-counting.

https://reviewboard.mozilla.org/r/82966/#review81698
Attachment #8797372 - Flags: review?(kaku) → review+
Comment on attachment 8797373 [details]
Bug 1307022. Part 4 - move some code from OnVideoDecoded() to various state objects.

https://reviewboard.mozilla.org/r/82968/#review81700
Attachment #8797373 - Flags: review?(kaku) → review+
Comment on attachment 8797374 [details]
Bug 1307022. Part 5 - move some code from OnNotDecoded() to various state objects.

https://reviewboard.mozilla.org/r/82970/#review81702
Attachment #8797374 - Flags: review?(kaku) → review+
Thanks!
Pushed by jwwang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2a1edf61eb2b
Part 1 - Use raw pointers to avoid unnecessary ref-counting. r=kaku
https://hg.mozilla.org/integration/autoland/rev/8b54936366ea
Part 2 - move some code from OnAudioDecoded() to various state objects. r=kaku
https://hg.mozilla.org/integration/autoland/rev/4e654ea0dfa6
Part 3 - Use raw pointers to avoid unnecessary ref-counting. r=kaku
https://hg.mozilla.org/integration/autoland/rev/e145b5d95343
Part 4 - move some code from OnVideoDecoded() to various state objects. r=kaku
https://hg.mozilla.org/integration/autoland/rev/1ebe25b02ffe
Part 5 - move some code from OnNotDecoded() to various state objects. r=kaku
You need to log in before you can comment on or make changes to this bug.