From 518729a4f53123e3a62b3f132052408a703e17e0 Mon Sep 17 00:00:00 2001
From: Josh Matthews <josh@joshmatthews.net>
Date: Fri, 20 Jun 2025 03:42:36 -0400
Subject: [PATCH] script: Expose IDBVersionChangeEvent to worker globals.
 (#37573)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

IDBVersionChangeEvent is already used internally from non-Window
globals, but the constructor wasn't set up to expose the interface
object in worker globals.

Testing: The effort involved in adding a test for this does not seem
worthwhile—we get this for free once we enable the indexeddb preference
while our interfaces.worker.js test is running.
Fixes: part of #6963

Signed-off-by: Josh Matthews <josh@joshmatthews.net>
---
 components/script/dom/idbversionchangeevent.rs   | 16 ++++++++++------
 .../webidls/IDBVersionChangeEvent.webidl         |  4 +---
 2 files changed, 11 insertions(+), 9 deletions(-)

diff --git a/components/script/dom/idbversionchangeevent.rs b/components/script/dom/idbversionchangeevent.rs
index f338a31ca11..6de90a4af6c 100644
--- a/components/script/dom/idbversionchangeevent.rs
+++ b/components/script/dom/idbversionchangeevent.rs
@@ -11,12 +11,11 @@ use crate::dom::bindings::codegen::Bindings::IDBVersionChangeEventBinding::{
 };
 use crate::dom::bindings::import::module::HandleObject;
 use crate::dom::bindings::inheritance::Castable;
-use crate::dom::bindings::reflector::{DomGlobal, reflect_dom_object};
+use crate::dom::bindings::reflector::reflect_dom_object_with_proto;
 use crate::dom::bindings::root::DomRoot;
 use crate::dom::bindings::str::DOMString;
 use crate::dom::event::{Event, EventBubbles, EventCancelable};
 use crate::dom::globalscope::GlobalScope;
-use crate::dom::window::Window;
 use crate::script_runtime::CanGc;
 
 #[dom_struct]
@@ -46,6 +45,7 @@ impl IDBVersionChangeEvent {
     ) -> DomRoot<IDBVersionChangeEvent> {
         Self::new_with_proto(
             global,
+            None,
             type_,
             bool::from(bubbles),
             bool::from(cancelable),
@@ -55,8 +55,10 @@ impl IDBVersionChangeEvent {
         )
     }
 
+    #[allow(clippy::too_many_arguments)]
     fn new_with_proto(
         global: &GlobalScope,
+        proto: Option<HandleObject>,
         type_: Atom,
         bubbles: bool,
         cancelable: bool,
@@ -64,12 +66,13 @@ impl IDBVersionChangeEvent {
         new_version: Option<u64>,
         can_gc: CanGc,
     ) -> DomRoot<Self> {
-        let ev = reflect_dom_object(
+        let ev = reflect_dom_object_with_proto(
             Box::new(IDBVersionChangeEvent::new_inherited(
                 old_version,
                 new_version,
             )),
             global,
+            proto,
             can_gc,
         );
         {
@@ -83,14 +86,15 @@ impl IDBVersionChangeEvent {
 impl IDBVersionChangeEventMethods<crate::DomTypeHolder> for IDBVersionChangeEvent {
     /// <https://w3c.github.io/IndexedDB/#dom-idbversionchangeevent-idbversionchangeevent>
     fn Constructor(
-        window: &Window,
-        _proto: Option<HandleObject>,
+        global: &GlobalScope,
+        proto: Option<HandleObject>,
         can_gc: CanGc,
         type_: DOMString,
         init: &IDBVersionChangeEventInit,
     ) -> DomRoot<Self> {
         Self::new_with_proto(
-            &window.global(),
+            global,
+            proto,
             Atom::from(type_),
             init.parent.bubbles,
             init.parent.cancelable,
diff --git a/components/script_bindings/webidls/IDBVersionChangeEvent.webidl b/components/script_bindings/webidls/IDBVersionChangeEvent.webidl
index f8a6623d5ee..04de19aede2 100644
--- a/components/script_bindings/webidls/IDBVersionChangeEvent.webidl
+++ b/components/script_bindings/webidls/IDBVersionChangeEvent.webidl
@@ -7,10 +7,8 @@
  *
  */
 
-// FIXME:(arihant2math) Expose to Worker too
-
 // https://w3c.github.io/IndexedDB/#idbversionchangeevent
-[Pref="dom_indexeddb_enabled", Exposed=(Window)]
+[Pref="dom_indexeddb_enabled", Exposed=(Window,Worker)]
 interface IDBVersionChangeEvent : Event {
   constructor(DOMString type, optional IDBVersionChangeEventInit eventInitDict = {});