Switch DOM bindings to using inlined GetObjectSlot

servo · Oct 12, 2012 · 8391e3b · 8391e3b
1 parent bdf480c
commit 8391e3b
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 6 deletions.
diff --git a/src/rust-mozjs b/src/rust-mozjs
diff --git a/src/servo/dom/bindings/node.rs b/src/servo/dom/bindings/node.rs
@@ -74,7 +74,7 @@ fn NodeBundle(n: Node, s: NodeScope) -> NodeBundle {
 }
 
 unsafe fn unwrap(obj: *JSObject) -> *rust_box<NodeBundle> {
-    let val = JS_GetReservedSlot(obj, 0);
+    let val = js::GetReservedSlot(obj, 0);
     cast::reinterpret_cast(&JSVAL_TO_PRIVATE(val))
 }
 

diff --git a/src/test/test_getter_time.js b/src/test/test_getter_time.js
@@ -1,12 +1,14 @@
 var elem = document.documentElement.firstChild;
 
-var start = (new Date()).getTime();
-for (var i = 0; i < 1000000; i++) {
+var start = new Date()
+var count = 1000000;
+for (var i = 0; i < count; i++) {
   var a = elem.nodeType;
 }
-window.alert((new Date()).getTime() - start);
+var stop = new Date()
+window.alert((stop - start) / count * 1e6);
 
 /*start = new Date().getTime();
 for (i = 0; i < 10000; i++)
   elem.width = i;
-window.alert(new Date().getTime() - start);*/
+window.alert(new Date().getTime() - start);*/