Start marking functions that can transitively trigger a GC (#33144)

* Mark JS reflector wrappers as CanGc.

Signed-off-by: Josh Matthews <josh@joshmatthews.net>

* Propagate CanGc from reflect_dom_object_with_proto.

Signed-off-by: Josh Matthews <josh@joshmatthews.net>

* Mark DOM constructors as GC operations.

Signed-off-by: Josh Matthews <josh@joshmatthews.net>

---------

Signed-off-by: Josh Matthews <josh@joshmatthews.net>

1 files changed, 16 insertions, 2 deletions

diff --git a/components/script/dom/securitypolicyviolationevent.rs b/components/script/dom/securitypolicyviolationevent.rs
index edbaf7d04dc..4826b75e17c 100644
--- a/components/script/dom/securitypolicyviolationevent.rs
+++ b/components/script/dom/securitypolicyviolationevent.rs
@@ -17,6 +17,7 @@ use crate::dom::bindings::root::DomRoot;
 use crate::dom::bindings::str::{DOMString, USVString};
 use crate::dom::event::{Event, EventBubbles, EventCancelable};
 use crate::dom::globalscope::GlobalScope;
+use crate::script_runtime::CanGc;
 
 // https://w3c.github.io/webappsec-csp/#securitypolicyviolationevent
 #[dom_struct]
@@ -59,11 +60,13 @@ impl SecurityPolicyViolationEvent {
         global: &GlobalScope,
         init: &SecurityPolicyViolationEventInit,
         proto: Option<HandleObject>,
+        can_gc: CanGc,
     ) -> DomRoot<SecurityPolicyViolationEvent> {
         reflect_dom_object_with_proto(
             Box::new(SecurityPolicyViolationEvent::new_inherited(init)),
             global,
             proto,
+            can_gc,
         )
     }
 
@@ -74,8 +77,9 @@ impl SecurityPolicyViolationEvent {
         bubbles: EventBubbles,
         cancelable: EventCancelable,
         init: &SecurityPolicyViolationEventInit,
+        can_gc: CanGc,
     ) -> DomRoot<Self> {
-        let ev = SecurityPolicyViolationEvent::new_initialized(global, init, proto);
+        let ev = SecurityPolicyViolationEvent::new_initialized(global, init, proto, can_gc);
         {
             let event = ev.upcast::<Event>();
             event.init_event(type_, bool::from(bubbles), bool::from(cancelable));
@@ -90,13 +94,22 @@ impl SecurityPolicyViolationEvent {
         cancelable: EventCancelable,
         init: &SecurityPolicyViolationEventInit,
     ) -> DomRoot<Self> {
-        Self::new_with_proto(global, None, type_, bubbles, cancelable, init)
+        Self::new_with_proto(
+            global,
+            None,
+            type_,
+            bubbles,
+            cancelable,
+            init,
+            CanGc::note(),
+        )
     }
 
     #[allow(non_snake_case)]
     pub fn Constructor(
         global: &GlobalScope,
         proto: Option<HandleObject>,
+        can_gc: CanGc,
         type_: DOMString,
         init: &SecurityPolicyViolationEventInit,
     ) -> DomRoot<Self> {
@@ -107,6 +120,7 @@ impl SecurityPolicyViolationEvent {
             EventBubbles::from(init.parent.bubbles),
             EventCancelable::from(init.parent.cancelable),
             init,
+            can_gc,
         )
     }
 }