Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(68)

Unified Diff: trunk/src/media/video/capture/video_capture_device.h

Issue 84393002: Revert 236927 "Reorganize media::VideoCapture* types" (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 7 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: trunk/src/media/video/capture/video_capture_device.h
===================================================================
--- trunk/src/media/video/capture/video_capture_device.h (revision 236934)
+++ trunk/src/media/video/capture/video_capture_device.h (working copy)
@@ -20,7 +20,6 @@
#include "base/memory/scoped_ptr.h"
#include "base/time/time.h"
#include "media/base/media_export.h"
-#include "media/base/video_frame.h"
#include "media/video/capture/video_capture_types.h"
namespace media {
@@ -173,7 +172,7 @@
int rotation, // Clockwise.
bool flip_vert,
bool flip_horiz,
- const VideoCaptureFormat& frame_format) = 0;
+ const VideoCaptureCapability& frame_info) = 0;
// Captured a new video frame, held in |buffer|.
//
@@ -209,10 +208,12 @@
VideoCaptureCapabilities* formats);
// Prepare the camera for use. After this function has been called no other
- // applications can use the camera. StopAndDeAllocate() must be called before
- // the object is deleted.
- virtual void AllocateAndStart(const VideoCaptureParams& params,
- scoped_ptr<Client> client) = 0;
+ // applications can use the camera. On completion Client::OnFrameInfo()
+ // is called informing of the resulting resolution and frame rate.
+ // StopAndDeAllocate() must be called before the object is deleted.
+ virtual void AllocateAndStart(
+ const VideoCaptureCapability& capture_format,
+ scoped_ptr<Client> client) = 0;
// Deallocates the camera, possibly asynchronously.
//

Powered by Google App Engine
This is Rietveld 408576698