Remove `Manipulator.toggleClass` (#31842)

It's only used in one place so it makes more sense to remove it for the time being.

2 files changed, 4 insertions, 16 deletions

diff --git a/js/src/dom/manipulator.js b/js/src/dom/manipulator.js
index a81354e78..750ee115e 100644
--- a/js/src/dom/manipulator.js
+++ b/js/src/dom/manipulator.js
@@ -72,18 +72,6 @@ const Manipulator = {
       top: element.offsetTop,
       left: element.offsetLeft
     }
-  },
-
-  toggleClass(element, className) {
-    if (!element) {
-      return
-    }
-
-    if (element.classList.contains(className)) {
-      element.classList.remove(className)
-    } else {
-      element.classList.add(className)
-    }
   }
 }
 
diff --git a/js/src/dropdown.js b/js/src/dropdown.js
index 37bdd1f24..28d8b7299 100644
--- a/js/src/dropdown.js
+++ b/js/src/dropdown.js
@@ -197,8 +197,8 @@ class Dropdown {
     this._element.focus()
     this._element.setAttribute('aria-expanded', true)
 
-    Manipulator.toggleClass(this._menu, CLASS_NAME_SHOW)
-    Manipulator.toggleClass(this._element, CLASS_NAME_SHOW)
+    this._menu.classList.toggle(CLASS_NAME_SHOW)
+    this._element.classList.toggle(CLASS_NAME_SHOW)
     EventHandler.trigger(parent, EVENT_SHOWN, relatedTarget)
   }
 
@@ -222,8 +222,8 @@ class Dropdown {
       this._popper.destroy()
     }
 
-    Manipulator.toggleClass(this._menu, CLASS_NAME_SHOW)
-    Manipulator.toggleClass(this._element, CLASS_NAME_SHOW)
+    this._menu.classList.toggle(CLASS_NAME_SHOW)
+    this._element.classList.toggle(CLASS_NAME_SHOW)
     EventHandler.trigger(parent, EVENT_HIDDEN, relatedTarget)
   }