Bug 1571908 - part4 : remove redudant function. r=heycam
authorAlastor Wu <alwu@mozilla.com>
Wed, 21 Aug 2019 07:10:26 +0000
changeset 489131 9b44d777f09f409725518a507be7a18a7bdd8ed3
parent 489130 d98a9a24c1708016cd84faa1a47979c94c655e91
child 489132 62ef7c15917a09e8194ecea263f2fcb2a334aa80
push id36465
push userdvarga@mozilla.com
push dateWed, 21 Aug 2019 16:47:43 +0000
treeherdermozilla-central@4ab60925635c [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersheycam
bugs1571908
milestone70.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 1571908 - part4 : remove redudant function. r=heycam We have already had a exactly same function, so can remove a redundant one. Differential Revision: https://phabricator.services.mozilla.com/D41132
dom/media/webvtt/vtt.jsm
--- a/dom/media/webvtt/vtt.jsm
+++ b/dom/media/webvtt/vtt.jsm
@@ -867,26 +867,16 @@ XPCOMUtils.defineLazyPreferenceGetter(th
       for (let i = 0; i < boxes.length; i++) {
         if (this.overlaps(boxes[i])) {
           return true;
         }
       }
       return false;
     }
 
-    // Check if this box overlaps any other boxes in boxes.
-    overlapsAny(boxes) {
-      for (let i = 0; i < boxes.length; i++) {
-        if (this.overlaps(boxes[i])) {
-          return true;
-        }
-      }
-      return false;
-    }
-
     // Check if this box is within another box.
     within(container) {
       return (this.top >= container.top - this.fuzz) &&
              (this.bottom <= container.bottom + this.fuzz) &&
              (this.left >= container.left - this.fuzz) &&
              (this.right <= container.right + this.fuzz);
     }