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

Unified Diff: media/base/video_frame.cc

Issue 2781283002: Rename functions in video frame to comply with Style Guide (Closed)
Patch Set: Fixed compilation errors for MAC Created 3 years, 8 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
« no previous file with comments | « media/base/video_frame.h ('k') | media/cast/sender/h264_vt_encoder_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/base/video_frame.cc
diff --git a/media/base/video_frame.cc b/media/base/video_frame.cc
index ea11b54a5fcf289204b552b6efc90b019fdcca4c..5148d003e73c38f01208519386677846b072074f 100644
--- a/media/base/video_frame.cc
+++ b/media/base/video_frame.cc
@@ -441,7 +441,7 @@ scoped_refptr<VideoFrame> VideoFrame::WrapVideoFrame(
if (frame->storage_type() == STORAGE_DMABUFS) {
std::vector<int> original_fds;
for (size_t i = 0; i < kMaxPlanes; ++i)
- original_fds.push_back(frame->dmabuf_fd(i));
+ original_fds.push_back(frame->DmabufFd(i));
if (!wrapping_frame->DuplicateFileDescriptors(original_fds)) {
LOG(DFATAL) << __func__ << " Couldn't duplicate fds.";
return nullptr;
@@ -723,7 +723,7 @@ size_t VideoFrame::shared_memory_offset() const {
}
#if defined(OS_LINUX)
-int VideoFrame::dmabuf_fd(size_t plane) const {
+int VideoFrame::DmabufFd(size_t plane) const {
DCHECK_EQ(storage_type_, STORAGE_DMABUFS);
DCHECK(IsValidPlane(plane, format_));
return dmabuf_fds_[plane].get();
@@ -763,7 +763,7 @@ void VideoFrame::AddSharedMemoryHandle(base::SharedMemoryHandle handle) {
}
#if defined(OS_MACOSX)
-CVPixelBufferRef VideoFrame::cv_pixel_buffer() const {
+CVPixelBufferRef VideoFrame::CvPixelBuffer() const {
return cv_pixel_buffer_.get();
}
#endif
« no previous file with comments | « media/base/video_frame.h ('k') | media/cast/sender/h264_vt_encoder_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698