core: Don't make a object for the item menu callback, and set default to Undefined

core/src/avm1/globals/context_menu_item.rs

@@ -20,7 +20,7 @@ pub fn constructor<'gc>(
         .get(0)
         .unwrap_or(&Value::Undefined)
         .coerce_to_string(activation)?;
-    let callback = args.get(1).map(|v| v.coerce_to_object(activation));
+    let callback = *args.get(1).unwrap_or(&Value::Undefined);
     let separator_before = args
         .get(2)
         .unwrap_or(&false.into())
@@ -35,11 +35,7 @@ pub fn constructor<'gc>(
         .as_bool(activation.swf_version());
 
     this.set("caption", caption.into(), activation)?;
-
-    if let Some(callback) = callback {
-        this.set("onSelect", callback.into(), activation)?;
-    }
-
+    this.set("onSelect", callback, activation)?;
     this.set("separatorBefore", separator_before.into(), activation)?;
     this.set("enabled", enabled.into(), activation)?;
     this.set("visible", visible.into(), activation)?;

tests/tests/swfs/avm1/context_menu_item_callback/output.txt

@@ -0,0 +1,12 @@
+undefined
+undefined
+[type Function]
+undefined
+[type Function]
+true
+true
+true
+true
+true
+ok
+ok

tests/tests/swfs/avm1/context_menu_item_callback/test.toml

@@ -0,0 +1 @@
+num_frames = 1