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

Unified Diff: webrtc/video/video_send_stream.cc

Issue 2089773002: Add EncodedImageCallback::OnEncodedImage(). (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: sync Created 4 years, 6 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/video/video_send_stream.cc
diff --git a/webrtc/video/video_send_stream.cc b/webrtc/video/video_send_stream.cc
index 01c2fc836c7d4a35dde12d1a13b4cd2fbb8e5cf0..95b37a23a83f6f4b87f9438144d96a08ce882b2d 100644
--- a/webrtc/video/video_send_stream.cc
+++ b/webrtc/video/video_send_stream.cc
@@ -703,9 +703,10 @@ void VideoSendStream::NormalUsage() {
config_.overuse_callback->OnLoadUpdate(LoadObserver::kUnderuse);
}
-int32_t VideoSendStream::Encoded(const EncodedImage& encoded_image,
- const CodecSpecificInfo* codec_specific_info,
- const RTPFragmentationHeader* fragmentation) {
+EncodedImageCallback::Result VideoSendStream::OnEncodedImage(
+ const EncodedImage& encoded_image,
+ const CodecSpecificInfo* codec_specific_info,
+ const RTPFragmentationHeader* fragmentation) {
if (config_.post_encode_callback) {
config_.post_encode_callback->EncodedFrameCallback(
EncodedFrame(encoded_image._buffer, encoded_image._length,
@@ -713,7 +714,7 @@ int32_t VideoSendStream::Encoded(const EncodedImage& encoded_image,
}
protection_bitrate_calculator_.UpdateWithEncodedData(encoded_image);
- int32_t return_value = payload_router_.Encoded(
+ EncodedImageCallback::Result result = payload_router_.OnEncodedImage(
encoded_image, codec_specific_info, fragmentation);
if (kEnableFrameRecording) {
@@ -739,7 +740,7 @@ int32_t VideoSendStream::Encoded(const EncodedImage& encoded_image,
}
}
- return return_value;
+ return result;
}
void VideoSendStream::ConfigureProtection() {

Powered by Google App Engine
This is Rietveld 408576698