Chromium Code Reviews| Index: remoting/host/video_scheduler.h |
| diff --git a/remoting/host/video_scheduler.h b/remoting/host/video_scheduler.h |
| index 825f07af6f14271d57253e732d14fcd63a563732..90161f50a237c2b4171f80eed6bf37bba9666932 100644 |
| --- a/remoting/host/video_scheduler.h |
| +++ b/remoting/host/video_scheduler.h |
| @@ -114,9 +114,9 @@ class VideoScheduler : public base::RefCountedThreadSafe<VideoScheduler>, |
| // only affects capture scheduling and does not stop/start the capturer. |
| void Pause(bool pause); |
| - // Updates the sequence number embedded in VideoPackets. |
| - // Sequence numbers are used for performance measurements. |
| - void UpdateSequenceNumber(int64 sequence_number); |
| + // Updates event timestamp from the last event received from the client. This |
| + // value is sent back to the client for roundtrip latency estimates. |
| + void SetLastEventTimestamp(int64 last_event_timestamp); |
|
Wez
2015/01/07 23:56:19
s/Last/Latest
Or rename this to OnEventTimestamp(
Sergey Ulanov
2015/01/08 17:40:26
Done.
|
| // Sets whether the video encoder should be requested to encode losslessly, |
| // or to use a lossless color space (typically requiring higher bandwidth). |
| @@ -166,10 +166,10 @@ class VideoScheduler : public base::RefCountedThreadSafe<VideoScheduler>, |
| // Encode a frame, passing generated VideoPackets to SendVideoPacket(). |
| void EncodeFrame(scoped_ptr<webrtc::DesktopFrame> frame, |
| - int64 sequence_number, |
| + int64 last_event_timestamp, |
| base::TimeTicks timestamp); |
| - void EncodedDataAvailableCallback(int64 sequence_number, |
| + void EncodedDataAvailableCallback(int64 last_event_timestamp, |
| scoped_ptr<VideoPacket> packet); |
| // Task runners used by this class. |
| @@ -213,7 +213,7 @@ class VideoScheduler : public base::RefCountedThreadSafe<VideoScheduler>, |
| bool is_paused_; |
| // Number updated by the caller to trace performance. |
| - int64 sequence_number_; |
| + int64 last_event_timestamp_; |
| // An object to schedule capturing. |
| CaptureScheduler scheduler_; |