DecodedStream::NotifyOutput() should compare |a->mTime| to GetPosition() instead of |aTime|

RESOLVED FIXED in Firefox 52

Status

()

defect
P3
normal
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: jwwang, Assigned: jwwang)

Tracking

unspecified
mozilla52
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox52 fixed)

Details

Attachments

(1 attachment)

Assignee

Updated

3 years ago
Assignee: nobody → jwwang
Depends on: 1306813
Priority: -- → P3
Assignee

Updated

3 years ago
Depends on: 1305353
No longer depends on: 1306813
Assignee

Updated

3 years ago
Attachment #8808028 - Flags: review?(pehrson)

Comment 2

3 years ago
mozreview-review
Comment on attachment 8808028 [details]
Bug 1315586 - DecodedStream::NotifyOutput() should compare |a->mTime| to GetPosition() instead of |aTime|.

https://reviewboard.mozilla.org/r/90948/#review90736
Attachment #8808028 - Flags: review?(pehrson) → review+
Assignee

Comment 3

3 years ago
Thanks!

Comment 4

3 years ago
Pushed by jwwang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/09c6816c1341
DecodedStream::NotifyOutput() should compare |a->mTime| to GetPosition() instead of |aTime|. r=pehrsons

Comment 5

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/09c6816c1341
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
You need to log in before you can comment on or make changes to this bug.