Added logging calls to FFmpegDemuxer.

media/base/video_frame.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef MEDIA_BASE_VIDEO_FRAME_H_
 #define MEDIA_BASE_VIDEO_FRAME_H_
 
 #include "base/callback.h"
 #include "base/md5.h"
 #include "base/memory/shared_memory.h"
@@ skipping 35 matching lines @@
     YV16 = 7,  // 16bpp YVU planar 1x1 Y, 2x1 VU samples
     EMPTY = 9,  // An empty frame.
     I420 = 11,  // 12bpp YVU planar 1x1 Y, 2x2 UV samples.
     NATIVE_TEXTURE = 12,  // Native texture.  Pixel-format agnostic.
 #if defined(GOOGLE_TV)
     HOLE = 13,  // Hole frame.
 #endif
     YV12A = 14,  // 20bpp YUVA planar 1x1 Y, 2x2 VU, 1x1 A samples.
   };
 
+  static std::string FormatToString(Format format) {

[scherkus (not reviewing), 2013/08/02 23:18:19]

don't inline this function declaration inside the .h

instead keep the definition here but declare it in the .cc

/* video_frame.h */
static std::string FormatToString(Format format);

/* video_frame.cc */
// static
std::string VideoFrame::FormatToString(Format format) {
  // ...
}

[Ty Overby, 2013/08/03 00:02:23]

Done.

+    switch(format){
+      case VideoFrame::Format::INVALID:
+        return "INVALID";
+      case VideoFrame::Format::RGB32:
+        return "RGB32";
+      case VideoFrame::Format::YV12:
+        return "YV12";
+      case VideoFrame::Format::YV16:
+        return "YV16";
+      case VideoFrame::Format::EMPTY:
+        return "EMPTY";
+      case VideoFrame::Format::I420:
+        return "I420";
+      case VideoFrame::Format::YV12A:
+        return "YV12A";
+      case VideoFrame::Format::NATIVE_TEXTURE:
+        return "NATIVE_TEXTURE";
+    }
+  }
+
   // This class calls the TextureNoLongerNeededCallback when the last reference
   // on the class is destroyed. The VideoFrame holds a reference to the mailbox
   // but anyone else who queries the mailbox should also hold a reference while
   // it is uses the mailbox, to ensure it remains valid. When finished with the
   // mailbox, call Return() with a new sync point, to ensure the mailbox remains
   // valid for the issued commands.
   class MEDIA_EXPORT MailboxHolder
       : public base::RefCountedThreadSafe<MailboxHolder> {
    public:
     typedef base::Callback<void(uint32 sync_point)>
@@ skipping 200 matching lines @@
   base::Closure no_longer_needed_cb_;
 
   base::TimeDelta timestamp_;
 
   DISALLOW_IMPLICIT_CONSTRUCTORS(VideoFrame);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_VIDEO_FRAME_H_