zqz/platform-external-webrtc
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Update EncodedFrameForMediaTransport to use Retain() rather than set_buffer + memcpy.

Browse Source 
Bug: webrtc:9378
Change-Id: I7f0d0f57bc38ecb25dd7de873c7c96a944ffb307
Reviewed-on: https://webrtc-review.googlesource.com/c/122781
Reviewed-by: Erik Språng <sprang@webrtc.org>
Commit-Queue: Niels Moller <nisse@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#26665}
This commit is contained in:
Niels Möller
2019-02-13 10:05:49 +01:00
committed by Commit Bot
parent 14a7cf9ba7
commit 69bb3afd53
1 changed files with 2 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
video/video_receive_stream.cc
View File

@ -130,14 +130,8 @@ class EncodedFrameForMediaTransport : public video_coding::EncodedFrame {
// buffer, in which case we need to make an owned copy.
*static_cast<class EncodedImage*>(this) = frame.encoded_image();
if (buffer()) {
// Unowned data. Make a copy we own.
set_buffer(nullptr, 0);
VerifyAndAllocate(frame.encoded_image().size());
set_size(frame.encoded_image().size());
memcpy(data(), frame.encoded_image().data(), size());
}
// If we don't already own the buffer, make a copy.
Retain();
_payloadType = static_cast<uint8_t>(frame.payload_type());