Author: [email protected]
Date: Mon Mar 30 01:16:14 2009
New Revision: 1637
Modified:
branches/bleeding_edge/src/debug.cc
Log:
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.
Review URL: http://codereview.chromium.org/58001
Modified: branches/bleeding_edge/src/debug.cc
==============================================================================
--- branches/bleeding_edge/src/debug.cc (original)
+++ branches/bleeding_edge/src/debug.cc Mon Mar 30 01:16:14 2009
@@ -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
-~----------~----~----~----~------~----~------~--~---
