Reviewers: Søren Gjesse, Message: Hi Søren,
Here are 2 new debugger unit tests that test the functionality of listbreakpoints and setexceptionbreak. In rev 6180, I only modified the implementation of listbreakpoints by adding the status of the break on exception states. However, I didn't see a pre-existing test for listbreakpoints. So I took a stab at it while adding the test for the effects for setexceptionbreak. I did not exhaustively implement tests for all of listbreakpoints' possible return values but did what I could with the breakpoint numbers as well as their corresponding line numbers. I hope that is enough for now. The other debugger functionality I added in r6180 includes gc, v8flags, and disconnect. I could not think of any reasonable tests that I can write with the existing infrastructure. So, I'm going to ignore those for now. Please let me know if you think I need to add more tests for the functionality that I've introduced. Thanks. Note: these tests will fail if r6180 is not reapplied to the code base. Description: Added a few new debugger tests. Please review this at http://codereview.chromium.org/6035016/ SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/ Affected files: A test/mjsunit/debug-listbreakpoints.js A test/mjsunit/debug-setexceptionbreak.js -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
