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

Unified Diff: webkit/plugins/ppapi/time_conversion.cc

Issue 7237044: Make PP_TimeTicks actually be a tick counter rather than be the same as the (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years, 5 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 | « webkit/plugins/ppapi/time_conversion.h ('k') | webkit/plugins/ppapi/time_conversion_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/plugins/ppapi/time_conversion.cc
===================================================================
--- webkit/plugins/ppapi/time_conversion.cc (revision 0)
+++ webkit/plugins/ppapi/time_conversion.cc (revision 0)
@@ -0,0 +1,51 @@
+// Copyright (c) 2011 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 "webkit/plugins/ppapi/time_conversion.h"
+
+namespace webkit {
+namespace ppapi {
+
+namespace {
+
+// Since WebKit doesn't use ticks for event times, we have to compute what
+// the time ticks would be assuming the wall clock time doesn't change.
+//
+// This should only be used for WebKit times which we can't change the
darin (slow to review) 2011/07/08 16:17:25 We might want to look into fixing this at some poi
+// definition of.
+double GetTimeToTimeTicksDeltaInSeconds() {
+ static double time_to_ticks_delta_seconds = 0.0;
+ if (time_to_ticks_delta_seconds == 0.0) {
+ double wall_clock = TimeToPPTime(base::Time::Now());
+ double ticks = TimeTicksToPPTimeTicks(base::TimeTicks::Now());
+ time_to_ticks_delta_seconds = ticks - wall_clock;
+ }
+ return time_to_ticks_delta_seconds;
+}
+
+} // namespace
+
+PP_Time TimeToPPTime(base::Time t) {
+ return t.ToDoubleT();
+}
+
+base::Time PPTimeToTime(PP_Time t) {
+ return base::Time::FromDoubleT(t);
+}
+
+PP_TimeTicks TimeTicksToPPTimeTicks(base::TimeTicks t) {
+ return static_cast<double>(t.ToInternalValue() /
+ base::Time::kMicrosecondsPerSecond);
+}
+
+PP_TimeTicks EventTimeToPPTimeTicks(double event_time) {
+ return event_time + GetTimeToTimeTicksDeltaInSeconds();
+}
+
+double PPTimeTicksToEventTime(PP_TimeTicks t) {
+ return t - GetTimeToTimeTicksDeltaInSeconds();
+}
+
+} // namespace ppapi
+} // namespace webkit
Property changes on: webkit/plugins/ppapi/time_conversion.cc
___________________________________________________________________
Added: svn:eol-style
+ LF
« no previous file with comments | « webkit/plugins/ppapi/time_conversion.h ('k') | webkit/plugins/ppapi/time_conversion_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698