Closed
Bug 1045915
Opened 10 years ago
Closed 10 years ago
Add a pref to allow testing of streams with a missing 'ftyp' atom.
Categories
(Core :: Audio/Video, defect)
Tracking
()
RESOLVED
FIXED
mozilla34
People
(Reporter: ajones, Assigned: ajones)
References
Details
Attachments
(1 file)
(deleted),
patch
|
kinetik
:
review+
|
Details | Diff | Splinter Review |
Add a pref so streams that start with 'moov' instead of 'ftyp' can be tested. These streams relate to an older version of the spec.
Assignee | ||
Comment 1•10 years ago
|
||
Attachment #8464339 -
Flags: review?(kinetik)
Comment 2•10 years ago
|
||
Comment on attachment 8464339 [details] [diff] [review]
Add a pref to test streams with a missing ftyp atom
Review of attachment 8464339 [details] [diff] [review]:
-----------------------------------------------------------------
Don't add the pref to all.js, it's only for testing and should be a hidden pref. r+ with that part removed.
Attachment #8464339 -
Flags: review?(kinetik) → review+
Assignee | ||
Comment 3•10 years ago
|
||
Backed out in https://hg.mozilla.org/integration/mozilla-inbound/rev/b1fb58e03f0d along with bug 1045909's backout for build bustage: https://tbpl.mozilla.org/php/getParsedLog.php?id=45210935&tree=Mozilla-Inbound
Flags: needinfo?(ajones)
Assignee | ||
Comment 5•10 years ago
|
||
Flags: needinfo?(ajones)
Comment 6•10 years ago
|
||
This is still causing build failures: https://tbpl.mozilla.org/php/getParsedLog.php?id=45224827&tree=Mozilla-Inbound
Comment 7•10 years ago
|
||
Comment 8•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/9b728101e05f
https://hg.mozilla.org/mozilla-central/rev/c7654768ca3f
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
QA Whiteboard: [qa-]
You need to log in
before you can comment on or make changes to this bug.
Description
•