Closed Bug 481057 Opened 13 years ago Closed 12 years ago
_ERR _DECODE machinery
We need to implement the part of the spec that covers the "if the media data is corrupted" case. This was implemented (partially?) as part of another patch, but this part never made it into the tree for reasons I can't remember now.
Bug 449159 comment 47 and bug 449159 comment 51 (which I can't find a bug for, so I guess might have just gotten lost). The Wave decoder already tries to fire an error for this case, but it's stubbed out in nsWaveDecoder::MediaErrorDecode.
Initial patch. Still need to make the Ogg decoder fire decode errors. This patch also renames MEDIA_ERR_NONE_SUPPORTED to MEDIA_ERR_SRC_NOT_SUPPORT per the spec.
Assignee: nobody → kinetik
Status: NEW → ASSIGNED
Er, that'd be MEDIA_ERR_SRC_NOT_SUPPORTED.
Flags: wanted1.9.2? → wanted1.9.2+
Rebased against trunk, uses new test infrastructure, adds tests for Ogg decoder and constants. Needs superview because it modifies web-exposed IDL.
Comment on attachment 399149 [details] [diff] [review] patch v1 Looks good. The only issue is whether we can make the test ogg file smaller.
Attachment #399149 - Flags: superreview?(roc) → superreview+
Same, but with the FLAC stripped out of the test file (saves ~200kB) and renamed from dirac_flac.ogg to dirac.ogg, some minor changes to the test, and slightly improved (but still not great) comments on gErrorTests and gDecodeErrorTests in manifest.js.
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Attachment #399153 - Flags: approval1.9.2?
Attachment #399153 - Flags: approval1.9.2? → approval1.9.2+
You need to log in before you can comment on or make changes to this bug.