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

Side by Side Diff: remoting/client/chromoting_stats.cc

Issue 8310001: Show framerate in chromoting debug log (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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 #include "remoting/client/chromoting_stats.h" 5 #include "remoting/client/chromoting_stats.h"
6 6
7 namespace { 7 namespace {
8 8
9 // The default window of bandwidth in seconds. Bandwidth is reported as 9 // The default window of bandwidth in seconds. Bandwidth is reported as
10 // number of numbers received in this time frame. 10 // number of bytes received in this time frame.
11 static const int kBandwidthWindow = 3; 11 static const int kBandwidthWindow = 3;
12 12
13 // Framerate is reported as number of frames received over the following
14 // time window.
Wez 2011/10/15 21:21:39 nit: Specify the units.
Jamie 2011/10/15 22:38:27 Or use a TimeDelta object. Not sure if that's allo
15 static const int kFramerateWindow = 3;
16
13 // We take the last 10 latency numbers and report the average. 17 // We take the last 10 latency numbers and report the average.
14 static const int kLatencyWindow = 10; 18 static const int kLatencyWindow = 10;
15 19
16 } // namespace 20 } // namespace
17 21
18 namespace remoting { 22 namespace remoting {
19 23
20 ChromotingStats::ChromotingStats() 24 ChromotingStats::ChromotingStats()
21 : video_bandwidth_(base::TimeDelta::FromSeconds(kBandwidthWindow)), 25 : video_bandwidth_(base::TimeDelta::FromSeconds(kBandwidthWindow)),
26 video_frame_rate_(base::TimeDelta::FromSeconds(kFramerateWindow)),
22 video_capture_ms_(kLatencyWindow), 27 video_capture_ms_(kLatencyWindow),
23 video_encode_ms_(kLatencyWindow), 28 video_encode_ms_(kLatencyWindow),
24 video_decode_ms_(kLatencyWindow), 29 video_decode_ms_(kLatencyWindow),
25 video_paint_ms_(kLatencyWindow), 30 video_paint_ms_(kLatencyWindow),
26 round_trip_ms_(kLatencyWindow) { 31 round_trip_ms_(kLatencyWindow) {
27 } 32 }
28 33
29 ChromotingStats::~ChromotingStats() { 34 ChromotingStats::~ChromotingStats() {
30 } 35 }
31 36
32 } // namespace remoting 37 } // namespace remoting
OLDNEW
« no previous file with comments | « remoting/client/chromoting_stats.h ('k') | remoting/client/plugin/chromoting_scriptable_object.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698