Reviewers: , Description: Correct debugger active check.
With r1627 (http://code.google.com/p/v8/source/detail?r=1627) the use of the debugger message thread was made optional. However the check for active debugger still included a check for the message thread. This is fixed by this change. Please review this at http://codereview.chromium.org/58001 SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/ Affected files: M src/debug.cc Index: src/debug.cc =================================================================== --- src/debug.cc (revision 1636) +++ src/debug.cc (working copy) @@ -2000,9 +2000,7 @@ void Debugger::UpdateActiveDebugger() { - set_debugger_active((message_thread_ != NULL && - message_handler_ != NULL) || - !event_listener_.is_null()); + set_debugger_active(message_handler_ != NULL | | !event_listener_.is_null()); if (!debugger_active() && message_thread_) { // Send an empty command to the debugger if in a break to make JavaScript // run again if the debugger is closed. --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
