Add HostObserverInterface and decouple HeartbeatSender and ChromotingHost.

chrome/service/remoting/chromoting_host_manager.cc

Index: chrome/service/remoting/chromoting_host_manager.cc
diff --git a/chrome/service/remoting/chromoting_host_manager.cc b/chrome/service/remoting/chromoting_host_manager.cc
index 12b2adc769d3141598613d4293cde0ab1bb12207..39fdd701ff765889c539b62d27830e314a8276a5 100644
--- a/chrome/service/remoting/chromoting_host_manager.cc
+++ b/chrome/service/remoting/chromoting_host_manager.cc
@@ -11,6 +11,8 @@
 #include "net/base/net_util.h"
 #include "remoting/base/constants.h"
 #include "remoting/host/chromoting_host_context.h"
+#include "remoting/host/heartbeat_sender.h"
+#include "remoting/host/host_key_pair.h"

[Wez, 2011/05/03 05:21:50]

What's host_key_pair.h used for here?

[Sergey Ulanov, 2011/05/03 17:24:18]

HostKeyPair is used here to generate private key and write it in the config.

 #include "remoting/host/json_host_config.h"
 
 namespace remoting {
@@ -179,6 +181,15 @@ void ChromotingHostManager::Start() {
   chromoting_host_ = remoting::ChromotingHost::Create(chromoting_context_.get(),
                                                       chromoting_config_);
 
+  // Initialize HeartbeatSender.
+  scoped_refptr<remoting::HeartbeatSender> heartbeat_sender =
+      new remoting::HeartbeatSender(chromoting_context_->network_message_loop(),
+                                    chromoting_config_);
+  if (!heartbeat_sender->Init())
+    LOG(ERROR) << "Failed to initialize heartbeat sender.";

[Wez, 2011/05/03 05:21:50]

Surely we even have reached this point if the configuration is invalid (which is
the only reason that Init() can fail, I think)?

[Sergey Ulanov, 2011/05/03 17:24:18]

Yes, normally this will not happen, but it may fail if, for example, somebody
edits config, e.g writes invalid private key in it.

+  chromoting_host_->AddStatusObserver(heartbeat_sender);
+
+
   // Then start the chromoting host.
   // When ChromotingHost is shutdown because of failure or a request that
   // we made OnChromotingShutdown() is calls.