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

Side by Side Diff: remoting/client/client_telemetry_logger.h

Issue 2753963002: Refactoring and rewriting the chromoting jni instance to be chromoting session. (Closed)
Patch Set: Fixed feedback. Created 3 years, 8 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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 #ifndef REMOTING_CLIENT_CLIENT_TELEMETRY_LOGGER_H_ 5 #ifndef REMOTING_CLIENT_CLIENT_TELEMETRY_LOGGER_H_
6 #define REMOTING_CLIENT_CLIENT_TELEMETRY_LOGGER_H_ 6 #define REMOTING_CLIENT_CLIENT_TELEMETRY_LOGGER_H_
7 7
8 #include <memory> 8 #include <memory>
9 #include <string> 9 #include <string>
10 10
(...skipping 41 matching lines...) Expand 10 before | Expand all | Expand 10 after
52 protocol::ErrorCode state); 52 protocol::ErrorCode state);
53 53
54 private: 54 private:
55 struct HostInfo; 55 struct HostInfo;
56 56
57 void FillEventContext(ChromotingEvent* event) const; 57 void FillEventContext(ChromotingEvent* event) const;
58 58
59 // Generates a new random session ID. 59 // Generates a new random session ID.
60 void GenerateSessionId(); 60 void GenerateSessionId();
61 61
62 // Print Log Statistics.
Lambros 2017/04/05 01:31:19 Remove comment, or document where the stats get "p
nicholss 2017/04/05 17:24:26 Done.
63 void PrintLogStatistics(protocol::PerformanceTracker* perf_tracker);
64
62 // If not session ID has been set, simply generates a new one without sending 65 // If not session ID has been set, simply generates a new one without sending
63 // any logs, otherwise expire the session ID if the maximum duration has been 66 // any logs, otherwise expire the session ID if the maximum duration has been
64 // exceeded, and sends SessionIdOld and SessionIdNew events describing the 67 // exceeded, and sends SessionIdOld and SessionIdNew events describing the
65 // change of id. 68 // change of id.
66 void RefreshSessionIdIfOutdated(); 69 void RefreshSessionIdIfOutdated();
67 70
68 ChromotingEvent MakeStatsEvent(protocol::PerformanceTracker* perf_tracker); 71 ChromotingEvent MakeStatsEvent(protocol::PerformanceTracker* perf_tracker);
69 ChromotingEvent MakeSessionStateChangeEvent( 72 ChromotingEvent MakeSessionStateChangeEvent(
70 ChromotingEvent::SessionState state, 73 ChromotingEvent::SessionState state,
71 ChromotingEvent::ConnectionError error); 74 ChromotingEvent::ConnectionError error);
(...skipping 16 matching lines...) Expand all
88 ChromotingEventLogWriter* log_writer_; 91 ChromotingEventLogWriter* log_writer_;
89 92
90 base::ThreadChecker thread_checker_; 93 base::ThreadChecker thread_checker_;
91 94
92 DISALLOW_COPY_AND_ASSIGN(ClientTelemetryLogger); 95 DISALLOW_COPY_AND_ASSIGN(ClientTelemetryLogger);
93 }; 96 };
94 97
95 } // namespace remoting 98 } // namespace remoting
96 99
97 #endif // REMOTING_CLIENT_CLIENT_TELEMETRY_LOGGER_H_ 100 #endif // REMOTING_CLIENT_CLIENT_TELEMETRY_LOGGER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698