Closed Bug 1209388 Opened 8 years ago Closed 8 years ago
Instead we should be using the new MP3Demuxer / MediaFormatReader
Following bug 1209410, it was no longer used.
Attachment #8667724 - Flags: review?(ajones)
Comment on attachment 8667724 [details] [diff] [review] [mp3] Remove AppleMP3Reader. Carrying r+ from bug 1209410. git bz got confused with the two bug numbers in the commit message and uploaded it in the wrong spot
Attachment #8667724 - Flags: review?(ajones) → review+
Time to land now?
Priority: -- → P2
Backed out in https://hg.mozilla.org/integration/mozilla-inbound/rev/ede421c34977 - https://treeherder.mozilla.org/logviewer.html#?job_id=17498962&repo=mozilla-inbound and https://treeherder.mozilla.org/logviewer.html#?job_id=17498970&repo=mozilla-inbound are of the opinion that you made us at least think we can't play audio, if not made us unable to play audio.
And ignore my commit message saying "on 10.6", while pulsebot was midairing me I was discovering that it did fail on 10.10, we just only manage to run tests on 10.10 once every three or four hours.
it is resolved, it's only the patch to remove the pref that got backed out (i incorrectly also removed the mp4 pref which is what caused the bustage)
Status: REOPENED → RESOLVED
Closed: 8 years ago → 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.