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

Unified Diff: media/capture/video/video_capture_device.cc

Issue 2573223002: [Mojo Video Capture] Simplify media::VideoCaptureDevice::Client:Buffer to a struct (Closed)
Patch Set: Backport naming improvements and Ownership interface from future CLs Created 4 years 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: media/capture/video/video_capture_device.cc
diff --git a/media/capture/video/video_capture_device.cc b/media/capture/video/video_capture_device.cc
index 5da33f01b7c39fcde86833e000fc04a04bbfd3da..0db6798c86e032086256f7b99c142d98f8b08c49 100644
--- a/media/capture/video/video_capture_device.cc
+++ b/media/capture/video/video_capture_device.cc
@@ -13,8 +13,25 @@
namespace media {
-VideoCaptureDevice::Client::Buffer::~Buffer() {
-}
+VideoCaptureDevice::Client::Buffer::Buffer() : id_(0), frame_feedback_id_(0) {}
+
+VideoCaptureDevice::Client::Buffer::Buffer(
+ int buffer_id,
+ int frame_feedback_id,
+ std::unique_ptr<BufferHandleProvider> handle_provider,
+ std::unique_ptr<Ownership> buffer_read_write_permission)
+ : handle_provider(std::move(handle_provider)),
+ buffer_read_write_permission(std::move(buffer_read_write_permission)),
+ id_(buffer_id),
+ frame_feedback_id_(frame_feedback_id) {}
+
+VideoCaptureDevice::Client::Buffer::Buffer(
+ VideoCaptureDevice::Client::Buffer&& other) = default;
+
+VideoCaptureDevice::Client::Buffer::~Buffer() = default;
+
+VideoCaptureDevice::Client::Buffer& VideoCaptureDevice::Client::Buffer::
+operator=(VideoCaptureDevice::Client::Buffer&& other) = default;
VideoCaptureDevice::~VideoCaptureDevice() {
}

Powered by Google App Engine
This is Rietveld 408576698