From a69c5da2083bdb6e80ad5f89ff44a4288b6a7024 Mon Sep 17 00:00:00 2001 From: Anthony Ramine Date: Fri, 22 Sep 2017 14:08:25 +0200 Subject: [PATCH] Make Promise::new_rejected unsafe --- components/script/dom/promise.rs | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/components/script/dom/promise.rs b/components/script/dom/promise.rs index e038d9ff4ca..45aff52f8fe 100644 --- a/components/script/dom/promise.rs +++ b/components/script/dom/promise.rs @@ -126,17 +126,15 @@ impl Promise { } #[allow(unrooted_must_root, unsafe_code)] - pub fn new_rejected( + pub unsafe fn new_rejected( global: &GlobalScope, cx: *mut JSContext, value: HandleValue, ) -> Fallible> { let _ac = JSAutoCompartment::new(cx, global.reflector().get_jsobject().get()); - rooted!(in(cx) let p = unsafe { CallOriginalPromiseReject(cx, value) }); + rooted!(in(cx) let p = CallOriginalPromiseReject(cx, value)); assert!(!p.handle().is_null()); - unsafe { - Ok(Promise::new_with_js_promise(p.handle(), cx)) - } + Ok(Promise::new_with_js_promise(p.handle(), cx)) } #[allow(unsafe_code)]