Added logging calls to FFmpegDemuxer.

media/base/video_decoder_config.cc

Index: media/base/video_decoder_config.cc
diff --git a/media/base/video_decoder_config.cc b/media/base/video_decoder_config.cc
index da914f22fb94c0a669b326dc6f7c6f32f2cc0d61..76e37d37600bd1c2f8dc1d0897cbddea0d312204 100644
--- a/media/base/video_decoder_config.cc
+++ b/media/base/video_decoder_config.cc
@@ -139,6 +139,37 @@ VideoFrame::Format VideoDecoderConfig::format() const {
   return format_;
 }
 
+std::string VideoDecoderConfig::FormatName() const {

[scherkus (not reviewing), 2013/08/02 21:53:48]

fix indenting

[Ty Overby, 2013/08/02 23:04:27]

Done.

+      std::string format_name;

[scherkus (not reviewing), 2013/08/02 21:53:48]

you don't need this -- instead just return string

[Ty Overby, 2013/08/02 23:04:27]

Done.

+      switch(format()){

[scherkus (not reviewing), 2013/08/02 21:53:48]

since this is a VideoFrame::Format, it should be a helper method on VideoFrame


static std::string FormatToString(Format format)

[Ty Overby, 2013/08/02 23:04:27]

Done.

+        case VideoFrame::Format::INVALID:
+          format_name = "INVALID";
+          break;
+        case VideoFrame::Format::RGB32:
+          format_name = "RGB32";
+          break;
+        case VideoFrame::Format::YV12:
+          format_name = "YV12";
+          break;
+        case VideoFrame::Format::YV16:
+          format_name = "YV16";
+          break;
+        case VideoFrame::Format::EMPTY:
+          format_name = "EMPTY";
+          break;
+        case VideoFrame::Format::I420:
+          format_name = "I420";
+          break;
+        case VideoFrame::Format::YV12A:
+          format_name = "YV12A";
+          break;
+        default:

[scherkus (not reviewing), 2013/08/02 21:53:48]

we don't like default cases -- it's better to leave them out and let the
compiler complain anytime someone changes the enum definition

[Ty Overby, 2013/08/02 23:04:27]

Done. 

+          format_name = "INVALID";
+          break;
+      }
+      return format_name;
+}
+
 gfx::Size VideoDecoderConfig::coded_size() const {
   return coded_size_;
 }