Update libvorbis to pick up fix for some files encoded with libvorbis beta1

RESOLVED FIXED

Status

()

RESOLVED FIXED
9 years ago
9 years ago

People

(Reporter: kinetik, Assigned: kinetik)

Tracking

({verified1.9.1})

Trunk
x86
Mac OS X
verified1.9.1
Points:
---
Bug Flags:
wanted1.9.2 +

Firefox Tracking Flags

(status1.9.2 beta1-fixed, status1.9.1 .4-fixed)

Details

(URL)

Attachments

(2 obsolete attachments)

(Assignee)

Description

9 years ago
SVN r16552 includes a fix to libvorbis that prevents the decoder from rejecting some valid files produced with the beta4 encoder.  These may not be particularly common files, but we should pick this simple fix up anyway.
Flags: wanted1.9.2?
(Assignee)

Comment 1

9 years ago
Created attachment 402938 [details] [diff] [review]
patch v0

Update libvorbis to SVN r16552.

This needs a test, but I don't have a file which is fixed by this patch handy.
(Assignee)

Comment 2

9 years ago
It's actually the beta1 encoder.  Test file in URL.  Unfortunately it doesn't play on trunk *or* with the patch applied.  Haven't investigated why yet.
Summary: Update libvorbis to pick up fix for some files encoded with libvorbis beta4 → Update libvorbis to pick up fix for some files encoded with libvorbis beta1
Flags: wanted1.9.2? → wanted1.9.2+
(Assignee)

Comment 3

9 years ago
Created attachment 404185 [details] [diff] [review]
patch v1

Same, but with a test.  This also fixes the comments above gPlayTests, which should've been above gErrorTests (I messed them up in bug 481057).

Included test file does not play on current trunk, but does play with patch applied (comment 2 is out of date--suspect that was fixed by oggplay updates).  Note that the test does not fail every time when the patch is not applied, but I can't work out why.
Attachment #402938 - Attachment is obsolete: true
(Assignee)

Comment 4

9 years ago
We took the broken libvorbis update on 1.9.1 for 1.9.1.4, so we should take this fix to ensure we don't regress anything.

Also see bug 507167 comment 16.
status1.9.1: --- → ?
(Assignee)

Updated

9 years ago
Attachment #404185 - Attachment is patch: true
Attachment #404185 - Attachment mime type: application/octet-stream → text/plain
(Assignee)

Comment 5

9 years ago
Posted the libvorbis update to bug 507167.  Includes the test from this bug.  Marking this as depending on that.
Depends on: 507167
(Assignee)

Comment 6

9 years ago
http://hg.mozilla.org/mozilla-central/rev/196956e36ed2
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
(Assignee)

Updated

9 years ago
Attachment #404185 - Attachment is obsolete: true
Fixed on 1.9.2 by checkin of bug 507167.
status1.9.2: --- → beta1-fixed
(Assignee)

Comment 8

9 years ago
Checked in to 1.9.1 too.
status1.9.1: ? → .4-fixed
Do we have sample rejected files somewhere that can be used to verify this fix for Firefox 3.5.4?
(Assignee)

Comment 10

9 years ago
There's one in the bug URL.  I don't have any others to test with.
Er, I missed the url. Sorry about that.

I verified that it doesn't play in 1.9.1.3 on OS X but does play on 1.9.1.4 (Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.6; en-US; rv:1.9.1.4) Gecko/20091016 Firefox/3.5.4) so this looks good.
Keywords: verified1.9.1
You need to log in before you can comment on or make changes to this bug.