Closed Bug 1285122 Opened 3 years ago Closed 3 years ago

Some SeekTask code cleanup

Categories

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

defect

Tracking

()

RESOLVED FIXED
mozilla50
Tracking Status
firefox50 --- fixed

People

(Reporter: jwwang, Assigned: jwwang)

References

Details

Attachments

(2 files)

No description provided.
Assignee: nobody → jwwang
Blocks: 1283751
Priority: -- → P3
Comment on attachment 8772261 [details]
Bug 1285122. Part 1 - remove SeekTask::mNeedToStopPrerolling{Audio,Video} per discussion in https://reviewboard.mozilla.org/r/43689/#comment54421.

https://reviewboard.mozilla.org/r/65128/#review62152
Attachment #8772261 - Flags: review?(kaku) → review+
Attachment #8772262 - Flags: review?(kaku) → review+
Comment on attachment 8772262 [details]
Bug 1285122. Part 2 - fix includes and forward-declaration.

https://reviewboard.mozilla.org/r/65130/#review62154
Thanks for the review!
Pushed by jwwang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/db59de0a5080
Part 1 - remove SeekTask::mNeedToStopPrerolling{Audio,Video} per discussion in https://reviewboard.mozilla.org/r/43689/#comment54421. r=kaku
https://hg.mozilla.org/integration/autoland/rev/7f49a2ed5991
Part 2 - fix includes and forward-declaration. r=kaku
https://hg.mozilla.org/mozilla-central/rev/db59de0a5080
https://hg.mozilla.org/mozilla-central/rev/7f49a2ed5991
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
You need to log in before you can comment on or make changes to this bug.