Test that debug stub works with browser.

chrome/browser/nacl_host/nacl_browser.h

Index: chrome/browser/nacl_host/nacl_browser.h
===================================================================
--- chrome/browser/nacl_host/nacl_browser.h	(revision 170891)
+++ chrome/browser/nacl_host/nacl_browser.h	(working copy)
@@ -52,6 +52,15 @@
   // Returns whether NaCl application with this manifest URL should be debugged.
   bool URLMatchesDebugPatterns(GURL manifest_url);
 
+  // Send message that new debug stub TCP port is allocated.
+  void DebugStubNewPortAllocated(int port);

[Mark Seaborn, 2012/12/04 21:46:42]

Please group these 4 methods together by removing empty lines between them, and
add a comment that these are used for testing only.

Also rename DebugStubNewPortAllocated so that the name matches up with the
others, e.g. CallDebugStubPortListener().

Also, maybe s/DebugStub/GdbDebugStub/ to make this as obvious as possible to the
uninitiated reader?

[halyavin, 2012/12/05 10:05:57]

Done
I renamed it according to java conventions.
Done

+
+  bool HasDebugStubPortListener();
+
+  void SetDebugStubPortListener(base::Callback<void(int)> listener);
+
+  void ClearDebugStubPortListener();
+
   bool ValidationCacheIsEnabled() const {
     return validation_cache_is_enabled_;
   }
@@ -111,6 +120,7 @@
   bool validation_cache_is_enabled_;
   bool validation_cache_is_modified_;
   NaClResourceState validation_cache_state_;
+  base::Callback<void(int)> debug_stub_port_listener_;
 
   bool ok_;