Closed Bug 1223599 Opened 9 years ago Closed 9 years ago

Remove the throttling argument from AbstractMediaDecoder::NotifyDataArrived()

Categories

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

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla45
Tracking Status
firefox45 --- fixed

People

(Reporter: jwwang, Assigned: jwwang)

References

Details

Attachments

(1 file)

Assignee: nobody → jwwang
Blocks: 1219142
Depends on: 1220558
Per bug 1220558 comment 5, the various NotifyDataArrivedInternal implementation now only ever parse new data . So throttling is unnecessary.
Bug 1223599 - Remove the throttling argument from AbstractMediaDecoder::NotifyDataArrived(). r=jya.
Attachment #8686001 - Flags: review?(jyavenard)
Attachment #8686001 - Flags: review?(jyavenard) → review+
Comment on attachment 8686001 [details]
MozReview Request: Bug 1223599 - Remove the throttling argument from AbstractMediaDecoder::NotifyDataArrived(). r=jya.

https://reviewboard.mozilla.org/r/24935/#review22533
Thanks!
https://hg.mozilla.org/mozilla-central/rev/7fbfb74b3dd3
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
Depends on: 1224947
No longer depends on: 1224947
Depends on: 1224947
Depends on: 1227299
See Also: → 1227299
Blocks: 1228187
Depends on: 1256038
Depends on: 1283724
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: