New interface EncodedImageBufferInterface, replacing use of CopyOnWriteBuffer
Bug: webrtc:9378
Change-Id: I62b7adbd9dd539c545b5b1b1520721482a4623c4
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/138820
Reviewed-by: Kári Helgason <kthelgason@webrtc.org>
Reviewed-by: Sami Kalliomäki <sakal@webrtc.org>
Reviewed-by: Karl Wiberg <kwiberg@webrtc.org>
Commit-Queue: Niels Moller <nisse@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#28317}
diff --git a/api/test/loopback_media_transport_unittest.cc b/api/test/loopback_media_transport_unittest.cc
index 8e510e6..afd544f 100644
--- a/api/test/loopback_media_transport_unittest.cc
+++ b/api/test/loopback_media_transport_unittest.cc
@@ -105,9 +105,8 @@
::testing::StrictMock<MockMediaTransportVideoSinkInterface> sink;
constexpr uint8_t encoded_data[] = {1, 2, 3};
EncodedImage encoded_image;
- encoded_image.Allocate(sizeof(encoded_data));
- memcpy(encoded_image.data(), encoded_data, sizeof(encoded_data));
- encoded_image.set_size(sizeof(encoded_data));
+ encoded_image.SetEncodedData(
+ EncodedImageBuffer::Create(encoded_data, sizeof(encoded_data)));
EXPECT_CALL(sink, OnData(1, ::testing::Property(
&MediaTransportEncodedVideoFrame::frame_id,
@@ -115,12 +114,8 @@
.WillOnce(::testing::Invoke(
[&encoded_image](int frame_id,
const MediaTransportEncodedVideoFrame& frame) {
- EXPECT_NE(frame.encoded_image().data(), encoded_image.data());
+ EXPECT_EQ(frame.encoded_image().data(), encoded_image.data());
EXPECT_EQ(frame.encoded_image().size(), encoded_image.size());
- EXPECT_EQ(0,
- memcmp(frame.encoded_image().data(), encoded_image.data(),
- std::min(frame.encoded_image().size(),
- encoded_image.size())));
}));
transport_pair.second()->SetReceiveVideoSink(&sink);