Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-08-04 21:20:23 +01:00
Code Issues Projects Releases Packages Wiki Activity

Make report_pending_exception unsafe

Browse source
This commit is contained in:
Anthony Ramine 2016-06-07 00:26:49 +02:00
parent bc52617d33
commit 27620320f3
3 changed files with 12 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

10
components/script/dom/bindings/error.rs
View file

@ -121,12 +121,10 @@ pub unsafe fn throw_dom_exception(cx: *mut JSContext, global: GlobalRef, result:
} }
/// Report a pending exception, thereby clearing it. /// Report a pending exception, thereby clearing it.
pub fn report_pending_exception(cx: *mut JSContext, obj: *mut JSObject) { pub unsafe fn report_pending_exception(cx: *mut JSContext, obj: *mut JSObject) {
unsafe { if JS_IsExceptionPending(cx) {
if JS_IsExceptionPending(cx) { let _ac = JSAutoCompartment::new(cx, obj);
let _ac = JSAutoCompartment::new(cx, obj); JS_ReportPendingException(cx);
JS_ReportPendingException(cx);
}
} }
} }

4
components/script/dom/eventtarget.rs
View file

@ -425,7 +425,9 @@ impl EventTarget {
}; };
if !rv || handler.ptr.is_null() { if !rv || handler.ptr.is_null() {
// Step 1.8.2 // Step 1.8.2
report_pending_exception(cx, self.reflector().get_jsobject().get()); unsafe {
report_pending_exception(cx, self.reflector().get_jsobject().get());
}
// Step 1.8.1 / 1.8.3 // Step 1.8.1 / 1.8.3
return None; return None;
} }

6
components/script/dom/workerglobalscope.rs
View file

@ -416,6 +416,7 @@ impl WorkerGlobalScopeMethods for WorkerGlobalScope {
impl WorkerGlobalScope { impl WorkerGlobalScope {
#[allow(unsafe_code)]
pub fn execute_script(&self, source: DOMString) { pub fn execute_script(&self, source: DOMString) {
let mut rval = RootedValue::new(self.runtime.cx(), UndefinedValue()); let mut rval = RootedValue::new(self.runtime.cx(), UndefinedValue());
match self.runtime.evaluate_script( match self.runtime.evaluate_script(
@ -428,7 +429,10 @@ impl WorkerGlobalScope {
// TODO: An error needs to be dispatched to the parent. // TODO: An error needs to be dispatched to the parent.
// https://github.com/servo/servo/issues/6422 // https://github.com/servo/servo/issues/6422
println!("evaluate_script failed"); println!("evaluate_script failed");
report_pending_exception(self.runtime.cx(), self.reflector().get_jsobject().get()); unsafe {
report_pending_exception(
self.runtime.cx(), self.reflector().get_jsobject().get());
}
} }
} }
} }