Changeset 79240 in webkit for trunk/Source/JavaScriptCore/runtime/JSArray.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/JSArray.cpp (modified) (4 diffs)

trunk/Source/JavaScriptCore/runtime/JSArray.cpp

@@ -130 +130 @@
     : JSNonFinalObject(createStructure(jsNull()))
 {
+    ASSERT(inherits(&s_info));
+
     unsigned initialCapacity = 0;
 
@@ -147 +149 @@
     : JSNonFinalObject(structure)
 {
+    ASSERT(inherits(&s_info));
+
     unsigned initialCapacity = 0;
 
@@ -162 +166 @@
     : JSNonFinalObject(structure)
 {
+    ASSERT(inherits(&s_info));
+
     unsigned initialCapacity;
     if (creationMode == CreateCompact)
@@ -202 +208 @@
     : JSNonFinalObject(structure)
 {
+    ASSERT(inherits(&s_info));
+
     unsigned initialCapacity = list.size();
     unsigned initialStorage;