Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(519)

Side by Side Diff: remoting/proto/video.proto

Issue 1365663003: Add UMA histograms for more detailed latency tracking on the CRD host. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // Protocol for video messages. 5 // Protocol for video messages.
6 6
7 syntax = "proto2"; 7 syntax = "proto2";
8 8
9 option optimize_for = LITE_RUNTIME; 9 option optimize_for = LITE_RUNTIME;
10 10
(...skipping 53 matching lines...) Expand 10 before | Expand all | Expand 10 after
64 // True when |desktop_shape_rects| should be used, false to indicate that 64 // True when |desktop_shape_rects| should be used, false to indicate that
65 // the frame is un-shaped. If the shape has not changed since the preceding 65 // the frame is un-shaped. If the shape has not changed since the preceding
66 // frame then this field should be omitted. 66 // frame then this field should be omitted.
67 optional bool use_desktop_shape = 11; 67 optional bool use_desktop_shape = 11;
68 68
69 // Optional frame timestamp. Used in tests to estimate frame latency. 69 // Optional frame timestamp. Used in tests to estimate frame latency.
70 optional int64 timestamp = 12; 70 optional int64 timestamp = 12;
71 71
72 // Frame identifier used to match VideoFrame and VideoAck. 72 // Frame identifier used to match VideoFrame and VideoAck.
73 optional int32 frame_id = 13; 73 optional int32 frame_id = 13;
74
75 // Time passed from when the last event was received until capturing has
anandc 2015/09/25 19:21:45 "Time between when the" etc., or just remove "pass
Sergey Ulanov 2015/09/25 20:35:26 Done.
76 // started.
77 optional int64 capture_pending_time_ms = 14;
78
79 // Total overhead time for IPC and threading when capturing frames.
80 optional int64 capture_overhead_time_ms = 15;
81
82 // Time passed between when the frame was captured and when encoder started
anandc 2015/09/25 19:21:45 Remove "passed".
Sergey Ulanov 2015/09/25 20:35:26 Done.
83 // encoding it.
84 optional int64 encode_pending_time_ms = 16;
85
86 // Total latency for the event being blocked until it's sent to the client.
anandc 2015/09/25 19:21:45 "Time for which the event is blocked until it's se
Sergey Ulanov 2015/09/25 20:35:26 Done. It's also frame, not event
87 optional int64 send_pending_time_ms = 17;
74 } 88 }
75 89
76 // VideoAck acknowledges that the frame in the VideoPacket with the same 90 // VideoAck acknowledges that the frame in the VideoPacket with the same
77 // frame_id has been rendered. VideoAck messages must be sent only for frames 91 // frame_id has been rendered. VideoAck messages must be sent only for frames
78 // that have frame_id field set. They must be sent the same order in which 92 // that have frame_id field set. They must be sent the same order in which
79 // the corresponding VideoPackets were received. 93 // the corresponding VideoPackets were received.
80 message VideoAck { 94 message VideoAck {
81 // Frame ID. 95 // Frame ID.
82 optional int32 frame_id = 1; 96 optional int32 frame_id = 1;
83 } 97 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698