From 259dd0058c19efe07c68e881c4a29ed3ee05eb2c Mon Sep 17 00:00:00 2001
From: Jonas Jenwald <jonas.jenwald@gmail.com>
Date: Fri, 11 Oct 2024 12:01:47 +0200
Subject: [PATCH] Remove event listeners with `AbortSignal` in the
 `AltTextManager` class

---
 web/alt_text_manager.js | 56 ++++++++++++++++++++++-------------------
 1 file changed, 30 insertions(+), 26 deletions(-)

diff --git a/web/alt_text_manager.js b/web/alt_text_manager.js
index b487fadd034a37..dc3fc0c3aa2b82 100644
--- a/web/alt_text_manager.js
+++ b/web/alt_text_manager.js
@@ -13,14 +13,10 @@
  * limitations under the License.
  */
 
-import { DOMSVGFactory, shadow } from "pdfjs-lib";
+import { DOMSVGFactory } from "pdfjs-lib";
 
 class AltTextManager {
-  #boundUpdateUIState = this.#updateUIState.bind(this);
-
-  #boundSetPosition = this.#setPosition.bind(this);
-
-  #boundOnClick = this.#onClick.bind(this);
+  #clickAC = null;
 
   #currentEditor = null;
 
@@ -46,6 +42,8 @@ class AltTextManager {
 
   #previousAltText = null;
 
+  #resizeAC = null;
+
   #svgElement = null;
 
   #rectElement = null;
@@ -77,6 +75,8 @@ class AltTextManager {
     this.#eventBus = eventBus;
     this.#container = container;
 
+    const onUpdateUIState = this.#updateUIState.bind(this);
+
     dialog.addEventListener("close", this.#close.bind(this));
     dialog.addEventListener("contextmenu", event => {
       if (event.target !== this.#textarea) {
@@ -85,22 +85,12 @@ class AltTextManager {
     });
     cancelButton.addEventListener("click", this.#finish.bind(this));
     saveButton.addEventListener("click", this.#save.bind(this));
-    optionDescription.addEventListener("change", this.#boundUpdateUIState);
-    optionDecorative.addEventListener("change", this.#boundUpdateUIState);
+    optionDescription.addEventListener("change", onUpdateUIState);
+    optionDecorative.addEventListener("change", onUpdateUIState);
 
     this.#overlayManager.register(dialog);
   }
 
-  get _elements() {
-    return shadow(this, "_elements", [
-      this.#optionDescription,
-      this.#optionDecorative,
-      this.#textarea,
-      this.#saveButton,
-      this.#cancelButton,
-    ]);
-  }
-
   #createSVGElement() {
     if (this.#svgElement) {
       return;
@@ -138,12 +128,22 @@ class AltTextManager {
     if (this.#currentEditor || !editor) {
       return;
     }
-
     this.#createSVGElement();
 
     this.#hasUsedPointer = false;
-    for (const element of this._elements) {
-      element.addEventListener("click", this.#boundOnClick);
+
+    this.#clickAC = new AbortController();
+    const clickOpts = { signal: this.#clickAC.signal },
+      onClick = this.#onClick.bind(this);
+
+    for (const element of [
+      this.#optionDescription,
+      this.#optionDecorative,
+      this.#textarea,
+      this.#saveButton,
+      this.#cancelButton,
+    ]) {
+      element.addEventListener("click", onClick, clickOpts);
     }
 
     const { altText, decorative } = editor.altTextData;
@@ -160,7 +160,11 @@ class AltTextManager {
     this.#currentEditor = editor;
     this.#uiManager = uiManager;
     this.#uiManager.removeEditListeners();
-    this.#eventBus._on("resize", this.#boundSetPosition);
+
+    this.#resizeAC = new AbortController();
+    this.#eventBus._on("resize", this.#setPosition.bind(this), {
+      signal: this.#resizeAC.signal,
+    });
 
     try {
       await this.#overlayManager.open(this.#dialog);
@@ -258,7 +262,8 @@ class AltTextManager {
 
     this.#removeOnClickListeners();
     this.#uiManager?.addEditListeners();
-    this.#eventBus._off("resize", this.#boundSetPosition);
+    this.#resizeAC?.abort();
+    this.#resizeAC = null;
     this.#currentEditor.altTextFinish();
     this.#currentEditor = null;
     this.#uiManager = null;
@@ -295,9 +300,8 @@ class AltTextManager {
   }
 
   #removeOnClickListeners() {
-    for (const element of this._elements) {
-      element.removeEventListener("click", this.#boundOnClick);
-    }
+    this.#clickAC?.abort();
+    this.#clickAC = null;
   }
 
   destroy() {