From Video Capture, abolish OnFrameInfo and enable resolution changes

media/video/capture/video_capture.h

 // Copyright (c) 2012 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.
 //
 // This file contains abstract classes used for media filter to handle video
 // capture devices.
 
 #ifndef MEDIA_VIDEO_CAPTURE_VIDEO_CAPTURE_H_
 #define MEDIA_VIDEO_CAPTURE_VIDEO_CAPTURE_H_
 
@@ skipping 30 matching lines @@
     virtual void OnRemoved(VideoCapture* capture) = 0;
 
     // Notify client that a buffer is available.
     virtual void OnFrameReady(
         VideoCapture* capture,
         const scoped_refptr<media::VideoFrame>& frame) = 0;
 
     // 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() {}
   };
 
   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()|.
   virtual void StopCapture(EventHandler* handler) = 0;
 
   virtual bool CaptureStarted() = 0;
   virtual int CaptureWidth() = 0;
   virtual int CaptureHeight() = 0;
   virtual int CaptureFrameRate() = 0;
 
  protected:
   virtual ~VideoCapture() {}
 
  private:
   DISALLOW_COPY_AND_ASSIGN(VideoCapture);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_VIDEO_CAPTURE_VIDEO_CAPTURE_H_