aboutsummaryrefslogtreecommitdiffstats
path: root/src/components/script/dom/bindings/utils.rs
diff options
context:
space:
mode:
authorBobby Holley <bobbyholley@gmail.com>2013-10-09 09:53:23 +0200
committerBobby Holley <bobbyholley@gmail.com>2013-10-09 12:15:56 +0200
commit549ac6a29b3707451b1f48d30a80d06ac31ccf27 (patch)
treee416702201403f260ee1052973485db40ea6966d /src/components/script/dom/bindings/utils.rs
parent622aa76515854f76d81635f312f6d989d369b4e1 (diff)
downloadservo-549ac6a29b3707451b1f48d30a80d06ac31ccf27.tar.gz
servo-549ac6a29b3707451b1f48d30a80d06ac31ccf27.zip
Rename some local variables and shorten some lines.
This was done manually, and might benefit from a more careful review.
Diffstat (limited to 'src/components/script/dom/bindings/utils.rs')
-rw-r--r--src/components/script/dom/bindings/utils.rs19
1 files changed, 9 insertions, 10 deletions
diff --git a/src/components/script/dom/bindings/utils.rs b/src/components/script/dom/bindings/utils.rs
index b03fc5f89c3..56b28298a07 100644
--- a/src/components/script/dom/bindings/utils.rs
+++ b/src/components/script/dom/bindings/utils.rs
@@ -561,8 +561,8 @@ pub fn WrapNewBindingObject(cx: *JSContext, scope: *JSObject,
value: @mut Reflectable,
vp: *mut JSVal) -> JSBool {
unsafe {
- let cache = value.reflector();
- let obj = cache.get_jsobject();
+ let reflector = value.reflector();
+ let obj = reflector.get_jsobject();
if obj.is_not_null() /*&& js::GetObjectCompartment(obj) == js::GetObjectCompartment(scope)*/ {
*vp = RUST_OBJECT_TO_JSVAL(obj);
return 1; // JS_TRUE
@@ -574,7 +574,7 @@ pub fn WrapNewBindingObject(cx: *JSContext, scope: *JSObject,
}
// MOZ_ASSERT(js::IsObjectInContextCompartment(scope, cx));
- cache.set_jsobject(obj);
+ reflector.set_jsobject(obj);
*vp = RUST_OBJECT_TO_JSVAL(obj);
return JS_WrapValue(cx, cast::transmute(vp));
}
@@ -584,14 +584,13 @@ pub fn WrapNewBindingObject(cx: *JSContext, scope: *JSObject,
pub fn WrapNativeParent(cx: *JSContext, scope: *JSObject, mut p: Option<@mut Reflectable>) -> *JSObject {
match p {
Some(ref mut p) => {
- let cache = p.reflector();
- let wrapper = cache.get_jsobject();
- if wrapper.is_not_null() {
- return wrapper;
+ let obj = p.reflector().get_jsobject();
+ if obj.is_not_null() {
+ return obj;
}
- let wrapper = p.wrap_object_shared(cx, scope);
- cache.set_jsobject(wrapper);
- wrapper
+ let obj = p.wrap_object_shared(cx, scope);
+ p.reflector().set_jsobject(obj);
+ obj
}
None => unsafe { JS_GetGlobalObject(cx) }
}