Closed Bug 1300479 Opened 5 years ago Closed 5 years ago

Small tweaks to MDSM functions

Categories

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

defect

Tracking

()

RESOLVED FIXED
mozilla51
Tracking Status
firefox51 --- fixed

People

(Reporter: jwwang, Assigned: jwwang)

References

Details

Attachments

(3 files)

Makes it easier to define state objects for each MDSM state.
Assignee: nobody → jwwang
Blocks: 1295892
Priority: -- → P3
Attachment #8788327 - Flags: review?(kaku)
Attachment #8788328 - Flags: review?(kaku)
Attachment #8788329 - Flags: review?(kaku)
Comment on attachment 8788327 [details]
Bug 1300479. Part 1 - remove the aState parameter from EnterState()/ExitState() for we always pass mState.

https://reviewboard.mozilla.org/r/76844/#review74972
Attachment #8788327 - Flags: review?(kaku) → review+
Comment on attachment 8788328 [details]
Bug 1300479. Part 2 - replace empty cases with the default case.

https://reviewboard.mozilla.org/r/76846/#review74974
Attachment #8788328 - Flags: review?(kaku) → review+
Comment on attachment 8788329 [details]
Bug 1300479. Part 3 - extract RunStateMachine() cases to its own function.

https://reviewboard.mozilla.org/r/76848/#review74976
Attachment #8788329 - Flags: review?(kaku) → review+
Thanks!
Pushed by jwwang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/80a96bf681f1
Part 1 - remove the aState parameter from EnterState()/ExitState() for we always pass mState. r=kaku
https://hg.mozilla.org/integration/autoland/rev/55a002b013f0
Part 2 - replace empty cases with the default case. r=kaku
https://hg.mozilla.org/integration/autoland/rev/21e90c727ae0
Part 3 - extract RunStateMachine() cases to its own function. r=kaku
You need to log in before you can comment on or make changes to this bug.