Changeset 18461 in webkit for trunk/JavaScriptCore/bindings/runtime_root.cpp

Timestamp:

Dec 28, 2006, 5:44:28 PM (18 years ago)

Author:

ggaren

Message:

JavaScriptCore:

Reviewed by Brady Eidson.

Some cleanup in preparation for fixing <rdar://problem/4608404>
WebScriptObject's _executionContext lack of ownership policy causes
crashes (e.g., in Dashcode)

I'm just trying to make heads or tails of this baffling code.

Renamed "root" | "execContext" | "executionContext" => "rootObject", because
that's the object's (admittedly vague) type name.

• bindings/runtime.cpp: Removed createLanguageInstanceForValue because I'll give you a dollar if you can explain to me what it actually did.

• bindings/runtime_root.cpp: Put everything in the KJS::Bindings namespace, removing the KJS::Bindings prefix from individual functions and datatypes. This matches the header and eliminates a lot of syntax cruft.

• bindings/c/c_utility.cpp: (KJS::Bindings::convertValueToNPVariant): Replaced use of createLanguageInstanceForValue with call to _NPN_CreateScriptObject because that's what createLanguageInstanceForValue actually did (but don't ask me for that dollar now; that's cheating.)

• bindings/objc/objc_utility.h:
• bindings/objc/objc_utility.mm: (KJS::Bindings::convertValueToObjcValue): Removed. Its only purpose was to call a single function for WebKit, which WebKit can do on its own.

• kjs/interpreter.h: Removed rtti() because it was unused, and this class is scheduled for demolition anyway.

• kjs/interpreter.cpp: Removed createLanguageInstanceForValue because it had nothing to do with the Interpreter, and nothing makes Chuck Norris more mad than a function whose sole purpose is to call another function of the same name. (Really, I asked him.)

WebCore:

Reviewed by Brady Eidson.

Some cleanup in preparation for fixing <rdar://problem/4608404>
WebScriptObject's _executionContext lack of ownership policy causes
crashes (e.g., in Dashcode)

Layout tests pass.

Renamed "root" | "execContext" | "executionContext" => rootObject, because
that's the object's (admittedly vague) type name.

• bindings/js/kjs_binding.cpp:
• bindings/js/kjs_binding.h: Removed createLanguageInstanceForValue and createObjcInstanceForValue because their only purpose was to confuse you.

• bindings/objc/DOMInternal.h: Moved declaration of createDOMWrapper here. createDOMWrapper is the new name for createObjcInstanceForValue.

• bindings/objc/DOMInternal.mm: Renamed Interpreter::createObjcInstanceForValue to createDOMWrapper because creating DOM wrappers has nothing to do with the interpreter, and everything to do with the DOM. Renamed value to object because it is one. Removed newObj nil check that is unnecessary in ObjC.

• bindings/objc/WebScriptObject.mm: Replaced call to createLanguageInstanceForValue with explicit code to do the same thing it would have done: (1) try to create a DOM wrapper; (2) if the object is not a wrappable DOM object, create a vanilla WebScriptObject for it instead.

File:

• trunk/JavaScriptCore/bindings/runtime_root.cpp (modified) (11 diffs)

trunk/JavaScriptCore/bindings/runtime_root.cpp

@@ -29 +29 @@
 #include <wtf/HashCountedSet.h>
 
