[Webkit-unassigned] [Bug 81588] Array.prototype.toString should be generic
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon Apr 2 18:00:55 PDT 2012
https://bugs.webkit.org/show_bug.cgi?id=81588
--- Comment #10 from hojong.han at samsung.com 2012-04-02 18:00:55 PST ---
(In reply to comment #9)
> (From update of attachment 135074 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=135074&action=review
> > Source/JavaScriptCore/runtime/ArrayPrototype.cpp:270
> > + return JSValue::encode(jsMakeNontrivialString(exec, "[object ", thisObject->methodTable()->className(thisObject), "]"));
> This does not seem right. You should call the original Object.prototype.toString instead of duplicating its implementation.
> ...and add tests
Thanks for reveiw. I'll apply what you recommend and tests to the new patch.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list