Implement video frame acknowledgements in the chromoting protocol.

remoting/protocol/message_reader.h

 // Copyright (c) 2012 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.
 
 #ifndef REMOTING_PROTOCOL_MESSAGE_READER_H_
 #define REMOTING_PROTOCOL_MESSAGE_READER_H_
 
 #include "base/callback.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
 #include "base/threading/non_thread_safe.h"
+#include "base/time/time.h"
 #include "remoting/base/compound_buffer.h"
 #include "remoting/protocol/message_decoder.h"
 
 namespace net {
 class IOBuffer;
 class Socket;
 }  // namespace net
 
 namespace remoting {
 namespace protocol {
@@ skipping 16 matching lines @@
 
   MessageReader();
   virtual ~MessageReader();
 
   // Sets the callback to be called for each incoming message.
   void SetMessageReceivedCallback(const MessageReceivedCallback& callback);
 
   // Starts reading from |socket|.
   void StartReading(net::Socket* socket);
 
+  base::TimeDelta last_message_read_delay() {

[Wez, 2015/01/21 01:35:40]

nit: Add a brief comment to explain what this means.
nit: It's not so much a delay as a duration, or an elapsed time?

[Sergey Ulanov, 2015/01/29 01:33:29]

Done.

+    return last_message_read_delay_;
+  }
+
  private:
   void DoRead();
   void OnRead(int result);
   void HandleReadResult(int result);
   void OnDataReceived(net::IOBuffer* data, int data_size);
-  void RunCallback(scoped_ptr<CompoundBuffer> message);
+  void RunCallback(scoped_ptr<CompoundBuffer> message,
+                   base::TimeDelta read_delay);
   void OnMessageDone();
 
   net::Socket* socket_;
 
   // Set to true, when we have a socket read pending, and expecting
   // OnRead() to be called when new data is received.
   bool read_pending_;
 
   // Number of messages that we received, but haven't finished
   // processing yet, i.e. |done_task| hasn't been called for these
   // messages.
   int pending_messages_;
 
   bool closed_;
   scoped_refptr<net::IOBuffer> read_buffer_;
 
   MessageDecoder message_decoder_;
 
   // Callback is called when a message is received.
   MessageReceivedCallback message_received_callback_;
 
+  base::TimeTicks last_message_started_time_;
+  base::TimeDelta last_message_read_delay_;

[Wez, 2015/01/21 01:35:40]

nit: Suggest adding a block comment e.g. "Used to track the time taken to
receive each message."

[Sergey Ulanov, 2015/01/29 01:33:29]

Done.

+
   base::WeakPtrFactory<MessageReader> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(MessageReader);
 };
 
 }  // namespace protocol
 }  // namespace remoting
 
 #endif  // REMOTING_PROTOCOL_MESSAGE_READER_H_