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+
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 36
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.