Changed the call to attach the debugger to V8 to run in the renderer thread. ...

chrome/renderer/debug_message_handler.h

 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // MessageFilter object to handle messages aimed at the debugger, and to
 // dispatch them from the main thread rather than the render thread.
 // Also owns the reference to the Debugger object and handles callbacks from it.
 
 #ifndef CHROME_RENDERER_DEBUG_MESSAGE_HANDLER_H_
 #define CHROME_RENDERER_DEBUG_MESSAGE_HANDLER_H_
 
 #include "chrome/common/ipc_channel_proxy.h"
 #include "webkit/glue/debugger.h"
 
 class RenderView;
 
 class DebugMessageHandler : public IPC::ChannelProxy::MessageFilter,
                             public Debugger::Delegate {
  public:
   DebugMessageHandler(RenderView* view);
   virtual ~DebugMessageHandler();
 
  private:
-  // evaluate javascript URL in the renderer
+  // Evaluate javascript URL in the renderer

[Erik does not do reviews, 2008/11/05 17:00:13]

add a period while you're at it

   void EvaluateScriptUrl(const std::wstring& url);
 
+  // Attach in the renderer

[Erik does not do reviews, 2008/11/05 17:00:13]

same here.  Also, rephrase to include the thread issue.

+  void Attach();
+
   // Debugger::Delegate callback method to handle debugger output.
   void DebuggerOutput(const std::wstring& out);
 
   // Schedule a debugger break.
   void OnBreak(bool force);
 
   // Sends a command to the debugger.
   void OnCommand(const std::wstring& cmd);
 
   // Sends an attach event to the debugger front-end.
@@ skipping 14 matching lines @@
   // Don't ever dereference view_ directly from another thread as it's not
   // threadsafe, instead proxy locally via its MessageLoop.
   RenderView* view_;
   MessageLoop* view_loop_;
 
   int32 view_routing_id_;
   IPC::Channel* channel_;
 };
 
 #endif // CHROME_RENDERER_DEBUG_MESSAGE_HANDLER_H_