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

Unified Diff: ui/events/fraction_of_time_without_user_input_recorder.cc

Issue 2751403002: Record the fraction of the time without user input - Aura only (Closed)
Patch Set: Address sadrul@'s feedback. Created 3 years, 9 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
Index: ui/events/fraction_of_time_without_user_input_recorder.cc
diff --git a/ui/events/fraction_of_time_without_user_input_recorder.cc b/ui/events/fraction_of_time_without_user_input_recorder.cc
new file mode 100644
index 0000000000000000000000000000000000000000..de07a510d57a8f5340a229800adfa079aadb8e6c
--- /dev/null
+++ b/ui/events/fraction_of_time_without_user_input_recorder.cc
@@ -0,0 +1,79 @@
+// Copyright 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "ui/events/fraction_of_time_without_user_input_recorder.h"
+
+#include <algorithm>
+
+#include "base/metrics/histogram_macros.h"
+
+namespace {
+
+const base::TimeDelta DEFAULT_WINDOW_SIZE = base::TimeDelta::FromSecondsD(10);
+const base::TimeDelta DEFAULT_IDLE_TIMEOUT =
+ base::TimeDelta::FromSecondsD(0.05);
+
+} // namespace
+
+namespace ui {
+
+FractionOfTimeWithoutUserInputRecorder::FractionOfTimeWithoutUserInputRecorder()
+ : window_size_(DEFAULT_WINDOW_SIZE), idle_timeout_(DEFAULT_IDLE_TIMEOUT) {}
+
+void FractionOfTimeWithoutUserInputRecorder::RecordEventAtTime(
+ base::TimeTicks start_time) {
+ base::TimeTicks event_end_time = start_time + idle_timeout_;
+
+ if (active_duration_start_time_.is_null())
+ active_duration_start_time_ = start_time;
+ if (previous_event_end_time_.is_null())
+ previous_event_end_time_ = start_time;
+
+ // The user is no longer interacting with the browser. Report the previous
+ // active duration.
+ if (previous_event_end_time_ < start_time) {
+ RecordActiveInterval(active_duration_start_time_, previous_event_end_time_);
+ active_duration_start_time_ = start_time;
+ }
+
+ previous_event_end_time_ = event_end_time;
+}
+
+void FractionOfTimeWithoutUserInputRecorder::RecordActiveInterval(
+ base::TimeTicks start_time,
+ base::TimeTicks end_time) {
+ if (window_start_time_.is_null())
+ window_start_time_ = start_time;
+
+ base::TimeTicks window_end_time;
+
+ while (true) {
+ window_end_time = window_start_time_ + window_size_;
+ base::TimeDelta interval_in_window_duration =
+ std::min(end_time, window_end_time) -
+ std::max(start_time, window_start_time_);
+ interval_in_window_duration =
+ std::max(interval_in_window_duration, base::TimeDelta());
+
+ current_window_active_time_ += interval_in_window_duration;
+
+ // If we haven't exceeded the window bounds, we're done.
+ if (end_time < window_end_time)
+ break;
+
+ RecordToUma(current_window_active_time_.InMillisecondsF() /
+ window_size_.InMillisecondsF());
+
+ current_window_active_time_ = base::TimeDelta();
+ window_start_time_ = window_end_time;
+ }
+}
+
+void FractionOfTimeWithoutUserInputRecorder::RecordToUma(
+ float fraction_active) const {
+ UMA_HISTOGRAM_PERCENTAGE("Event.FractionOfTimeWithoutUserInput",
+ std::round((1 - fraction_active) * 100));
+}
+
+} // namespace ui

Powered by Google App Engine
This is Rietveld 408576698