From c0f7082fdc8b239027b196a777180899bf7f5ce3 Mon Sep 17 00:00:00 2001 From: "hclam@chromium.org" Date: Fri, 6 May 2011 21:19:29 +0000 Subject: Chromoting to report roundtrip latency Doing so by sending a sequence number, essentially the timestamp in every envet message. Capturer at the host will pick up the latest sequence number and pass it through the pipeline. Client will then receive it and determine the latency. This roundtrip latency number however doesn't include time in decoding and rendering. BUG=None TEST=None Review URL: http://codereview.chromium.org/6792038 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84504 0039d316-1c4b-4281-b951-d872f2087c98 --- remoting/proto/internal.proto | 4 ++-- remoting/proto/video.proto | 6 +++++- 2 files changed, 7 insertions(+), 3 deletions(-) (limited to 'remoting/proto') diff --git a/remoting/proto/internal.proto b/remoting/proto/internal.proto index c52dd2d..bee7d03 100644 --- a/remoting/proto/internal.proto +++ b/remoting/proto/internal.proto @@ -1,4 +1,4 @@ -// Copyright (c) 2010 The Chromium Authors. All rights reserved. +// Copyright (c) 2011 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. @@ -25,7 +25,7 @@ message ControlMessage { // Defines an event message on the event channel. message EventMessage { - required int32 timestamp = 1; // Client timestamp for event + required int64 sequence_number = 1; // Client timestamp for event optional bool dummy = 2; // Is this a dummy event? optional KeyEvent key_event = 3; diff --git a/remoting/proto/video.proto b/remoting/proto/video.proto index b9bd53c..049351e 100644 --- a/remoting/proto/video.proto +++ b/remoting/proto/video.proto @@ -1,4 +1,4 @@ -// Copyright (c) 2010 The Chromium Authors. All rights reserved. +// Copyright (c) 2011 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. @@ -91,4 +91,8 @@ message VideoPacket { // Time in milliseconds spent in encoding this video frame. optional int32 encode_time_ms = 8; + + // The most recent sequence number received from the client on the event + // channel. + optional int64 client_sequence_number = 9; } -- cgit v1.1