Closed Bug 800235 Opened 9 years ago Closed 9 years ago
Uploading and playing ogg videos doesn't work
843 bytes, patch
|Details | Diff | Splinter Review|
360 bytes, text/html
I downloaded: http://en.wikipedia.org/wiki/File:Big_Buck_Bunny_Trailer_400p.ogg and pushed it to the device. When I start the music app I see: E/GeckoConsole( 668): Content JS WARN at app://music.gaiamobile.org/shared/js/mediadb.js:1203 in metadataError: MediaDB: error parsing metadata for Movies/123.ogg : malformed ogg comment packet E/GeckoConsole( 668): Content JS WARN at app://music.gaiamobile.org/shared/js/mediadb.js:1203 in metadataError: MediaDB: error parsing metadata for Movies/123.ogg : Error: advance past end of buffer I verified that I can play the file again if I download it from the card.
David Clarke, can you help see if this is reproducible?
Assignee: nobody → dclarke
(In reply to Andrew Overholt [:overholt] from comment #1) > David Clarke, can you help see if this is reproducible? QA Contact, not assignee for QA btw.
Assignee: dclarke → nobody
QA Contact: dclarke
This is in fact reproducible, but I think possibly this is a gaia bug, possibly in the mediaDB.js. If we are unable to parse the metadata, we don't allow the file to be played. That looks like what is happening here, my recommend would be to create this as a gaia bug.
dflanagan can probably tell the exact issue, or i can dig in, if further clarification is needed.
blocking-basecamp: ? → +
I tried to play the ogg on desktop and it says the file is corrupt.
Oops. ignore comment 5. must have been a bad download. I was able to play the ogg video just fine.
- Ogg video doesn't show in the video app... - music app is finding the videos and saying it's an unknown type. We should probably split this into two bugs?
Duplicate of this bug: 802044
Assigning to Dale for the video part. Dale, can you file a new separate bug for the Music app problem, and assign to Dominic?
Assignee: nobody → dale
Will the camera be able to record ogg video?
Should we just not support ogv, then?
We can't do that. Is the problem here that the hard-coded file extensions in mediastorage are classifying a .ogg video as audio?
Can we sniff files that are ambiguous? Don't we have this problem with mp4 too?
Generally .oga and .ogg is for ogg audio, .ogv for video. See RFC 5334: http://xiph.org/ogg/doc/rfc5334.txt
Do we have evidence that .ogg videos are very uncommon?
I thought that we had mechanics to support autodetecting mimetypes only when loading files locally. I.e. some sort of concept of a "weak" mimetype indication. Or does our audio/video code generally not do any sniffing in any situations?
With ogg files recognised as videos and this PR https://github.com/mozilla-b2g/gaia/pull/5970 .ogg videos play fine, audio .ogg files are always recognised as video/ogg however we can check a videoWidth property to check if actual video is present The crash in the Music app is doing exactly as its supposed to, we might want to clean up the actual error thats thrown, however it is correctly refusing to play Video ogg files.
r- on the gaia patch, but it should be easy to fix. See my comments on github. Dale, in the future, please attach a dummy file on bugzilla to point to github, and request review here. As we've transitioned to bugzilla, I haven't been keeping up with review requests on github. Note that we've got a similar issue with .mp4 files, which can be audio or video. See https://github.com/mozilla-b2g/gaia/issues/5872 which probably needs to be converted to a bugzilla bug.
Attachment #674494 - Flags: review?(doug.turner) → review+
Cheers David, I fixed up the patch, and I will make sure to get the bugzilla review thing fixed for next time.
and as a note this should fix mp4 audio files as well, updated the commit message / comments to reflect that
Comment on attachment 675092 [details] Add check to make sure audio files with video/ content types are not confused with videos A couple of optional nits to consider on github, but overall looks good.
Attachment #675092 - Flags: review?(dflanagan) → review+
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.