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

Unified Diff: webrtc/system_wrappers/source/tick_util.cc

Issue 1639543005: Use rtc::time for all your timing needs! (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Error on unsupported platform Created 4 years, 11 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 | « webrtc/base/timeutils.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/system_wrappers/source/tick_util.cc
diff --git a/webrtc/system_wrappers/source/tick_util.cc b/webrtc/system_wrappers/source/tick_util.cc
index c02b5314a4f3b0e634b14e95b66d6c7b0f71b738..0485e429211a114f48ebe6900a8c558850531f63 100644
--- a/webrtc/system_wrappers/source/tick_util.cc
+++ b/webrtc/system_wrappers/source/tick_util.cc
@@ -10,7 +10,7 @@
#include "webrtc/system_wrappers/include/tick_util.h"
-#include <assert.h>
+#include "webrtc/base/timeutils.h"
namespace webrtc {
@@ -23,108 +23,20 @@ int64_t TickTime::MicrosecondTimestamp() {
}
int64_t TickTime::MillisecondsToTicks(const int64_t ms) {
-#if _WIN32
- return ms;
-#elif defined(WEBRTC_LINUX)
- return ms * 1000000LL;
-#elif defined(WEBRTC_MAC)
- // TODO(pbos): Fix unsafe use of static locals.
- static double timebase_from_millisecond_fract = 0.0;
- if (timebase_from_millisecond_fract == 0.0) {
- mach_timebase_info_data_t timebase;
- (void)mach_timebase_info(&timebase);
- timebase_from_millisecond_fract = (timebase.denom * 1e6) / timebase.numer;
- }
- return ms * timebase_from_millisecond_fract;
-#else
- return ms * 1000LL;
-#endif
+ return ms * rtc::kNumNanosecsPerMillisec;
}
int64_t TickTime::TicksToMilliseconds(const int64_t ticks) {
-#if _WIN32
- return ticks;
-#elif defined(WEBRTC_LINUX)
- return ticks / 1000000LL;
-#elif defined(WEBRTC_MAC)
- // TODO(pbos): Fix unsafe use of static locals.
- static double timebase_microsecond_fract = 0.0;
- if (timebase_microsecond_fract == 0.0) {
- mach_timebase_info_data_t timebase;
- (void)mach_timebase_info(&timebase);
- timebase_microsecond_fract = timebase.numer / (timebase.denom * 1e6);
- }
- return ticks * timebase_microsecond_fract;
-#else
- return ticks;
-#endif
+ return ticks / rtc::kNumNanosecsPerMillisec;
}
int64_t TickTime::TicksToMicroseconds(const int64_t ticks) {
-#if _WIN32
- return ticks * 1000LL;
-#elif defined(WEBRTC_LINUX)
- return ticks / 1000LL;
-#elif defined(WEBRTC_MAC)
- // TODO(pbos): Fix unsafe use of static locals.
- static double timebase_microsecond_fract = 0.0;
- if (timebase_microsecond_fract == 0.0) {
- mach_timebase_info_data_t timebase;
- (void)mach_timebase_info(&timebase);
- timebase_microsecond_fract = timebase.numer / (timebase.denom * 1e3);
- }
- return ticks * timebase_microsecond_fract;
-#else
- return ticks;
-#endif
+ return ticks / rtc::kNumNanosecsPerMicrosec;
}
-// Gets the native system tick count. The actual unit, resolution, and epoch
-// varies by platform:
-// Windows: Milliseconds of uptime with rollover count in the upper 32-bits.
-// Linux/Android: Nanoseconds since the Unix epoch.
-// Mach (Mac/iOS): "absolute" time since first call.
-// Unknown POSIX: Microseconds since the Unix epoch.
+// Gets the native system tick count, converted to nanoseconds.
int64_t TickTime::QueryOsForTicks() {
-#if _WIN32
- static volatile LONG last_time_get_time = 0;
- static volatile int64_t num_wrap_time_get_time = 0;
- volatile LONG* last_time_get_time_ptr = &last_time_get_time;
- DWORD now = timeGetTime();
- // Atomically update the last gotten time
- DWORD old = InterlockedExchange(last_time_get_time_ptr, now);
- if (now < old) {
- // If now is earlier than old, there may have been a race between
- // threads.
- // 0x0fffffff ~3.1 days, the code will not take that long to execute
- // so it must have been a wrap around.
- if (old > 0xf0000000 && now < 0x0fffffff) {
- // TODO(pbos): Fix unsafe use of static locals.
- num_wrap_time_get_time++;
- }
- }
- return now + (num_wrap_time_get_time << 32);
-#elif defined(WEBRTC_LINUX)
- struct timespec ts;
- clock_gettime(CLOCK_MONOTONIC, &ts);
- return 1000000000LL * ts.tv_sec + ts.tv_nsec;
-#elif defined(WEBRTC_MAC)
- // Return absolute time as an offset from the first call to this function, so
- // that we can do floating-point (double) operations on it without losing
- // precision. This holds true until the elapsed time is ~11 days,
- // at which point we'll start to lose some precision, though not enough to
- // matter for millisecond accuracy for another couple years after that.
- // TODO(pbos): Fix unsafe use of static locals.
- static uint64_t timebase_start = 0;
- if (timebase_start == 0) {
- timebase_start = mach_absolute_time();
- }
- return mach_absolute_time() - timebase_start;
-#else
- struct timeval tv;
- gettimeofday(&tv, NULL);
- return 1000000LL * tv.tv_sec + tv.tv_usec;
-#endif
+ return rtc::TimeNanos();
}
} // namespace webrtc
« no previous file with comments | « webrtc/base/timeutils.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698