Unittests for RTP packetizer and depacketizer.

BUG=None
TEST=Unittests

Review URL: http://codereview.chromium.org/4946001

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@66536 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 18 insertions, 2 deletions

diff --git a/remoting/protocol/rtp_video_reader.h b/remoting/protocol/rtp_video_reader.h
index 9a0ae89..412a4f0 100644
--- a/remoting/protocol/rtp_video_reader.h
+++ b/remoting/protocol/rtp_video_reader.h
@@ -25,7 +25,23 @@ class RtpVideoReader : public VideoReader {
  private:
   friend class RtpVideoReaderTest;
 
-  typedef std::deque<const RtpPacket*> PacketsQueue;
+  // Following struct is used to store pending packets in |packets_queue_|.
+  // Each entry may be in three different states:
+  //  |received| == false, |packet| == NULL - packet with the corresponding
+  //    sequence number hasn't been received.
+  //  |received| == true, |packet| != NULL - packet with the corresponding
+  //    sequence number has been received, but hasn't been processed, still
+  //    waiting for other fragments.
+  //  |received| == true, |packet| == NULL - packet with the corresponding
+  //    sequence number has been received and processed. Ignore any additional
+  //    packet with the same sequence number.
+  struct PacketsQueueEntry {
+    PacketsQueueEntry();
+    bool received;
+    const RtpPacket* packet;
+  };
+
+  typedef std::deque<PacketsQueueEntry> PacketsQueue;
 
   void OnRtpPacket(const RtpPacket* rtp_packet);
   void CheckFullPacket(PacketsQueue::iterator pos);
@@ -36,7 +52,7 @@ class RtpVideoReader : public VideoReader {
   RtpReader rtp_reader_;
 
   PacketsQueue packets_queue_;
-  uint32 last_sequence_number_;
+  uint16 last_sequence_number_;
 
   // The stub that processes all received packets.
   VideoStub* video_stub_;