Make cc::animation::TrimTimeToCurrentIteration,cc::AnimationCurve::Duration use TimeTicks/TimeDelta

cc/base/time_util.h

+// Copyright 2014 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.
+
+#ifndef CC_BASE_TIME_UTIL_H_
+#define CC_BASE_TIME_UTIL_H_
+
+namespace base {
+class TimeDelta;
+}
+
+namespace cc {
+
+base::TimeDelta operator*(const base::TimeDelta time_delta, double value) {

[ajuma, 2014/11/06 18:03:23]

Rather than overloading operator* (which will be confusing, since then the
meaning of operator* will change depending on whether this .h file is
#included), we should just define a separate function for this. e.g. define a
TimeUtil class, with a static Scale(base::TimeDelta time_delta, double
scale_factor). That should be all that we need; to multiply time_delta by d,
call Scale(time_delta, d), and to divide by d call Scale(time_delta, 1.0/d).

[patro, 2014/11/07 07:03:40]

Done.

+  return base::TimeDelta::FromInternalValue(time_delta.ToInternalValue() *

[ajuma, 2014/11/06 18:03:23]

Make the double/int64 conversions explicit, e.g:

base::TimeDelta::FromInternalValue(static_cast<int64>(static_cast<double>(time_delta.ToInternalValue()
* value)));

[ajuma, 2014/11/06 18:07:20]

Sorry, messed up the parentheses there. I meant:
base::TimeDelta::FromInternalValue(static_cast<int64>(static_cast<double>(time_delta.ToInternalValue())
* value))

[patro, 2014/11/07 07:03:40]

Done.

+                                            value);
+}
+
+base::TimeDelta operator*(double value, const base::TimeDelta time_delta) {
+  return base::TimeDelta::FromInternalValue(time_delta.ToInternalValue() *
+                                            value);
+}
+
+base::TimeDelta operator/(const base::TimeDelta time_delta, double value) {
+  return base::TimeDelta::FromInternalValue(time_delta.ToInternalValue() /
+                                            value);
+}
+
+base::TimeDelta operator/(double value, const base::TimeDelta time_delta) {
+  return base::TimeDelta::FromInternalValue(time_delta.ToInternalValue() /
+                                            value);
+}
+
+}  // namespace cc
+
+#endif  // CC_BASE_TIME_UTIL_H_