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

Unified Diff: content/browser/media/session/media_session_uma_helper.cc

Issue 1918433003: Use TimeTicks for MediaSessionUmaHelper. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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
Index: content/browser/media/session/media_session_uma_helper.cc
diff --git a/content/browser/media/session/media_session_uma_helper.cc b/content/browser/media/session/media_session_uma_helper.cc
index 17a43b03a1ff8bcd401d0a2a4d1e7e6e6bd28221..57d03b20bc256840868282227a49307c1e03939b 100644
--- a/content/browser/media/session/media_session_uma_helper.cc
+++ b/content/browser/media/session/media_session_uma_helper.cc
@@ -8,14 +8,14 @@
#include "base/metrics/histogram_base.h"
#include "base/metrics/histogram_macros.h"
-#include "base/time/default_clock.h"
+#include "base/time/default_tick_clock.h"
namespace content {
using HistogramBase = base::HistogramBase;
MediaSessionUmaHelper::MediaSessionUmaHelper()
- : clock_(new base::DefaultClock())
+ : clock_(new base::DefaultTickClock())
{}
MediaSessionUmaHelper::~MediaSessionUmaHelper()
@@ -34,21 +34,21 @@ void MediaSessionUmaHelper::RecordRequestAudioFocusResult(bool result) const {
}
void MediaSessionUmaHelper::OnSessionActive() {
- current_active_time_ = clock_->Now();
+ current_active_time_ = clock_->NowTicks();
}
void MediaSessionUmaHelper::OnSessionSuspended() {
if (current_active_time_.is_null())
return;
- total_active_time_ += clock_->Now() - current_active_time_;
- current_active_time_ = base::Time();
+ total_active_time_ += clock_->NowTicks() - current_active_time_;
+ current_active_time_ = base::TimeTicks();
}
void MediaSessionUmaHelper::OnSessionInactive() {
if (!current_active_time_.is_null()) {
- total_active_time_ += clock_->Now() - current_active_time_;
- current_active_time_ = base::Time();
+ total_active_time_ += clock_->NowTicks() - current_active_time_;
+ current_active_time_ = base::TimeTicks();
}
if (total_active_time_.is_zero())
@@ -59,7 +59,7 @@ void MediaSessionUmaHelper::OnSessionInactive() {
}
void MediaSessionUmaHelper::SetClockForTest(
- std::unique_ptr<base::Clock> testing_clock) {
+ std::unique_ptr<base::TickClock> testing_clock) {
clock_ = std::move(testing_clock);
}

Powered by Google App Engine
This is Rietveld 408576698