Remove SelectorEngine.matches(). (#32339)

It's basically unused.
twbs · Dec 7, 2020 · d15a024 · d15a024
1 parent 33b275c
commit d15a024
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 14 deletions.
diff --git a/js/src/dom/selector-engine.js b/js/src/dom/selector-engine.js
@@ -14,10 +14,6 @@
 const NODE_TEXT = 3
 
 const SelectorEngine = {
-  matches(element, selector) {
-    return element.matches(selector)
-  },
-
   find(selector, element = document.documentElement) {
     return [].concat(...Element.prototype.querySelectorAll.call(element, selector))
   },
@@ -38,7 +34,7 @@ const SelectorEngine = {
     let ancestor = element.parentNode
 
     while (ancestor && ancestor.nodeType === Node.ELEMENT_NODE && ancestor.nodeType !== NODE_TEXT) {
-      if (this.matches(ancestor, selector)) {
+      if (ancestor.matches(selector)) {
         parents.push(ancestor)
       }
 
@@ -66,7 +62,7 @@ const SelectorEngine = {
     let next = element.nextElementSibling
 
     while (next) {
-      if (this.matches(next, selector)) {
+      if (next.matches(selector)) {
         return [next]
       }
 

diff --git a/js/tests/unit/dom/selector-engine.spec.js b/js/tests/unit/dom/selector-engine.spec.js
@@ -14,14 +14,6 @@ describe('SelectorEngine', () => {
     clearFixture()
   })
 
-  describe('matches', () => {
-    it('should return matched elements', () => {
-      fixtureEl.innerHTML = '<div></div>'
-
-      expect(SelectorEngine.matches(fixtureEl, 'div')).toEqual(true)
-    })
-  })
-
   describe('find', () => {
     it('should find elements', () => {
       fixtureEl.innerHTML = '<div></div>'