From Video Capture, abolish OnFrameInfo and enable resolution changes

media/video/capture/video_capture.h

Index: media/video/capture/video_capture.h
diff --git a/media/video/capture/video_capture.h b/media/video/capture/video_capture.h
index 3a4eb0e2d3271a9582ea9e8e51e348db81f77b28..e4fde1343d4687f955e69046d94757cda74d7408 100644
--- a/media/video/capture/video_capture.h
+++ b/media/video/capture/video_capture.h
@@ -48,12 +48,12 @@ class MEDIA_EXPORT VideoCapture {
     // Notify client about device info.
     virtual void OnDeviceInfoReceived(
         VideoCapture* capture,
-        const VideoCaptureParams& device_info) = 0;
+        const VideoCaptureFormat& device_info) = 0;

[Ami GONE FROM CHROMIUM, 2013/10/04 00:24:15]

Any reason not to provide empty body for this, like ODIChanged has?

[ncarter (slow), 2013/10/16 02:08:40]

Removed altogether.

 
     // Notify client about the newly changed device info.
     virtual void OnDeviceInfoChanged(

[Ami GONE FROM CHROMIUM, 2013/10/04 00:24:15]

CL description makes me think OnDeviceInfo{Changed,Received} are only needed for
pepper legacy.  Can they be contained to pepper-specific code instead of
permeating all classes that implement EventHandler?

[ncarter (slow), 2013/10/16 02:08:40]

Done.

         VideoCapture* capture,
-        const VideoCaptureParams& device_info) {};
+        const VideoCaptureFormat& device_info) {};
 
    protected:
     virtual ~EventHandler() {}
@@ -61,11 +61,11 @@ class MEDIA_EXPORT VideoCapture {
 
   VideoCapture() {}
 
-  // Request video capture to start capturing with |capability|.
+  // Request video capture to start capturing with |params|.
   // Also register |handler| with video capture for event handling.
   // |handler| must remain valid until it has received |OnRemoved()|.
   virtual void StartCapture(EventHandler* handler,
-                            const VideoCaptureCapability& capability) = 0;
+                            const VideoCaptureParams& params) = 0;
 
   // Request video capture to stop capturing for client |handler|.
   // |handler| must remain valid until it has received |OnRemoved()|.