[Modern Media Controls] remove window event listeners at deinitialization

https://bugs.webkit.org/show_bug.cgi?id=280503

Reviewed by Ryan Reno.

DOMWindow keeps media controls object alive because of installed event listeners (see
JSDOMWindow::visitAdditionalChildren). This results in extra memory usage undil DOMWindow cleared (on top level
navigation).

Original patch by Eugene Mutavchi <[email protected]>.

* Source/WebCore/Modules/modern-media-controls/controls/media-controls.js:
(MediaControls.prototype.reenable):
* Source/WebCore/Modules/modern-media-controls/media/media-controller.js:
(MediaController.prototype.deinitialize):
(MediaController.prototype.reinitialize):

Canonical link: https://commits.webkit.org/284440@main

Author: calvaris

Source/WebCore/Modules/modern-media-controls/controls/media-controls.js

@@ -254,4 +254,10 @@ class MediaControls extends LayoutNode
         this.element.removeEventListener("focusin", this);
         window.removeEventListener("dragstart", this, true);
     }
+
+    reenable()
+    {
+        this.element.addEventListener("focusin", this);
+        window.addEventListener("dragstart", this, true);
+    }
 }

Source/WebCore/Modules/modern-media-controls/media/media-controller.js

@@ -222,6 +222,9 @@ class MediaController
     deinitialize()
     {
         this.shadowRoot.removeChild(this.container);
+        window.removeEventListener("keydown", this);
+        if (this.controls)
+            this.controls.disable();
         return true;
     }
 
@@ -232,6 +235,9 @@ class MediaController
         this.mediaWeakRef = new WeakRef(media);
         this.host = host;
         shadowRoot.appendChild(this.container);
+        window.addEventListener("keydown", this);
+        if (this.controls)
+            this.controls.reenable();
         return true;
     }