Changeset 29817 in webkit for trunk/JavaScriptCore/kjs/array_object.cpp

Timestamp:

Jan 27, 2008, 12:54:25 AM (17 years ago)

Author:

Darin Adler

Message:

JavaScriptCore:

Reviewed by Oliver.

• fix ​http://bugs.webkit.org/show_bug.cgi?id=16498 .constructor.toString() gives [function]

Test: fast/js/function-names.html

• kjs/array_object.cpp: (KJS::ArrayObjectImp::ArrayObjectImp): Use the class name as the constructor's function name.
• kjs/bool_object.cpp: (KJS::BooleanObjectImp::BooleanObjectImp): Ditto.
• kjs/date_object.cpp: (KJS::DateObjectImp::DateObjectImp): Ditto.
• kjs/error_object.cpp: (KJS::ErrorPrototype::ErrorPrototype): Make the error object be an Error. (KJS::ErrorObjectImp::ErrorObjectImp): Use the class name as the constructor's function name. (KJS::NativeErrorPrototype::NativeErrorPrototype): Take const UString&. (KJS::NativeErrorImp::NativeErrorImp): Use the prototype's name as the constructor's function name.
• kjs/error_object.h: Change ErrorPrototype to inherit from ErrorInstance. Change the NativeErrorImp constructor to take a NativeErrorPrototype pointer for its prototype.
• kjs/function.h: Removed unneeded constructor for internal functions without names. We want to avoid those!
• kjs/function_object.cpp: (KJS::functionProtoFuncToString): Removed code that writes out just [function] for functions that have no names. There's no reason to do that. (KJS::FunctionObjectImp::FunctionObjectImp): Use the class name as the constructor's function name.
• kjs/internal.cpp: Removed the unused constructor.
• kjs/number_object.cpp: (KJS::fractionalPartToString): Marked static for internal linkage. (KJS::exponentialPartToString): Ditto. (KJS::numberProtoFuncToPrecision): Removed an unneeded else. (KJS::NumberObjectImp::NumberObjectImp): Use the class name as the constructor's function name. (KJS::NumberObjectImp::getValueProperty): Tweaked formatting.
• kjs/object_object.cpp: (KJS::ObjectObjectImp::ObjectObjectImp): Use "Object" for the function name.
• kjs/regexp_object.cpp: (KJS::RegExpObjectImp::RegExpObjectImp): Use "RegExp" for the function name.
• kjs/string_object.cpp: (KJS::StringObjectImp::StringObjectImp): Use the class name as the constructor's function name.

LayoutTests:

Reviewed by Oliver.

• test for ​http://bugs.webkit.org/show_bug.cgi?id=16498 .constructor.toString() gives [function]

• fast/js/function-names-expected.txt: Updated for new tests.
• fast/js/kde/resources/function.js: Updated test to expect the format that Gecko uses for native code, which we now match character for character.
• fast/js/resources/function-names.js: Added tests for the names of all the constructors.

File:

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

trunk/JavaScriptCore/kjs/array_object.cpp

@@ -1 +1 @@
 /*
  *  Copyright (C) 1999-2000 Harri Porten ([email protected])
- *  Copyright (C) 2003, 2007 Apple Inc. All rights reserved.
+ *  Copyright (C) 2003, 2007, 2008 Apple Inc. All rights reserved.
  *  Copyright (C) 2003 Peter Kelly ([email protected])
  *  Copyright (C) 2006 Alexey Proskuryakov ([email protected])
@@ -721 +721 @@
 // ------------------------------ ArrayObjectImp -------------------------------
 
-ArrayObjectImp::ArrayObjectImp(ExecState* exec,
-                               FunctionPrototype* funcProto,
-                               ArrayPrototype* arrayProto)
-        : InternalFunctionImp(funcProto)
+ArrayObjectImp::ArrayObjectImp(ExecState* exec, FunctionPrototype* funcProto, ArrayPrototype* arrayProto)
+    : InternalFunctionImp(funcProto, arrayProto->classInfo()->className)
 {
     // ECMA 15.4.3.1 Array.prototype