Changeset 34946 in webkit for trunk/JavaScriptCore/kjs/JSVariableObject.h

Timestamp:

Jul 1, 2008, 11:35:03 PM (17 years ago)

Author:

[email protected]

Message:

Reviewed by Darin.

​https://bugs.webkit.org/show_bug.cgi?id=19834
Failed assertion in JavaScriptCore/VM/SegmentedVector.h:82

Creating a global object with a custom prototype resets it twice (wasteful!).
So, addStaticGlobals() was called twice, but JSGlobalObject::reset() didn't reset
the register array.

• kjs/JSGlobalObject.cpp: (KJS::JSGlobalObject::reset): Call setRegisterArray(0, 0).

• kjs/JSVariableObject.h: Changed registerArray to OwnArrayPtr. Also, added private copy constructor and operator= to ensure that no one attempts to copy this object (for whatever reason, I couldn't make Noncopyable work).

• kjs/JSGlobalObject.h: (KJS::JSGlobalObject::addStaticGlobals): Allocate registerArray with new[].

• kjs/JSVariableObject.cpp: (KJS::JSVariableObject::copyRegisterArray): Allocate registerArray with new[]. (KJS::JSVariableObject::setRegisterArray): Avoid hitting an assertion in OwnArrayPtr when "changing" the value from 0 to 0.

File:

• trunk/JavaScriptCore/kjs/JSVariableObject.h (modified) (2 diffs)

trunk/JavaScriptCore/kjs/JSVariableObject.h

@@ -34 +34 @@
 #include "SymbolTable.h"
 #include "UnusedParam.h"
-#include <wtf/OwnPtr.h>
+#include <wtf/OwnArrayPtr.h>
 #include <wtf/UnusedParam.h>
 
@@ -70 +70 @@
                 ASSERT(symbolTable_);
             }
-            
+
             SymbolTable* symbolTable; // Maps name -> offset from "r" in register file.
             Register* registers; // Pointers to the register past the end of local storage. (Local storage indexes are negative.)
-            OwnPtr<Register> registerArray; // Independent copy of registers, used when a variable object copies its registers out of the register file.
+            OwnArrayPtr<Register> registerArray; // Independent copy of registers, used when a variable object copies its registers out of the register file.
             size_t registerArraySize;
+
+        private:
+            JSVariableObjectData(const JSVariableObjectData&);
+            JSVariableObjectData& operator=(const JSVariableObjectData&);
         };