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

Unified Diff: webrtc/test/frame_utils.cc

Issue 1878623002: Added new VideoFrameBuffer methods Data[YUV]() etc. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 4 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
Index: webrtc/test/frame_utils.cc
diff --git a/webrtc/test/frame_utils.cc b/webrtc/test/frame_utils.cc
index 0f411447458173331b48db59de325b7a4cc174c4..ed2d44c5a967520a746dbed209cb0a5d4ec5adce 100644
--- a/webrtc/test/frame_utils.cc
+++ b/webrtc/test/frame_utils.cc
@@ -50,19 +50,19 @@ bool FramesEqual(const webrtc::VideoFrame& f1, const webrtc::VideoFrame& f2) {
bool FrameBufsEqual(const rtc::scoped_refptr<webrtc::VideoFrameBuffer>& f1,
const rtc::scoped_refptr<webrtc::VideoFrameBuffer>& f2) {
if (f1->width() != f2->width() || f1->height() != f2->height() ||
- f1->stride(webrtc::kYPlane) != f2->stride(webrtc::kYPlane) ||
- f1->stride(webrtc::kUPlane) != f2->stride(webrtc::kUPlane) ||
- f1->stride(webrtc::kVPlane) != f2->stride(webrtc::kVPlane)) {
+ f1->StrideY() != f2->StrideY() ||
+ f1->StrideU() != f2->StrideU() ||
+ f1->StrideV() != f2->StrideV()) {
return false;
}
const int half_width = (f1->width() + 1) / 2;
const int half_height = (f1->height() + 1) / 2;
- return EqualPlane(f1->data(webrtc::kYPlane), f2->data(webrtc::kYPlane),
- f1->stride(webrtc::kYPlane), f1->width(), f1->height()) &&
- EqualPlane(f1->data(webrtc::kUPlane), f2->data(webrtc::kUPlane),
- f1->stride(webrtc::kUPlane), half_width, half_height) &&
- EqualPlane(f1->data(webrtc::kVPlane), f2->data(webrtc::kVPlane),
- f1->stride(webrtc::kVPlane), half_width, half_height);
+ return EqualPlane(f1->DataY(), f2->DataY(),
+ f1->StrideY(), f1->width(), f1->height()) &&
+ EqualPlane(f1->DataU(), f2->DataU(),
+ f1->StrideU(), half_width, half_height) &&
+ EqualPlane(f1->DataV(), f2->DataV(),
+ f1->StrideV(), half_width, half_height);
}
} // namespace test
« webrtc/common_video/i420_buffer_pool.cc ('K') | « webrtc/test/fake_texture_frame.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698