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

Unified Diff: webrtc/modules/video_coding/codecs/test/videoprocessor.cc

Issue 2995403002: Remove unnecessary RTPFragmentationHeader from VideoProcessor callbacks. (Closed)
Patch Set: asapersson comments 1. Created 3 years, 4 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/modules/video_coding/codecs/test/videoprocessor.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/modules/video_coding/codecs/test/videoprocessor.cc
diff --git a/webrtc/modules/video_coding/codecs/test/videoprocessor.cc b/webrtc/modules/video_coding/codecs/test/videoprocessor.cc
index ac5efd23f2c84a361222d5eb3e3bd709d7be79c9..52f8d7c22544af02e7029212b70d3c3568b96f90 100644
--- a/webrtc/modules/video_coding/codecs/test/videoprocessor.cc
+++ b/webrtc/modules/video_coding/codecs/test/videoprocessor.cc
@@ -286,10 +286,8 @@ int VideoProcessor::NumberSpatialResizes() {
return num_spatial_resizes_;
}
-void VideoProcessor::FrameEncoded(
- webrtc::VideoCodecType codec,
- const EncodedImage& encoded_image,
- const webrtc::RTPFragmentationHeader* fragmentation) {
+void VideoProcessor::FrameEncoded(webrtc::VideoCodecType codec,
+ const EncodedImage& encoded_image) {
RTC_DCHECK_CALLED_SEQUENTIALLY(&sequence_checker_);
// For the highest measurement accuracy of the encode time, the start/stop
@@ -395,11 +393,6 @@ void VideoProcessor::FrameEncoded(
}
frame_info->manipulated_length = copied_image._length;
- // Keep track of if frames are lost due to packet loss so we can tell
- // this to the encoder (this is handled by the RTP logic in the full stack).
- // TODO(kjellander): Pass fragmentation header to the decoder when
- // CL 172001 has been submitted and PacketManipulator supports this.
-
// For the highest measurement accuracy of the decode time, the start/stop
// time recordings should wrap the Decode call as tightly as possible.
frame_info->decode_start_ns = rtc::TimeNanos();
« no previous file with comments | « webrtc/modules/video_coding/codecs/test/videoprocessor.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698