Bug 1297580: P1. Remove broken optimisation in IntervalSet. r=gerald
authorJean-Yves Avenard <jyavenard@mozilla.com>
Wed, 24 Aug 2016 17:17:25 +1200
changeset 310961 3fe525095a250cc051a7e899f3728b83603b617c
parent 310960 20f2ba4d124729b45733548b58575ce10170abb7
child 310962 33e9d496b8be64a95a4841840287563a6412093f
push id31703
push userjyavenard@mozilla.com
push dateThu, 25 Aug 2016 10:43:31 +0000
treeherderautoland@c84540c8e42d [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersgerald
bugs1297580
milestone51.0a1
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
Bug 1297580: P1. Remove broken optimisation in IntervalSet. r=gerald Interval.LeftOf would always return true if the length of the interval is less than the fuzzing value. MozReview-Commit-ID: L9uyZjQXnGA
dom/media/Intervals.h
--- a/dom/media/Intervals.h
+++ b/dom/media/Intervals.h
@@ -569,20 +569,16 @@ public:
       return mIntervals[aIndex].mEnd;
     } else {
       return T();
     }
   }
 
   bool Contains(const ElemType& aInterval) const {
     for (const auto& interval : mIntervals) {
-      if (aInterval.LeftOf(interval)) {
-        // Will never succeed.
-        return false;
-      }
       if (interval.Contains(aInterval)) {
         return true;
       }
     }
     return false;
   }
 
   bool Contains(const T& aX) const {