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

VideoRenderCallback::RenderFrame: Make I420VideoFrame& ref const.

Browse Source 
RenderFrame should not modify the I420VideoFrame (and we don't).

This CL changes the declaration of RenderFrame from:
int32_t RenderFrame(const uint32_t streamId, I420VideoFrame& videoFrame)
to:
int32_t RenderFrame(const uint32_t streamId, const I420VideoFrame& videoFrame)

BUG=1128
R=mflodman@webrtc.org, perkj@webrtc.org

Review URL: https://webrtc-codereview.appspot.com/46689005

Cr-Commit-Position: refs/heads/master@{#8902}
This commit is contained in:
Magnus Jedvert
2015-03-31 19:52:13 +02:00
parent 0828a0c094
commit 379069f676
22 changed files with 30 additions and 31 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
webrtc/modules/video_render/test/testAPI/testAPI.cc
View File

@ -262,7 +262,7 @@ public:
}
;
virtual int32_t RenderFrame(const uint32_t streamId,
I420VideoFrame& videoFrame)
const I420VideoFrame& videoFrame)
{
_cnt++;
if (_cnt % 100 == 0)