|
| 1 | +2011-09-05 Oliver Hunt <oliver@apple.com> |
| 2 | + |
| 3 | + An object's structure should reference the global object responsible for its creation |
| 4 | + https://bugs.webkit.org/show_bug.cgi?id=67624 |
| 5 | + |
| 6 | + Reviewed by Gavin Barraclough. |
| 7 | + |
| 8 | + Add a reference to a GlobalObject to Structure, and update all calls to |
| 9 | + Structure::create() to pass the global object that is the origin for that |
| 10 | + structure. For objects where the appropriate global object isn't available |
| 11 | + at construction time (global object prototypes, etc), or objects that |
| 12 | + logically don't have a global object (strings, etc) we just pass null. |
| 13 | + |
| 14 | + This change is largely mechanical (passing a new globalObject parameter |
| 15 | + around). |
| 16 | + |
| 17 | + * API/JSCallbackConstructor.h: |
| 18 | + (JSC::JSCallbackConstructor::createStructure): |
| 19 | + * API/JSCallbackFunction.h: |
| 20 | + (JSC::JSCallbackFunction::createStructure): |
| 21 | + * API/JSCallbackObject.h: |
| 22 | + (JSC::JSCallbackObject::createStructure): |
| 23 | + * API/JSContextRef.cpp: |
| 24 | + * JavaScriptCore.exp: |
| 25 | + * debugger/DebuggerActivation.h: |
| 26 | + (JSC::DebuggerActivation::createStructure): |
| 27 | + * runtime/Arguments.h: |
| 28 | + (JSC::Arguments::createStructure): |
| 29 | + * runtime/ArrayConstructor.h: |
| 30 | + (JSC::ArrayConstructor::createStructure): |
| 31 | + * runtime/ArrayPrototype.h: |
| 32 | + (JSC::ArrayPrototype::createStructure): |
| 33 | + * runtime/BooleanObject.h: |
| 34 | + (JSC::BooleanObject::createStructure): |
| 35 | + * runtime/BooleanPrototype.h: |
| 36 | + (JSC::BooleanPrototype::createStructure): |
| 37 | + * runtime/DateConstructor.h: |
| 38 | + (JSC::DateConstructor::createStructure): |
| 39 | + * runtime/DateInstance.h: |
| 40 | + (JSC::DateInstance::createStructure): |
| 41 | + * runtime/DatePrototype.h: |
| 42 | + (JSC::DatePrototype::createStructure): |
| 43 | + * runtime/ErrorInstance.h: |
| 44 | + (JSC::ErrorInstance::createStructure): |
| 45 | + * runtime/ErrorPrototype.h: |
| 46 | + (JSC::ErrorPrototype::createStructure): |
| 47 | + * runtime/Executable.h: |
| 48 | + (JSC::ExecutableBase::createStructure): |
| 49 | + (JSC::NativeExecutable::createStructure): |
| 50 | + (JSC::EvalExecutable::createStructure): |
| 51 | + (JSC::ProgramExecutable::createStructure): |
| 52 | + (JSC::FunctionExecutable::createStructure): |
| 53 | + * runtime/FunctionPrototype.h: |
| 54 | + (JSC::FunctionPrototype::createStructure): |
| 55 | + * runtime/GetterSetter.h: |
| 56 | + (JSC::GetterSetter::createStructure): |
| 57 | + * runtime/InternalFunction.h: |
| 58 | + (JSC::InternalFunction::createStructure): |
| 59 | + * runtime/JSAPIValueWrapper.h: |
| 60 | + (JSC::JSAPIValueWrapper::createStructure): |
| 61 | + * runtime/JSActivation.h: |
| 62 | + (JSC::JSActivation::createStructure): |
| 63 | + * runtime/JSArray.h: |
| 64 | + (JSC::JSArray::createStructure): |
| 65 | + * runtime/JSByteArray.cpp: |
| 66 | + (JSC::JSByteArray::createStructure): |
| 67 | + * runtime/JSByteArray.h: |
| 68 | + * runtime/JSFunction.h: |
| 69 | + (JSC::JSFunction::createStructure): |
| 70 | + * runtime/JSGlobalData.cpp: |
| 71 | + (JSC::JSGlobalData::JSGlobalData): |
| 72 | + * runtime/JSGlobalObject.cpp: |
| 73 | + (JSC::JSGlobalObject::reset): |
| 74 | + * runtime/JSGlobalObject.h: |
| 75 | + (JSC::JSGlobalObject::finishCreation): |
| 76 | + (JSC::JSGlobalObject::createStructure): |
| 77 | + * runtime/JSNotAnObject.h: |
| 78 | + (JSC::JSNotAnObject::createStructure): |
| 79 | + * runtime/JSONObject.h: |
| 80 | + (JSC::JSONObject::createStructure): |
| 81 | + * runtime/JSObject.cpp: |
| 82 | + (JSC::JSObject::createInheritorID): |
| 83 | + * runtime/JSObject.h: |
| 84 | + (JSC::JSObject::createStructure): |
| 85 | + (JSC::JSNonFinalObject::createStructure): |
| 86 | + (JSC::JSFinalObject::createStructure): |
| 87 | + (JSC::createEmptyObjectStructure): |
| 88 | + * runtime/JSObjectWithGlobalObject.h: |
| 89 | + (JSC::JSObjectWithGlobalObject::createStructure): |
| 90 | + * runtime/JSPropertyNameIterator.h: |
| 91 | + (JSC::JSPropertyNameIterator::createStructure): |
| 92 | + * runtime/JSStaticScopeObject.h: |
| 93 | + (JSC::JSStaticScopeObject::createStructure): |
| 94 | + * runtime/JSString.h: |
| 95 | + (JSC::RopeBuilder::createStructure): |
| 96 | + * runtime/JSVariableObject.h: |
| 97 | + (JSC::JSVariableObject::createStructure): |
| 98 | + * runtime/JSWrapperObject.h: |
| 99 | + (JSC::JSWrapperObject::createStructure): |
| 100 | + * runtime/MathObject.h: |
| 101 | + (JSC::MathObject::createStructure): |
| 102 | + * runtime/NativeErrorConstructor.h: |
| 103 | + (JSC::NativeErrorConstructor::createStructure): |
| 104 | + (JSC::NativeErrorConstructor::constructorBody): |
| 105 | + * runtime/NumberConstructor.h: |
| 106 | + (JSC::NumberConstructor::createStructure): |
| 107 | + * runtime/NumberObject.h: |
| 108 | + (JSC::NumberObject::createStructure): |
| 109 | + * runtime/NumberPrototype.h: |
| 110 | + (JSC::NumberPrototype::createStructure): |
| 111 | + * runtime/ObjectConstructor.h: |
| 112 | + (JSC::ObjectConstructor::createStructure): |
| 113 | + * runtime/ObjectPrototype.h: |
| 114 | + (JSC::ObjectPrototype::createStructure): |
| 115 | + * runtime/RegExp.h: |
| 116 | + (JSC::RegExp::createStructure): |
| 117 | + * runtime/RegExpConstructor.h: |
| 118 | + (JSC::RegExpConstructor::createStructure): |
| 119 | + * runtime/RegExpObject.h: |
| 120 | + (JSC::RegExpObject::createStructure): |
| 121 | + * runtime/RegExpPrototype.h: |
| 122 | + (JSC::RegExpPrototype::createStructure): |
| 123 | + * runtime/ScopeChain.h: |
| 124 | + (JSC::ScopeChainNode::createStructure): |
| 125 | + * runtime/StrictEvalActivation.h: |
| 126 | + (JSC::StrictEvalActivation::createStructure): |
| 127 | + * runtime/StringConstructor.h: |
| 128 | + (JSC::StringConstructor::createStructure): |
| 129 | + * runtime/StringObject.h: |
| 130 | + (JSC::StringObject::createStructure): |
| 131 | + * runtime/StringObjectThatMasqueradesAsUndefined.h: |
| 132 | + (JSC::StringObjectThatMasqueradesAsUndefined::create): |
| 133 | + (JSC::StringObjectThatMasqueradesAsUndefined::createStructure): |
| 134 | + * runtime/StringPrototype.h: |
| 135 | + (JSC::StringPrototype::createStructure): |
| 136 | + * runtime/Structure.cpp: |
| 137 | + (JSC::Structure::Structure): |
| 138 | + (JSC::Structure::visitChildren): |
| 139 | + * runtime/Structure.h: |
| 140 | + (JSC::Structure::create): |
| 141 | + (JSC::Structure::globalObject): |
| 142 | + (JSC::Structure::setGlobalObject): |
| 143 | + * runtime/StructureChain.h: |
| 144 | + (JSC::StructureChain::createStructure): |
| 145 | + |
1 | 146 | 2011-09-06 Michael Saboff <msaboff@apple.com> |
2 | 147 |
|
3 | 148 | Add windows changes for JSC:RegExp functional tests |
|
0 commit comments