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

Unified Diff: remoting/client/client_telemetry_logger.cc

Issue 2753963002: Refactoring and rewriting the chromoting jni instance to be chromoting session. (Closed)
Patch Set: Updating based on 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « remoting/client/client_telemetry_logger.h ('k') | remoting/client/connect_to_host_info.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/client/client_telemetry_logger.cc
diff --git a/remoting/client/client_telemetry_logger.cc b/remoting/client/client_telemetry_logger.cc
index 40f218778548fc60477026272476c426d5df4adf..908a527373e31e144489d56debcc0a614192711e 100644
--- a/remoting/client/client_telemetry_logger.cc
+++ b/remoting/client/client_telemetry_logger.cc
@@ -4,10 +4,16 @@
#include "remoting/client/client_telemetry_logger.h"
+#include "base/format_macros.h"
#include "base/logging.h"
#include "base/rand_util.h"
+#include "base/strings/stringprintf.h"
#include "remoting/base/telemetry_log_writer.h"
+#if defined(OS_ANDROID)
+#include <android/log.h>
+#endif // OS_ANDROID
+
namespace {
const char kSessionIdAlphabet[] =
@@ -62,10 +68,36 @@ void ClientTelemetryLogger::LogStatistics(
DCHECK(thread_checker_.CalledOnValidThread());
RefreshSessionIdIfOutdated();
+ PrintLogStatistics(perf_tracker);
+
ChromotingEvent event = MakeStatsEvent(perf_tracker);
log_writer_->Log(event);
}
+void ClientTelemetryLogger::PrintLogStatistics(
+ protocol::PerformanceTracker* perf_tracker) {
+#if defined(OS_ANDROID)
+ __android_log_print(
+ ANDROID_LOG_INFO, "stats",
+#else
+ VLOG(1) << base::StringPrintf(
+#endif // OS_ANDROID
+ "Bandwidth:%.0f FrameRate:%.1f;"
+ " (Avg, Max) Capture:%.1f, %" PRId64 " Encode:%.1f, %" PRId64
+ " Decode:%.1f, %" PRId64 " Render:%.1f, %" PRId64 " RTL:%.0f, %" PRId64,
+ perf_tracker->video_bandwidth(), perf_tracker->video_frame_rate(),
+ perf_tracker->video_capture_ms().Average(),
+ perf_tracker->video_capture_ms().Max(),
+ perf_tracker->video_encode_ms().Average(),
+ perf_tracker->video_encode_ms().Max(),
+ perf_tracker->video_decode_ms().Average(),
+ perf_tracker->video_decode_ms().Max(),
+ perf_tracker->video_paint_ms().Average(),
+ perf_tracker->video_paint_ms().Max(),
+ perf_tracker->round_trip_ms().Average(),
+ perf_tracker->round_trip_ms().Max());
+}
+
void ClientTelemetryLogger::SetSessionIdGenerationTimeForTest(
base::TimeTicks gen_time) {
session_id_generation_time_ = gen_time;
« no previous file with comments | « remoting/client/client_telemetry_logger.h ('k') | remoting/client/connect_to_host_info.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698