Add test case for custom error's toString.

Add test case for custom error's toString.

R=arv@chromium.org
BUG=
TEST=error-tostring.js

Committed: https://code.google.com/p/v8/source/detail?r=12151

[Yang, 2012-05-21 07:57:23]

PTAL.

[arv (Not doing code reviews), 2012-05-21 16:40:03]

http://codereview.chromium.org/10388208/diff/1/test/mjsunit/error-tostring.js
File test/mjsunit/error-tostring.js (right):

http://codereview.chromium.org/10388208/diff/1/test/mjsunit/error-tostring.js...
test/mjsunit/error-tostring.js:103: assertEquals('MyError toString',
e.toString());
I don't think this tests the right thing. This just calls our toString. What we
need to test is that v8::Message uses our toString.

This test would pass before the code change too.

[Yang, 2012-06-19 14:00:18]

Sorry that this took that long. I have had my plate full. I changed the test to
include v8::Message in the API. I also confirmed that prior your patch
http://codereview.chromium.org/10392158/ this test would fail.
Please take a look.

[Yang, 2012-06-29 13:07:33]

On 2012/06/19 14:00:18, Yang wrote:
> Sorry that this took that long. I have had my plate full. I changed the test
to
> include v8::Message in the API. I also confirmed that prior your patch
> http://codereview.chromium.org/10392158/ this test would fail.
> Please take a look.

Ping.

[Yang, 2012-07-19 08:07:46]

On 2012/06/29 13:07:33, Yang wrote:
> On 2012/06/19 14:00:18, Yang wrote:
> > Sorry that this took that long. I have had my plate full. I changed the test
> to
> > include v8::Message in the API. I also confirmed that prior your patch
> > http://codereview.chromium.org/10392158/ this test would fail.
> > Please take a look.
> 
> Ping.

Another ping.

[arv (Not doing code reviews), 2012-07-19 20:50:11]

LGTM

Sorry. This did not show up in my priority inbox.