Changes between Version 8 and Version 9 of DOMInJavaScript

Timestamp:

Sep 27, 2012, 12:13:30 AM (12 years ago)

Author:

abarth@webkit.org

Comment:

--

DOMInJavaScript

@@ -16 +16 @@
 Node* Node::nextSibling()
 {
-    m_wrapper->fastGetProperty(kNextSiblingIndex)->impl();
+    m_wrapper->getAtKnownOffset(kNextSiblingOffset)->impl();
 }
 
 void Note::setNextSibling(Node* node)
 {
-    m_wrapper->fastSetProperty(kNextSiblingIndex, node->m_wrapper);
+    m_wrapper->setAtKnownOffset(kNextSiblingOffset, node->m_wrapper);
 }
 }}}
@@ -27 +27 @@
 In order to get or set the nextSibling property, the C++ code needs to consult its JavaScript wrapper. (Note: This implies that we'll need to eagerly create JavaScript wrappers for DOM nodes.) In this approach, the JavaScript wrapper stores the four DOM pointers at fixed offsets in memory, letting the C++ code read or write the property directly rather than having to do a hash table lookup.
 
-My understanding is that fastGetProperty is similar to JSC::JSObject::getDirectOffset. The key is to create JavaScript wrappers for DOM nodes in such a way that initial offsets are know to correspond to particular properties (e.g., the four basic DOM pointers.)
+My understanding is that getAtKnownOffset is similar to JSC::JSObject::getDirectOffset. The key is to create JavaScript wrappers for DOM nodes in such a way that initial offsets are know to correspond to particular properties (e.g., the four basic DOM pointers.)