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

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: fixed tests Created 5 years, 2 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
« no previous file with comments | « remoting/host/video_frame_pump.cc ('k') | remoting/protocol/connection_to_client.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 from when the last event was received until capturing has started.
76 optional int64 capture_pending_time_ms = 14;
77
78 // Total overhead time for IPC and threading when capturing frames.
79 optional int64 capture_overhead_time_ms = 15;
80
81 // Time between when the frame was captured and when encoder started encoding
82 // it.
83 optional int64 encode_pending_time_ms = 16;
84
85 // Time for which the frame is blocked until it's sent to the client.
86 optional int64 send_pending_time_ms = 17;
74 } 87 }
75 88
76 // VideoAck acknowledges that the frame in the VideoPacket with the same 89 // 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 90 // 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 91 // that have frame_id field set. They must be sent the same order in which
79 // the corresponding VideoPackets were received. 92 // the corresponding VideoPackets were received.
80 message VideoAck { 93 message VideoAck {
81 // Frame ID. 94 // Frame ID.
82 optional int32 frame_id = 1; 95 optional int32 frame_id = 1;
83 } 96 }
OLDNEW
« no previous file with comments | « remoting/host/video_frame_pump.cc ('k') | remoting/protocol/connection_to_client.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698