Backed out changeset 002714e2ccba (bug 1222866)
authorCarsten "Tomcat" Book <cbook@mozilla.com>
Fri, 20 Nov 2015 16:39:37 +0100
changeset 273467 f1f5133ac4d1b66d6040230cce9943ed6dea3ea3
parent 273466 827100d647e37fbc3aa7b04fe8e3e60601805c84
child 273468 cc124f809d4a5b7bc7a7d50ec127acf61c4fe737
push id29703
push usercbook@mozilla.com
push dateFri, 20 Nov 2015 15:43:05 +0000
treeherdermozilla-central@0b2b0570777f [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs1222866
milestone45.0a1
backs out002714e2ccbac1d3bd2a73f86a3fc2eef277d1a0
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Backed out changeset 002714e2ccba (bug 1222866)
dom/media/TimeUnits.h
--- a/dom/media/TimeUnits.h
+++ b/dom/media/TimeUnits.h
@@ -88,19 +88,17 @@ struct Microseconds {
 class TimeUnit final {
 public:
   static TimeUnit FromSeconds(double aValue) {
     MOZ_ASSERT(!IsNaN(aValue));
 
     if (mozilla::IsInfinite<double>(aValue)) {
       return FromInfinity();
     }
-    // Due to internal double representation, this
-    // operation is not commutative, do not attempt to simplify.
-    double val = (aValue + .0000005) * USECS_PER_S;
+    double val = aValue * USECS_PER_S;
     if (val >= double(INT64_MAX)) {
       return FromMicroseconds(INT64_MAX);
     } else if (val <= double(INT64_MIN)) {
       return FromMicroseconds(INT64_MIN);
     } else {
       return FromMicroseconds(int64_t(val));
     }
   }