Fix codec name logging in ivf_file_writer.cc

webrtc/modules/video_coding/utility/ivf_file_writer.cc

Index: webrtc/modules/video_coding/utility/ivf_file_writer.cc
diff --git a/webrtc/modules/video_coding/utility/ivf_file_writer.cc b/webrtc/modules/video_coding/utility/ivf_file_writer.cc
index a3b680d72cacda1ce96cecc8ac663f984d875091..97f1da30e40a1b7fe7b8ad07fd4bd1c684c835b6 100644
--- a/webrtc/modules/video_coding/utility/ivf_file_writer.cc
+++ b/webrtc/modules/video_coding/utility/ivf_file_writer.cc
@@ -118,17 +118,17 @@ bool IvfFileWriter::InitFromFirstFrame(const EncodedImage& encoded_image) {
 
   std::string codec_name;
   switch (codec_type_) {

[pbos-webrtc, 2016/05/09 16:44:33]

Can you store codec_type_ as the proper enum type so that this gets caught in
compile-time?

[hta-webrtc, 2016/05/09 16:59:08]

It was, and it didn't. codec_type_ is the proper enum type, and the compiler did
not catch the use of enum values from another enum.

-    case kRtpVideoVp8:
+    case kVideoCodecVP8:
       codec_name = "VP8";
       break;
-    case kRtpVideoVp9:
+    case kVideoCodecVP9:
       codec_name = "VP9";
       break;
-    case kRtpVideoH264:
+    case kVideoCodecH264:
       codec_name = "H264";
       break;
     default:
-      codec_name = "Unkown";
+      codec_name = "Unknown";
   }
   LOG(LS_WARNING) << "Created IVF file " << file_name_
                   << " for codec data of type " << codec_name