Closed
Bug 1074004
Opened 7 years ago
Closed 7 years ago
Update nestegg from upstream
Categories
(Core :: Audio/Video, defect)
Core
Audio/Video
Tracking
()
RESOLVED
FIXED
mozilla35
People
(Reporter: kinetik, Assigned: kinetik)
References
Details
Attachments
(2 files)
19.52 KB,
patch
|
rillian
:
review+
|
Details | Diff | Splinter Review |
1.45 KB,
patch
|
rillian
:
review+
|
Details | Diff | Splinter Review |
Import latest libnestegg from upstream to sync the sources. Only significant change is libnestegg now does internal buffering to reduce read overhead caused by its byte-at-a-time parser.
Assignee | ||
Comment 1•7 years ago
|
||
Attachment #8496628 -
Flags: review?(giles)
Assignee | ||
Comment 2•7 years ago
|
||
The internal buffering changes revealed a couple of bugs in TestWebMWriters WebMioData implementation: - Seek should return 0 on success - Reading past the end of stream should return EOS rather than an error
Attachment #8496629 -
Flags: review?(giles)
Assignee | ||
Comment 3•7 years ago
|
||
https://tbpl.mozilla.org/?tree=Try&rev=bcbd795fa2f7
Updated•7 years ago
|
Attachment #8496628 -
Flags: review?(giles) → review+
Updated•7 years ago
|
Attachment #8496629 -
Flags: review?(giles) → review+
Assignee | ||
Comment 4•7 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/571cef4dcf52 https://hg.mozilla.org/integration/mozilla-inbound/rev/2af5453da3ab
Assignee | ||
Comment 5•7 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/405b97368630
Comment 6•7 years ago
|
||
tests were still failing after the bustage fix so i had to backout this 3 changes for ongoing test failures like https://tbpl.mozilla.org/php/getParsedLog.php?id=49168555&tree=Mozilla-Inbound - sorry!
Assignee | ||
Comment 7•7 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/27bcbf49cfd7 https://hg.mozilla.org/integration/mozilla-inbound/rev/5997c0f393ba
Assignee | ||
Comment 8•7 years ago
|
||
Sorry, I tested the bustage fix locally but accidentally used a --disable-debug tree, so the assert didn't fire. Correct fix included in latest push.
Comment 9•7 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/27bcbf49cfd7 https://hg.mozilla.org/mozilla-central/rev/5997c0f393ba
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla35
You need to log in
before you can comment on or make changes to this bug.
Description
•