-using namespace KJS;
-using namespace KJS::Bindings;
+namespace KJS { namespace Bindings {
 
 // Java does NOT always call finalize (and thus KJS_JSObject_JSFinalize) when
@@ -46 +45 @@
 // are represented by a jlong.
 
-typedef HashMap<const Bindings::RootObject*, ReferencesSet*> ReferencesByRootMap; 
+typedef HashMap<const RootObject*, ReferencesSet*> ReferencesByRootMap; 
 
 static ReferencesByRootMap* getReferencesByRootMap()
@@ -58 +57 @@
 }
 
-static ReferencesSet* getReferencesSet(const Bindings::RootObject *root)
+static ReferencesSet* getReferencesSet(const RootObject* rootObject)
 {
     ReferencesByRootMap* refsByRoot = getReferencesByRootMap();
     ReferencesSet* referencesSet = 0;
     
-    referencesSet = refsByRoot->get(root);
+    referencesSet = refsByRoot->get(rootObject);
     if (!referencesSet) {
         referencesSet  = new ReferencesSet;
-        refsByRoot->add(root, referencesSet);
+        refsByRoot->add(rootObject, referencesSet);
     }
     return referencesSet;
@@ -76 +75 @@
 // FIXME:  This is a potential performance bottleneck with many applets.  We could fix be adding a
 // imp to root dictionary.
-ReferencesSet* KJS::Bindings::findReferenceSet(JSObject *imp)
+ReferencesSet* findReferenceSet(JSObject* imp)
 {
     ReferencesByRootMap* refsByRoot = getReferencesByRootMap ();
@@ -94 +93 @@
 // FIXME:  This is a potential performance bottleneck with many applets.  We could fix be adding a
 // imp to root dictionary.
-const Bindings::RootObject *KJS::Bindings::rootForImp (JSObject *imp)
+const RootObject* rootObjectForImp (JSObject* imp)
 {
     ReferencesByRootMap* refsByRoot = getReferencesByRootMap ();
-    const Bindings::RootObject *rootObject = 0;
+    const RootObject* rootObject = 0;
     
     if (refsByRoot) {
@@ -113 +112 @@
 }
 
-const Bindings::RootObject *KJS::Bindings::rootForInterpreter (KJS::Interpreter *interpreter)
+const RootObject* rootObjectForInterpreter(Interpreter* interpreter)
 {
     ReferencesByRootMap* refsByRoot = getReferencesByRootMap ();
@@ -120 +119 @@
         ReferencesByRootMap::const_iterator end = refsByRoot->end();
         for (ReferencesByRootMap::const_iterator it = refsByRoot->begin(); it != end; ++it) {
-            const Bindings::RootObject* aRootObject = it->first;
+            const RootObject* aRootObject = it->first;
             
             if (aRootObject->interpreter() == interpreter)
@@ -130 +129 @@
 }
 
-void KJS::Bindings::addNativeReference (const Bindings::RootObject *root, JSObject *imp)
-{
-    if (root) {
-        ReferencesSet* referenceMap = getReferencesSet (root);
+void addNativeReference(const RootObject* rootObject, JSObject* imp)
+{
+    if (rootObject) {
+        ReferencesSet* referenceMap = getReferencesSet(rootObject);
         
         unsigned numReferences = referenceMap->count(imp);
@@ -144 +143 @@
 }
 
-void KJS::Bindings::removeNativeReference (JSObject *imp)
+void removeNativeReference(JSObject* imp)
 {
     if (!imp)
@@ -280 +279 @@
     // thread that a JavaScript call needs to be invoked.
     CFRunLoopSourceContext sourceContext = {0, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, performJavaScriptAccess};
-    Bindings::RootObject::_performJavaScriptSource = CFRunLoopSourceCreate(NULL, 0, &sourceContext);
-    CFRunLoopAddSource(Bindings::RootObject::_runLoop, Bindings::RootObject::_performJavaScriptSource, kCFRunLoopDefaultMode);
+    RootObject::_performJavaScriptSource = CFRunLoopSourceCreate(NULL, 0, &sourceContext);
+    CFRunLoopAddSource(RootObject::_runLoop, RootObject::_performJavaScriptSource, kCFRunLoopDefaultMode);
 }
 #endif
@@ -303 +302 @@
 }
 
-void RootObject::setInterpreter (KJS::Interpreter *i)
-{
-    _interpreter = i;
-}
-
-
+void RootObject::setInterpreter (Interpreter* interpreter)
+{
+    _interpreter = interpreter;
+}
+
+} } // namespace KJS::Bindings