Changeset 12911 in webkit for trunk/JavaScriptCore/kjs/bool_object.cpp

Timestamp:

Feb 20, 2006, 11:54:55 PM (19 years ago)

Author:

mjs

Message:

Reviewed by Geoff and Darin.

Patch from Maks Orlovich, based on work by David Faure, hand-applied and
significantly reworked by me.

• Patch: give internal function names (KJS merge) ​http://bugzilla.opendarwin.org/show_bug.cgi?id=6279

• tests/mozilla/expected.html: Updated for newly fixed test.

• kjs/array_object.cpp: (ArrayProtoFunc::ArrayProtoFunc):
• kjs/array_object.h:
• kjs/bool_object.cpp: (BooleanPrototype::BooleanPrototype): (BooleanProtoFunc::BooleanProtoFunc):
• kjs/bool_object.h:
• kjs/date_object.cpp: (KJS::DateProtoFunc::DateProtoFunc): (KJS::DateObjectImp::DateObjectImp): (KJS::DateObjectFuncImp::DateObjectFuncImp):
• kjs/error_object.cpp: (ErrorPrototype::ErrorPrototype): (ErrorProtoFunc::ErrorProtoFunc):
• kjs/error_object.h:
• kjs/function.cpp: (KJS::FunctionImp::FunctionImp): (KJS::GlobalFuncImp::GlobalFuncImp):
• kjs/function.h:
• kjs/function_object.cpp: (FunctionPrototype::FunctionPrototype): (FunctionProtoFunc::FunctionProtoFunc): (FunctionProtoFunc::callAsFunction):
• kjs/function_object.h:
• kjs/internal.cpp: (KJS::InterpreterImp::initGlobalObject): (KJS::InternalFunctionImp::InternalFunctionImp):
• kjs/internal.h: (KJS::InternalFunctionImp::functionName):
• kjs/lookup.h: (KJS::staticFunctionGetter): (KJS::HashEntryFunction::HashEntryFunction): (KJS::HashEntryFunction::implementsCall): (KJS::HashEntryFunction::toBoolean): (KJS::HashEntryFunction::implementsHasInstance): (KJS::HashEntryFunction::hasInstance):
• kjs/math_object.cpp: (MathFuncImp::MathFuncImp):
• kjs/math_object.h:
• kjs/number_object.cpp: (NumberPrototype::NumberPrototype): (NumberProtoFunc::NumberProtoFunc):
• kjs/number_object.h:
• kjs/object.cpp: (KJS::JSObject::putDirectFunction): (KJS::Error::create):
• kjs/object.h:
• kjs/object_object.cpp: (ObjectPrototype::ObjectPrototype): (ObjectProtoFunc::ObjectProtoFunc):
• kjs/object_object.h:
• kjs/regexp_object.cpp: (RegExpPrototype::RegExpPrototype): (RegExpProtoFunc::RegExpProtoFunc):
• kjs/regexp_object.h:
• kjs/string_object.cpp: (StringProtoFunc::StringProtoFunc): (StringObjectImp::StringObjectImp): (StringObjectFuncImp::StringObjectFuncImp):
• kjs/string_object.h:

File:

• trunk/JavaScriptCore/kjs/bool_object.cpp (modified) (2 diffs)

trunk/JavaScriptCore/kjs/bool_object.cpp

@@ -47 +47 @@
 // ECMA 15.6.4
 
-BooleanPrototype::BooleanPrototype(ExecState *exec,
-                                         ObjectPrototype *objectProto,
-                                         FunctionPrototype *funcProto)
+BooleanPrototype::BooleanPrototype(ExecState* exec, ObjectPrototype* objectProto, FunctionPrototype* funcProto)
   : BooleanInstance(objectProto)
 {
   // The constructor will be added later by InterpreterImp::InterpreterImp()
 
-  putDirect(toStringPropertyName, new BooleanProtoFunc(exec,funcProto,BooleanProtoFunc::ToString,0), DontEnum);
-  putDirect(valueOfPropertyName,  new BooleanProtoFunc(exec,funcProto,BooleanProtoFunc::ValueOf,0),  DontEnum);
+  putDirectFunction(new BooleanProtoFunc(exec, funcProto, BooleanProtoFunc::ToString, 0, toStringPropertyName), DontEnum);
+  putDirectFunction(new BooleanProtoFunc(exec, funcProto, BooleanProtoFunc::ValueOf, 0, valueOfPropertyName),  DontEnum);
   setInternalValue(jsBoolean(false));
 }
@@ -62 +60 @@
 // ------------------------------ BooleanProtoFunc --------------------------
 
-BooleanProtoFunc::BooleanProtoFunc(ExecState *exec,
-                                         FunctionPrototype *funcProto, int i, int len)
-  : InternalFunctionImp(funcProto), id(i)
+BooleanProtoFunc::BooleanProtoFunc(ExecState*, FunctionPrototype* funcProto, int i, int len, const Identifier& name)
+  : InternalFunctionImp(funcProto, name)
+  , id(i)
 {
   putDirect(lengthPropertyName, len, DontDelete|ReadOnly|DontEnum);