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

Unified Diff: media/cast/net/rtp/framer.cc

Issue 339743002: Cast: Make vp8 3-buffer mode work (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: comments addressed, but I am going to do more testing.. Created 6 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
Index: media/cast/net/rtp/framer.cc
diff --git a/media/cast/net/rtp/framer.cc b/media/cast/net/rtp/framer.cc
index c94dc0c7125802a1c651f9c8b6b3b3f24db8f236..dc26d1cbea94458caa1fed3cafcff50ace1efc36 100644
--- a/media/cast/net/rtp/framer.cc
+++ b/media/cast/net/rtp/framer.cc
@@ -83,11 +83,6 @@ bool Framer::GetEncodedFrame(EncodedFrame* frame,
*next_frame = false;
}
- if (*next_frame) {
- VLOG(2) << "ACK frame " << frame_id;
- cast_msg_builder_->CompleteFrameReceived(frame_id);
- }
-
ConstFrameIterator it = frames_.find(frame_id);
DCHECK(it != frames_.end());
if (it == frames_.end())
@@ -96,6 +91,11 @@ bool Framer::GetEncodedFrame(EncodedFrame* frame,
return it->second->AssembleEncodedFrame(frame);
}
+void Framer::AckFrame(uint32 frame_id) {
+ VLOG(2) << "ACK frame " << frame_id;
+ cast_msg_builder_->CompleteFrameReceived(frame_id);
+}
+
void Framer::Reset() {
frame_id_map_.Clear();
frames_.clear();

Powered by Google App Engine
This is Rietveld 408576698