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

Unified Diff: webrtc/video/video_send_stream.cc

Issue 2745523002: Add FullStack test for simulcast screenshare mode. (Closed)
Patch Set: Implement Stefan@ comments Created 3 years, 9 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 | « webrtc/video/video_receive_stream.cc ('k') | webrtc/video/vie_encoder.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/video/video_send_stream.cc
diff --git a/webrtc/video/video_send_stream.cc b/webrtc/video/video_send_stream.cc
index bc7338e5ee36a50399e50190d62824116b0f01a8..ebe5892fc1fa0b22a8a49e2e502fbf0af6b88d29 100644
--- a/webrtc/video/video_send_stream.cc
+++ b/webrtc/video/video_send_stream.cc
@@ -1032,11 +1032,14 @@ EncodedImageCallback::Result VideoSendStreamImpl::OnEncodedImage(
// Encoded is called on whatever thread the real encoder implementation run
// on. In the case of hardware encoders, there might be several encoders
// running in parallel on different threads.
+ size_t simulcast_idx = 0;
+ if (codec_specific_info->codecType == kVideoCodecVP8) {
+ simulcast_idx = codec_specific_info->codecSpecific.VP8.simulcastIdx;
+ }
if (config_->post_encode_callback) {
- config_->post_encode_callback->EncodedFrameCallback(
- EncodedFrame(encoded_image._buffer, encoded_image._length,
- encoded_image._frameType, encoded_image._encodedWidth,
- encoded_image._encodedHeight, encoded_image._timeStamp));
+ config_->post_encode_callback->EncodedFrameCallback(EncodedFrame(
+ encoded_image._buffer, encoded_image._length, encoded_image._frameType,
+ simulcast_idx, encoded_image._timeStamp));
}
{
rtc::CritScope lock(&encoder_activity_crit_sect_);
« no previous file with comments | « webrtc/video/video_receive_stream.cc ('k') | webrtc/video/vie_encoder.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698