Closed Bug 1065850 Opened 7 years ago Closed 7 years ago

MP4 demuxer blocks waiting for the first moof box

Categories

(Core :: Audio/Video, defect)

x86_64
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla35

People

(Reporter: ajones, Assigned: ajones)

References

Details

Attachments

(1 file)

The MP4 demuxer reads up to the first moof box before it completes reading the metadata. This contributes to bug 1065827 for MSE.
Comment on attachment 8487681 [details] [diff] [review]
Read as little as possible to get the MP4 track metadata

Review of attachment 8487681 [details] [diff] [review]:
-----------------------------------------------------------------

::: media/libstagefright/frameworks/av/media/libstagefright/MPEG4Extractor.cpp
@@ +441,5 @@
>              && !track->includes_expensive_metadata) {
>          track->includes_expensive_metadata = true;
>  
>          const char *mime;
>          CHECK(track->meta->findCString(kKeyMIMEType, &mime));

Do you want to remove the mime lookup too?
Attachment #8487681 - Flags: review?(giles) → review+
https://hg.mozilla.org/mozilla-central/rev/5876e536d628
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla35
You need to log in before you can comment on or make changes to this bug.