MSE: Increase log visibility when unfragmented MP4 causes parse failure

media/formats/mp4/box_reader.cc

Index: media/formats/mp4/box_reader.cc
diff --git a/media/formats/mp4/box_reader.cc b/media/formats/mp4/box_reader.cc
index 925859d20bda820f31caf76f7949d106cb432f33..978f17d68737a3d27a0cca0d703bc05e75ad4948 100644
--- a/media/formats/mp4/box_reader.cc
+++ b/media/formats/mp4/box_reader.cc
@@ -222,7 +222,8 @@ bool BoxReader::ReadHeader(bool* err) {
   CHECK(Read4Into8(&size) && ReadFourCC(&type_));
 
   if (size == 0) {
-    // Media Source specific: we do not support boxes that run to EOS.
+    MEDIA_LOG(DEBUG, log_cb_) << "Media Source Extensions do not support ISO "
+                                 "BMFF boxes that run to EOS";

[chcunningham, 2015/05/15 17:19:33]

I don't know what this means. Why is size 0? 

[wolenetz, 2015/05/15 18:16:25]

:) I was simply log-ifying the previous comment in a drive-by.
I also believe both the previous comment and the log-ified version are correct:
As I understand it from the ISO IEC 14497-12 (2012) spec, section 4.2 (Object
Structure, which describes the pertinent common box header stuff that is parsed
here), there is very specific code in the spec describing that "if (size==0) {
// box extends to end of file }". MSE does not support such boxes in ISO BMFF.
My guess about a use case for something like a box extending to EOS might be an
MDAT containing media data, and whose end is signaled by reaching the end of
media data, rather than the end of the box. This is similar to webm "unknown
cluster lengths" (which *are* supported by MSE due to the nature of block
sequence with Clusters being required to be muxed in DTS sequence, unlike MDAT
in ISO BMFF).

     *err = true;
     return false;
   } else if (size == 1) {