Fix up AndroidDecoderModule error handling and logging

RESOLVED FIXED in Firefox 36

Status

()

Firefox for Android
Audio/Video
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: snorp, Assigned: snorp)

Tracking

unspecified
Firefox 36
All
Android
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

There is a bunch of printf_stderr() and duplicated boiler plate for handling the nsresult. It should get cleaned up.
Created attachment 8523076 [details] [diff] [review]
Clean up logging and error handling in AndroidDecoderModule
Attachment #8523076 - Flags: review?(cpearce)
Attachment #8523076 - Flags: review?(cpearce) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/b550a7c475fc
https://hg.mozilla.org/mozilla-central/rev/b550a7c475fc
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 36

Comment 4

3 years ago
Can QA pick up this bug for testing?will someone update test steps
This is a cleanup bug and not expected that QA would verify.
You need to log in before you can comment on or make changes to this bug.