Video Capture: Allow 0 fps frame rates to be deserializable

media/video/capture/win/video_capture_device_factory_win.cc

 // Copyright 2014 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.
 
 #include "media/video/capture/win/video_capture_device_factory_win.h"
 
 #include <mfapi.h>
 #include <mferror.h>
 
 #include "base/command_line.h"
@@ skipping 280 matching lines @@
         VideoCaptureFormat format;
         format.pixel_format =
             VideoCaptureDeviceWin::TranslateMediaSubtypeToPixelFormat(
                 media_type->subtype);
         if (format.pixel_format == PIXEL_FORMAT_UNKNOWN)
           continue;
         VIDEOINFOHEADER* h =
             reinterpret_cast<VIDEOINFOHEADER*>(media_type->pbFormat);
         format.frame_size.SetSize(h->bmiHeader.biWidth,
                                   h->bmiHeader.biHeight);
+        if (!h->AvgTimePerFrame)

[perkj_chrome, 2014/06/19 11:23:35]

So this does not add the format at all. Is that really what you want? 

+          continue;
         // Trust the frame rate from the VIDEOINFOHEADER.
-        format.frame_rate = (h->AvgTimePerFrame > 0) ?
-            kSecondsToReferenceTime / static_cast<float>(h->AvgTimePerFrame) :
-            0.0;
+        format.frame_rate =
+            kSecondsToReferenceTime / static_cast<float>(h->AvgTimePerFrame);
 
         formats->push_back(format);
         DVLOG(1) << device.name() << " resolution: "
              << format.frame_size.ToString() << ", fps: " << format.frame_rate
              << ", pixel format: " << format.pixel_format;
       }
     }
   }
 }
 
@@ skipping 126 matching lines @@
     const VideoCaptureDevice::Name& device,
     VideoCaptureFormats* formats) {
   DCHECK(thread_checker_.CalledOnValidThread());
   if (use_media_foundation_)
     GetDeviceSupportedFormatsMediaFoundation(device, formats);
   else
     GetDeviceSupportedFormatsDirectShow(device, formats);
 }
 
 }  // namespace media