Windows video capture: Remove duplicated code from GetDeviceSupportedFormats*

media/video/capture/win/video_capture_device_win.h

Index: media/video/capture/win/video_capture_device_win.h
diff --git a/media/video/capture/win/video_capture_device_win.h b/media/video/capture/win/video_capture_device_win.h
index 48bc5e6ec28d61cc80268620d715f1c86b4abb06..39d73beb00d5ec7f4ab6789f6a12da190d7f32cc 100644
--- a/media/video/capture/win/video_capture_device_win.h
+++ b/media/video/capture/win/video_capture_device_win.h
@@ -62,6 +62,12 @@ class VideoCaptureDeviceWin
   static VideoPixelFormat TranslateMediaSubtypeToPixelFormat(
       const GUID& sub_type);
 
+  static bool VideoCaptureDeviceWin::GetDeviceSupportedFormats(
+      const Name& device,
+      base::win::ScopedComPtr<IBaseFilter>* capture_filter,

[tommi (sloooow) - chröme, 2014/10/22 16:23:13]

this is a bit strange for COM code.  If this is meant to be an out pointer, the
convention is like GetDeviceFilter is.  Same for the next param.

+      base::win::ScopedComPtr<IPin>* output_capture_pin,
+      CapabilityList* capabilities);
+
   explicit VideoCaptureDeviceWin(const Name& device_name);
   virtual ~VideoCaptureDeviceWin();
   // Opens the device driver for this device.
@@ -84,7 +90,6 @@ class VideoCaptureDeviceWin
   // Implements SinkFilterObserver.
   virtual void FrameReceived(const uint8* buffer, int length);
 
-  bool CreateCapabilityMap();
   void SetAntiFlickerInCaptureFilter();
   void SetErrorState(const std::string& reason);