https://bugs.webkit.org/show_bug.cgi?id=142862
Reviewed by Benjamin Poulain.
Source/JavaScriptCore:
Add a test that used to fail in DFG now that the bug has been fixed by r181993.
* tests/stress/class-syntax-derived-default-constructor.js: Added.
Source/WebInspectorUI:
Removed the workaround for the bug since it has been fixed by r181993.
* UserInterface/Base/Object.js:
* UserInterface/Models/DebuggerDashboard.js:
* UserInterface/Models/NetworkTimeline.js:
* UserInterface/Models/ReplayDashboard.js:
* UserInterface/Models/Revision.js:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@182100
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
+2015-03-27 Ryosuke Niwa <rniwa@webkit.org>
+
+ ES6 Classes: Runtime error in JIT'd class calling super() with arguments and superclass has default constructor
+ https://bugs.webkit.org/show_bug.cgi?id=142862
+
+ Reviewed by Benjamin Poulain.
+
+ Add a test that used to fail in DFG now that the bug has been fixed by r181993.
+
+ * tests/stress/class-syntax-derived-default-constructor.js: Added.
+
2015-03-27 Michael Saboff <msaboff@apple.com>
load8Signed() and load16Signed() should be renamed to avoid confusion
--- /dev/null
+
+var A = class A { };
+var B = class B extends A { };
+var C = class C extends B { constructor() { super(); } };
+
+noInline(C);
+
+(function() {
+ var x;
+ for (var i = 0; i < 1e5; ++i)
+ x = new C(false);
+})();
+
+var D = class D extends A { constructor() {
+ super(...arguments);
+ return function () { return arguments; }
+} };
+var E = class E extends D { constructor() { super(); } };
+
+noInline(E);
+
+(function() {
+ var x;
+ for (var i = 0; i < 1e5; ++i)
+ x = new C(false);
+})();
+2015-03-27 Ryosuke Niwa <rniwa@webkit.org>
+
+ ES6 Classes: Runtime error in JIT'd class calling super() with arguments and superclass has default constructor
+ https://bugs.webkit.org/show_bug.cgi?id=142862
+
+ Reviewed by Benjamin Poulain.
+
+ Removed the workaround for the bug since it has been fixed by r181993.
+
+ * UserInterface/Base/Object.js:
+ * UserInterface/Models/DebuggerDashboard.js:
+ * UserInterface/Models/NetworkTimeline.js:
+ * UserInterface/Models/ReplayDashboard.js:
+ * UserInterface/Models/Revision.js:
+
2015-03-27 Joseph Pecoraro <pecoraro@apple.com>
Web Inspector: Add Setter Icon for ObjectTreeView
WebInspector.Object = class Object
{
- constructor()
- {
- // FIXME: Remove once <https://webkit.org/b/142862> is fixed.
- }
-
// Static
static addEventListener(eventType, listener, thisObject)
WebInspector.DebuggerDashboard = class DebuggerDashboard extends WebInspector.Object
{
- constructor()
- {
- // FIXME: Remove once <https://webkit.org/b/142862> is fixed.
- super();
- }
};
WebInspector.NetworkTimeline = class NetworkTimeline extends WebInspector.Timeline
{
- constructor(type)
- {
- // FIXME: Remove once <https://webkit.org/b/142862> is fixed.
- super(type);
- }
-
// Public
recordForResource(resource)
WebInspector.ReplayDashboard = class ReplayDashboard extends WebInspector.Object
{
- constructor()
- {
- // FIXME: Remove once <https://webkit.org/b/142862> is fixed.
- super();
- }
};
WebInspector.Revision = class Revision extends WebInspector.Object
{
- constructor()
- {
- // FIXME: Remove once <https://webkit.org/b/142862> is fixed.
- super();
- }
-
// Public
apply()