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

Side by Side Diff: remoting/test/connection_time_observer.cc

Issue 1238343002: Added ConnectionTimeObserver to calculate the times to authenticate and connect. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Moved implementation to connection_to_host_impl.cc and removed inline. Created 5 years, 4 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
(Empty)
1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "remoting/test/connection_time_observer.h"
6
7 #include <utility>
8
9 #include "base/strings/stringprintf.h"
10 #include "base/time/time.h"
11 #include "base/timer/timer.h"
12
13 namespace remoting {
14 namespace test {
15
16 ConnectionTimeObserver::ConnectionTimeObserver() {
17 }
18
19 ConnectionTimeObserver::~ConnectionTimeObserver() {
20 }
21
22 void ConnectionTimeObserver::SetTransitionTimesMapForTest(
23 const std::map<protocol::ConnectionToHost::State, base::TimeTicks>& map) {
24 transition_times_map_ = map;
25 }
26
27 void ConnectionTimeObserver::ConnectionStateChanged(
28 protocol::ConnectionToHost::State state,
29 protocol::ErrorCode error_code) {
30 if (transition_times_map_.find(state) != transition_times_map_.end()) {
31 std::string connection_state = protocol::ConnectionStateToString(state);
32 LOG(ERROR) << connection_state << " state has already been set";
33 return;
34 }
35 transition_times_map_.insert(std::make_pair(state, base::TimeTicks::Now()));
36 current_connection_state_ = state;
37 }
38
39 void ConnectionTimeObserver::DisplayConnectionStats() const {
40 protocol::ConnectionToHost::State initializing =
41 protocol::ConnectionToHost::State::INITIALIZING;
42 protocol::ConnectionToHost::State current_state = initializing;
43
44 const char kStateChangeTitleFormatString[] = "%-35s%-15s";
45 LOG(INFO) << base::StringPrintf(kStateChangeTitleFormatString,
46 "State to State", "Delta Time");
47 LOG(INFO) << base::StringPrintf(kStateChangeTitleFormatString,
48 "--------------", "----------");
49
50 // Note: the order of |connected_states| mimics the expected order of when a
51 // connection is made.
52 std::vector<protocol::ConnectionToHost::State> connected_states;
53 connected_states.push_back(protocol::ConnectionToHost::State::CONNECTING);
54 connected_states.push_back(protocol::ConnectionToHost::State::AUTHENTICATED);
55 connected_states.push_back(protocol::ConnectionToHost::State::CONNECTED);
56 connected_states.push_back(protocol::ConnectionToHost::State::FAILED);
57
58 const char kStateChangeFormatString[] = "%-13s to %-18s%-7dms";
59 auto iter_end = transition_times_map_.end();
60 for (protocol::ConnectionToHost::State state : connected_states) {
61 auto iter_state = transition_times_map_.find(state);
62 if (iter_state != iter_end) {
63 int state_transition_time =
64 GetStateTransitionTime(current_state, state).InMilliseconds();
65 LOG(INFO) << base::StringPrintf(kStateChangeFormatString,
66 protocol::ConnectionStateToString(current_state),
67 protocol::ConnectionStateToString(state),
68 state_transition_time);
69 current_state = state;
70 }
71 }
72
73 int connected_time =
74 GetStateTransitionTime(initializing, current_state).InMilliseconds();
75
76 // |current state| will either be FAILED or CONNECTED.
77 LOG(INFO) << "Total Connection Duration (INITIALIZING to "
78 << protocol::ConnectionStateToString(current_state) << "): "
79 << connected_time << " ms";
80 }
81
82 base::TimeDelta ConnectionTimeObserver::GetStateTransitionTime(
83 protocol::ConnectionToHost::State start,
84 protocol::ConnectionToHost::State end) const {
85 auto iter_end = transition_times_map_.end();
86
87 auto iter_start_state = transition_times_map_.find(start);
88 std::string start_state = protocol::ConnectionStateToString(start);
89 if (iter_start_state == iter_end) {
90 LOG(ERROR) << "No time found for state: " << start_state;
91 return base::TimeDelta::Max();
92 }
93
94 auto iter_end_state = transition_times_map_.find(end);
95 std::string end_state = protocol::ConnectionStateToString(end);
96 if (iter_end_state == iter_end) {
97 LOG(ERROR) << "No time found for state: " << end_state;
98 return base::TimeDelta::Max();
99 }
100
101 base::TimeDelta delta = iter_end_state->second - iter_start_state->second;
102 if (delta.InMilliseconds() < 0) {
103 LOG(ERROR) << "Transition delay is negative. Check the state ordering: "
104 << "[start: " << start_state << ", end: " << end_state << "]";
105 return base::TimeDelta::Max();
106 }
107
108 return delta;
109 }
110
111 } // namespace test
112 } // namespace remoting
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698