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

Unified Diff: media/base/clock.cc

Issue 11607003: Add a Clock and TickClock interface for mocking out time (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix warnings Created 7 years, 10 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 | « media/base/clock.h ('k') | media/base/clock_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/base/clock.cc
diff --git a/media/base/clock.cc b/media/base/clock.cc
index 2432f91e47bf75418de23a12faffde85ccf38215..733c5fe28ff705d6d424314e2027ac76e71816ca 100644
--- a/media/base/clock.cc
+++ b/media/base/clock.cc
@@ -7,12 +7,13 @@
#include <algorithm>
#include "base/logging.h"
+#include "base/time/clock.h"
#include "media/base/buffers.h"
namespace media {
-Clock::Clock(TimeProvider* time_provider)
- : time_provider_(time_provider) {
+Clock::Clock(base::Clock* clock) : clock_(clock) {
+ DCHECK(clock_);
Reset();
}
@@ -95,12 +96,6 @@ base::TimeDelta Clock::ElapsedViaProvidedTime(const base::Time& time) const {
return media_time_ + base::TimeDelta::FromMicroseconds(now_us);
}
-base::Time Clock::GetTimeFromProvider() const {
- if (time_provider_)
- return time_provider_();
- return base::Time();
-}
-
base::TimeDelta Clock::ClampToValidTimeRange(base::TimeDelta time) const {
if (duration_ == kNoTimestamp())
return base::TimeDelta();
@@ -124,12 +119,11 @@ void Clock::UpdateReferencePoints() {
void Clock::UpdateReferencePoints(base::TimeDelta current_time) {
media_time_ = ClampToValidTimeRange(current_time);
- reference_ = GetTimeFromProvider();
+ reference_ = clock_->Now();
}
base::TimeDelta Clock::EstimatedElapsedTime() {
- return ClampToValidTimeRange(
- ElapsedViaProvidedTime(GetTimeFromProvider()));
+ return ClampToValidTimeRange(ElapsedViaProvidedTime(clock_->Now()));
}
void Clock::Reset() {
« no previous file with comments | « media/base/clock.h ('k') | media/base/clock_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698