Bug 3601 – Error instance type info

• Status: RESOLVED
• Resolution: FIXED
• Component: JavaScriptCore

[Harri Porten]

error_object.* lacks some fixes applied to the KDE versions. This is one:

-const ClassInfo NativeErrorImp::info = {"Error", &InternalFunctionImp::info, 0, 0};
+const ClassInfo NativeErrorImp::info = {"Function", &InternalFunctionImp::info,
0, 0};

The other would be to introduce the ErrorInstanceImp class to be found in KDE's
.h and .cpp file.

The relevant test cases from khtmltests/regression/tests/js/Error.js:

shouldBe("Object.prototype.toString.apply(Error())", "'[object Error]'");
shouldBe("Object.prototype.toString.apply(Error)", "'[object Function]'");
shouldBe("Object.prototype.toString.apply(EvalError)", "'[object Function]'");

[Geoffrey Garen]

Created attachment 2969 [details]
Patch

Fix.

[Geoffrey Garen]

Causes the following test failures:
        ecma_3/Function/regress-104584.js
        ecma_3/Object/class-001.js
        ecma_3/Object/class-003.js
        ecma_3/Object/class-004.js
        ecma_3/Object/regress-72773.js

[Maciej Stachowiak]

Comment on attachment 2969 [details]
Patch

r=me

[Geoffrey Garen]

(Actually, failure in ecma_3/Function/regress-104584.js is caused by bug #4006.)