[Cast] EncodedAudioFrame+EncodedVideoFrame+reference_time --> EncodedFrame

media/cast/framer/frame_buffer.h

Index: media/cast/framer/frame_buffer.h
diff --git a/media/cast/framer/frame_buffer.h b/media/cast/framer/frame_buffer.h
index 65df021d2542c963f88ff370006942f9ac495b4b..ad7071416663adb74d3aeb78227cce79a8520fbe 100644
--- a/media/cast/framer/frame_buffer.h
+++ b/media/cast/framer/frame_buffer.h
@@ -25,11 +25,13 @@ class FrameBuffer {
                     const RtpCastHeader& rtp_header);
   bool Complete() const;
 
-  bool GetEncodedAudioFrame(transport::EncodedAudioFrame* audio_frame) const;
+  // If a frame is complete, sets the frame IDs and RTP timestamp in |frame|,
+  // and also copies the data from all packets into the data field in |frame|.
+  // Returns true if the frame was complete; false if incomplete and |frame|
+  // remains unchanged.
+  bool AssembleEncodedFrame(transport::EncodedFrame* frame) const;
 
-  bool GetEncodedVideoFrame(transport::EncodedVideoFrame* video_frame) const;
-
-  bool is_key_frame() const { return is_key_frame_; }
+  bool is_key_frame() const { return frame_id_ == last_referenced_frame_id_; }

[Alpha Left Google, 2014/05/15 22:11:23]

I don't think we need to change this.

A key frame has two meanings:
1. It can be decoded independently.
2. Subsequent frames will not depend on frames prior to the key frame. Meaning a
restart of the encoded stream.

While referenced frame id == frame id means it is independently decodable but it
doesn't necessarily mean a key frame.

[miu, 2014/05/16 04:27:25]

Done.

You make an excellent point.  I talked with hubbe@ earlier today and also
thought about this a bit more.  So, I've decided to add back the "is key frame"
in EncodedFrame (in the form of an enum).

 
   uint32 last_referenced_frame_id() const { return last_referenced_frame_id_; }
 
@@ -37,7 +39,6 @@ class FrameBuffer {
   uint32 frame_id_;
   uint16 max_packet_id_;
   uint16 num_packets_received_;
-  bool is_key_frame_;
   size_t total_data_size_;
   uint32 last_referenced_frame_id_;
   uint32 rtp_timestamp_;