Cast: Make vp8 3-buffer mode work

media/cast/net/rtp/framer.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "media/cast/net/rtp/framer.h"
 
 #include "base/logging.h"
 
 namespace media {
 namespace cast {
@@ skipping 65 matching lines @@
     // Check if we can skip frames when our decoder is too slow.
     if (!decoder_faster_than_max_frame_rate_)
       return false;
 
     if (!frame_id_map_.NextFrameAllowingSkippingFrames(&frame_id)) {
       return false;
     }
     *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())
     return false;
 
   return it->second->AssembleEncodedFrame(frame);
 }
 
+void Framer::AckFrame(uint32 frame_id) {
+  VLOG(2) << "ACK frame " << frame_id;
+  cast_msg_builder_->CompleteFrameReceived(frame_id);
+}
+
+

[Alpha Left Google, 2014/08/25 23:23:44]

nit: please remove empty line.

[hubbe, 2014/08/27 04:14:03]

Done.

 void Framer::Reset() {
   frame_id_map_.Clear();
   frames_.clear();
   cast_msg_builder_->Reset();
 }
 
 void Framer::ReleaseFrame(uint32 frame_id) {
   frame_id_map_.RemoveOldFrames(frame_id);
   frames_.erase(frame_id);
 
@@ skipping 14 matching lines @@
 }
 
 bool Framer::TimeToSendNextCastMessage(base::TimeTicks* time_to_send) {
   return cast_msg_builder_->TimeToSendNextCastMessage(time_to_send);
 }
 
 void Framer::SendCastMessage() { cast_msg_builder_->UpdateCastMessage(); }
 
 }  // namespace cast
 }  // namespace media