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

Unified Diff: media/video/capture/linux/v4l2_capture_delegate_multi_plane.cc

Issue 1147693002: Send bytes_used of v4l2 captured frame when video capture (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: keep payload in BufferTracker Created 5 years, 7 months 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/video/capture/linux/v4l2_capture_delegate_multi_plane.cc
diff --git a/media/video/capture/linux/v4l2_capture_delegate_multi_plane.cc b/media/video/capture/linux/v4l2_capture_delegate_multi_plane.cc
index 7551bed434c7f7b5603a12bb842715616aa95e00..15c95e26f8960656ae0673077aba0bf248b38daa 100644
--- a/media/video/capture/linux/v4l2_capture_delegate_multi_plane.cc
+++ b/media/video/capture/linux/v4l2_capture_delegate_multi_plane.cc
@@ -55,6 +55,13 @@ void V4L2CaptureDelegateMultiPlane::FinishFillingV4L2Buffer(
buffer->m.planes = v4l2_planes_.data();
}
+void V4L2CaptureDelegateMultiPlane::FetchBytesUsed(
wuchengli 2015/05/25 07:22:19 s/FetchBytesUsed/SetBytesUsed/. The direction shou
kcwu 2015/05/25 07:36:25 Changed to SetPayloadSize per offline discussion.
+ const scoped_refptr<BufferTracker>& buffer_tracker,
+ const v4l2_buffer* buffer) const {
wuchengli 2015/05/25 07:22:19 Use const reference. This is input.
kcwu 2015/05/25 07:36:25 Done.
+ for (size_t i = 0; i < v4l2_planes_.size(); i++)
+ buffer_tracker->SetPlanePayload(i, buffer->m.planes[i].bytesused);
wuchengli 2015/05/25 07:22:19 s/SetPlanePayload/SetPlanePayloadSize/. Payload us
kcwu 2015/05/25 07:36:25 Done.
+}
+
void V4L2CaptureDelegateMultiPlane::SendBuffer(
const scoped_refptr<BufferTracker>& buffer_tracker,
const v4l2_format& format) const {

Powered by Google App Engine
This is Rietveld 408576698