Accept transfer argument for StructuredCloneData::write

components/script/dom/bindings/structuredclone.rs

@@ -255,7 +255,11 @@ pub enum StructuredCloneData {
 impl StructuredCloneData {
     // TODO: should this be unsafe?
     /// Writes a structured clone. Returns a `DataClone` error if that fails.
-    pub fn write(cx: *mut JSContext, message: HandleValue) -> Fallible<StructuredCloneData> {
+    pub fn write(
+        cx: *mut JSContext,
+        message: HandleValue,
+        transfer: HandleValue,
+    ) -> Fallible<StructuredCloneData> {
         unsafe {
             let scbuf = NewJSAutoStructuredCloneBuffer(
                 StructuredCloneScope::DifferentProcess,
@@ -274,7 +278,7 @@ impl StructuredCloneData {
                 policy,
                 &STRUCTURED_CLONE_CALLBACKS,
                 ptr::null_mut(),
-                HandleValue::undefined(),
+                transfer,
             );
             if !result {
                 JS_ClearPendingException(cx);

components/script/dom/dedicatedworkerglobalscope.rs

@@ -560,7 +560,8 @@ impl DedicatedWorkerGlobalScopeMethods for DedicatedWorkerGlobalScope {
     #[allow(unsafe_code)]
     // https://html.spec.whatwg.org/multipage/#dom-dedicatedworkerglobalscope-postmessage
     unsafe fn PostMessage(&self, cx: *mut JSContext, message: HandleValue) -> ErrorResult {
-        let data = StructuredCloneData::write(cx, message)?;
+        rooted!(in(cx) let transfer = UndefinedValue());
+        let data = StructuredCloneData::write(cx, message, transfer.handle())?;
         let worker = self.worker.borrow().as_ref().unwrap().clone();
         let pipeline_id = self.upcast::<GlobalScope>().pipeline_id();
         let task = Box::new(task!(post_worker_message: move || {

components/script/dom/dissimilaroriginwindow.rs

@@ -154,7 +154,8 @@ impl DissimilarOriginWindowMethods for DissimilarOriginWindow {
 
         // Step 1-2, 6-8.
         // TODO(#12717): Should implement the `transfer` argument.
-        let data = StructuredCloneData::write(cx, message)?;
+        rooted!(in(cx) let transfer = UndefinedValue());
+        let data = StructuredCloneData::write(cx, message, transfer.handle())?;
 
         // Step 9.
         self.post_message(origin, data);

components/script/dom/history.rs

@@ -184,7 +184,8 @@ impl History {
         // TODO: Step 4
 
         // Step 5
-        let serialized_data = StructuredCloneData::write(cx, data)?.move_to_arraybuffer();
+        rooted!(in(cx) let transfer = UndefinedValue());
+        let serialized_data = StructuredCloneData::write(cx, data, transfer.handle())?.move_to_arraybuffer();
 
         let new_url: ServoUrl = match url {
             // Step 6

components/script/dom/serviceworker.rs

@@ -19,6 +19,7 @@ use crate::dom::globalscope::GlobalScope;
 use crate::task::TaskOnce;
 use dom_struct::dom_struct;
 use js::jsapi::JSContext;
+use js::jsval::UndefinedValue;
 use js::rust::HandleValue;
 use script_traits::{DOMMessage, ScriptMsg};
 use servo_url::ServoUrl;
@@ -98,7 +99,8 @@ impl ServiceWorkerMethods for ServiceWorker {
             return Err(Error::InvalidState);
         }
         // Step 7
-        let data = StructuredCloneData::write(cx, message)?;
+        rooted!(in(cx) let transfer = UndefinedValue());
+        let data = StructuredCloneData::write(cx, message, transfer.handle())?;
         let msg_vec = DOMMessage(data.move_to_arraybuffer());
         let _ = self
             .global()

components/script/dom/window.rs

@@ -877,7 +877,8 @@ impl WindowMethods for Window {
 
         // Step 1-2, 6-8.
         // TODO(#12717): Should implement the `transfer` argument.
-        let data = StructuredCloneData::write(cx, message)?;
+        rooted!(in(cx) let transfer = UndefinedValue());
+        let data = StructuredCloneData::write(cx, message, transfer.handle())?;
 
         // Step 9.
         self.post_message(origin, &*source.window_proxy(), data);

components/script/dom/worker.rs

@@ -160,7 +160,8 @@ impl WorkerMethods for Worker {
     #[allow(unsafe_code)]
     // https://html.spec.whatwg.org/multipage/#dom-worker-postmessage
     unsafe fn PostMessage(&self, cx: *mut JSContext, message: HandleValue) -> ErrorResult {
-        let data = StructuredCloneData::write(cx, message)?;
+        rooted!(in(cx) let transfer = UndefinedValue());
+        let data = StructuredCloneData::write(cx, message, transfer.handle())?;
         let address = Trusted::new(self);
 
         // NOTE: step 9 of https://html.spec.whatwg.org/multipage/#dom-messageport-postmessage