Reviewers: Mads Ager, Description: Changed the debugger API to allow only one debug event listener to be registered. The public API now only has SetDebugEventListener instead of AddDebugEventListener and RemoveDebugEventListener.
Moved the registrered debug event listener from the context to a global handle in the Debugger class. Storing it in the context did not make much sense. Changed a lot of tests to handle the API change. BUG=1242707 Please review this at http://codereview.chromium.org/19753 SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/ Affected files: M include/v8-debug.h M include/v8.h M src/api.cc M src/bootstrapper.cc M src/contexts.h M src/d8.cc M src/debug-delay.js M src/debug.h M src/debug.cc M src/heap.h M src/heap.cc M src/runtime.h M src/runtime.cc M test/mjsunit/debug-backtrace-text.js M test/mjsunit/debug-backtrace.js M test/mjsunit/debug-changebreakpoint.js M test/mjsunit/debug-clearbreakpoint.js M test/mjsunit/debug-conditional-breakpoints.js M test/mjsunit/debug-constructor.js M test/mjsunit/debug-continue.js M test/mjsunit/debug-enable-disable-breakpoints.js M test/mjsunit/debug-evaluate-arguments.js M test/mjsunit/debug-evaluate-locals.js M test/mjsunit/debug-evaluate-recursive.js M test/mjsunit/debug-evaluate-with.js M test/mjsunit/debug-evaluate.js M test/mjsunit/debug-event-listener.js M test/mjsunit/debug-handle.js M test/mjsunit/debug-ignore-breakpoints.js M test/mjsunit/debug-mirror-cache.js M test/mjsunit/debug-multiple-breakpoints.js M test/mjsunit/debug-scripts-request.js M test/mjsunit/debug-setbreakpoint.js M test/mjsunit/debug-step-stub-callfunction.js M test/mjsunit/debug-step.js M test/mjsunit/debug-stepin-constructor.js M test/mjsunit/regress/regress-1081309.js M test/mjsunit/regress/regress-1170187.js M test/mjsunit/regress/regress-998565.js --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---