Add ToString function for media::VideoCaptureFormat

media/video/capture/video_capture_types.cc

Index: media/video/capture/video_capture_types.cc
diff --git a/media/video/capture/video_capture_types.cc b/media/video/capture/video_capture_types.cc
index 17f2d654c6193e697b8dbc9a188bcaa099d4e74d..066794f61bfe975455ed14099b786ebbe39d4b02 100644
--- a/media/video/capture/video_capture_types.cc
+++ b/media/video/capture/video_capture_types.cc
@@ -5,6 +5,7 @@
 #include "media/video/capture/video_capture_types.h"
 
 #include "base/logging.h"
+#include "base/strings/stringprintf.h"
 #include "media/base/limits.h"
 
 namespace media {
@@ -30,6 +31,15 @@ bool VideoCaptureFormat::IsValid() const {
          (pixel_format < PIXEL_FORMAT_MAX);
 }
 
+#if !defined(NDEBUG)

[Sébastien Marchand, 2014/09/05 19:23:10]

I don't think that this prepocessor check is necessary... The unreferenced code
will be automatically removed by the linker so there's no real gain in doing it
manually. 

Also the trick used to ignore the parameters passed to DVLOG & al doesn't seem
to work well for the official builds with some specific optimizations turned
on... Like the PGO, which is currently failing because of this:
http://build.chromium.org/p/chromium.fyi/builders/Chromium%20Win%20PGO%20Builder

We could probably try to fix the EAT_STREAM_PARAMETERS macro:
https://code.google.com/p/chromium/codesearch#chromium/src/base/logging.h&l=426
to support the PGO, but in the meantime I don't see any clear reason to put this
code behind an "#if !defined(NDEBUG)" (but I might be wrong :).

+std::string VideoCaptureFormat::ToString() const {
+  return base::StringPrintf("resolution: %s, fps: %f, pixel format: %s",
+                            frame_size.ToString().c_str(),
+                            frame_rate,
+                            PixelFormatToString(pixel_format).c_str());
+}
+#endif
+
 std::string VideoCaptureFormat::PixelFormatToString(VideoPixelFormat format) {
   switch (format) {
   case PIXEL_FORMAT_UNKNOWN: