Added framer to cast.

media/cast/framer/cast_message_builder.h

+// Copyright 2013 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.
+
+// Handle NACK list and manage ACK.

[Alpha Left Google, 2013/08/28 00:27:31]

nit: *Handles and *manages.

[pwestin, 2013/08/28 16:40:44]

Done.

+
+#ifndef MEDIA_CAST_FRAMER_CAST_MESSAGE_BUILDER_H_
+#define MEDIA_CAST_FRAMER_CAST_MESSAGE_BUILDER_H_
+
+#include <map>
+
+#include "media/cast/framer/frame_id_map.h"
+#include "media/cast/rtcp/rtcp.h"
+#include "media/cast/rtp_common/rtp_defines.h"
+
+namespace media {
+namespace cast {
+
+class RtpPayloadFeedback;
+
+class CastMessageBuilder {
+ public:
+  CastMessageBuilder(RtpPayloadFeedback* incoming_payload_feedback,
+                     FrameIdMap* frame_id_map,
+                     uint32 media_ssrc,

[Alpha Left Google, 2013/08/28 00:27:31]

nit: this can go to the previous line i think.

[pwestin, 2013/08/28 16:40:44]

yes, however I think this is much more readable; ok?

+                     bool decoder_faster_than_max_frame_rate,
+                     int max_unacked_frames);
+  ~CastMessageBuilder();
+
+  void CompleteFrameReceived(uint8 frame_id, bool is_key_frame);
+  bool TimeToSendNextCastMessage(base::TimeTicks* time_to_send);
+  void UpdateCastMessage();
+  void Reset();
+
+  void set_clock(base::TickClock* clock) {
+    clock_ = clock;
+  }
+
+ private:
+  bool UpdateAckMessage();
+  void BuildPacketList();
+  bool UpdateCastMessageInternal(RtcpCastMessage* message);
+
+  RtpPayloadFeedback* const cast_feedback_;
+
+  // CastMessageBuilder has only const access to the frame id mapper.
+  const FrameIdMap* const frame_id_map_;
+  const uint32 media_ssrc_;
+  const bool decoder_faster_than_max_frame_rate_;
+  const int max_unacked_frames_;
+
+  RtcpCastMessage cast_msg_;
+  base::TimeTicks last_update_time_;
+  bool waiting_for_key_frame_;
+
+  std::map<uint8, base::TimeTicks> time_last_nacked_map_;

[Alpha Left Google, 2013/08/28 00:27:31]

nit: Use a typedef here.

[pwestin, 2013/08/28 16:40:44]

Done.

+
+  bool slowing_down_ack_;
+  bool acked_last_frame_;
+  uint8 last_acked_frame_id_;
+
+  scoped_ptr<base::TickClock> default_tick_clock_;
+  base::TickClock* clock_;
+
+  DISALLOW_COPY_AND_ASSIGN(CastMessageBuilder);
+};
+
+}  // namespace cast
+}  // namespace media
+
+#endif  //  MEDIA_CAST_FRAMER_CAST_MESSAGE_BUILDER_H_