Changeset 79240 in webkit for trunk/Source/JavaScriptCore/runtime/StringObject.cpp

Timestamp:

Feb 21, 2011, 11:31:42 AM (14 years ago)

Author:

[email protected]

Message:

Bug 54894 - Make inheritance structure described by ClassInfo match C++ class hierarchy.

Reviewed by Sam Weinig.

The ClassInfo objects describe an inheritance hierarchy, with each ClassInfo instance
containing a pointer to its parent class. These links should reflect the inheritance
hierarchy of C++ classes below JSObject. For the large part it does, but in some cases
entries in the C++ hierarchy are skipped over. This presently likely doesn't matter,
since intervening C++ classes may not have ClassInfo - but would be a potential bug
were ClassInfo were to be added.

Source/JavaScriptCore:

• API/JSCallbackConstructor.cpp:
• API/JSCallbackFunction.cpp:
• API/JSCallbackObjectFunctions.h:
• runtime/Arguments.h:
• runtime/ArrayPrototype.cpp:
• runtime/BooleanObject.cpp:
• runtime/DateInstance.cpp:
• runtime/DatePrototype.cpp:
• runtime/ErrorInstance.cpp:
• runtime/InternalFunction.cpp:
• runtime/JSActivation.cpp:
• runtime/JSArray.cpp:
• runtime/JSFunction.cpp:
• runtime/JSONObject.cpp:
• runtime/JSObject.h:
• runtime/JSZombie.h:
• runtime/MathObject.cpp:
• runtime/NativeErrorConstructor.cpp:
• runtime/NumberConstructor.cpp:
• runtime/NumberObject.cpp:
• runtime/RegExpConstructor.cpp:
• runtime/RegExpObject.cpp:
• runtime/StringObject.cpp:
• runtime/StringPrototype.cpp:

Source/WebCore:

• bindings/js/JSAudioConstructor.cpp:
• bindings/js/JSDOMGlobalObject.cpp:
• bindings/js/JSImageConstructor.cpp:
• bindings/js/JSOptionConstructor.cpp:
• bindings/scripts/CodeGeneratorJS.pm:
• bridge/objc/objc_runtime.mm:
• bridge/runtime_object.cpp:

Source/WebKit2:

• WebProcess/Plugins/Netscape/JSNPObject.cpp:

File:

• trunk/Source/JavaScriptCore/runtime/StringObject.cpp (modified) (3 diffs)

trunk/Source/JavaScriptCore/runtime/StringObject.cpp

@@ -33 +33 @@
     : JSWrapperObject(exec->globalData(), structure)
 {
+    ASSERT(inherits(&s_info));
     setInternalValue(exec->globalData(), jsEmptyString(exec));
 }
@@ -39 +40 @@
     : JSWrapperObject(globalData, structure)
 {
+    ASSERT(inherits(&s_info));
     setInternalValue(globalData, string);
 }
@@ -45 +47 @@
     : JSWrapperObject(exec->globalData(), structure)
 {
+    ASSERT(inherits(&s_info));
     setInternalValue(exec->globalData(), jsString(exec, string));
 }