unittests for Chromoting native messaging host.

remoting/host/setup/native_messaging_host.h

Index: remoting/host/setup/native_messaging_host.h
diff --git a/remoting/host/setup/native_messaging_host.h b/remoting/host/setup/native_messaging_host.h
index 4bfd09a03818354e0f345992626f6c569900aca7..c5138d23ca9be6dca38fcb2757462d7fa5a3361b 100644
--- a/remoting/host/setup/native_messaging_host.h
+++ b/remoting/host/setup/native_messaging_host.h
@@ -39,6 +39,10 @@ class NativeMessagingHost {
   // error is encountered during reading and processing messages.
   void Shutdown();
 
+  // Allow unittests to use a mock DaemonController instance.
+  void SetDaemonControllerForTest(

[Sergey Ulanov, 2013/05/18 01:59:31]

can daemon_controller be passed to the constructor? Then you wouldn't need this
method.

[Lambros, 2013/05/22 21:42:18]

Done.

+      scoped_ptr<DaemonController> daemon_controller);
+
  private:
   // Processes a message received from the client app.
   void ProcessMessage(scoped_ptr<base::Value> message);
@@ -79,11 +83,10 @@ class NativeMessagingHost {
                                 DaemonController::AsyncResult result);
   void SendConfigResponse(scoped_ptr<base::DictionaryValue> response,
                           scoped_ptr<base::DictionaryValue> config);
-  void SendUsageStatsConsentResponse(
-      scoped_ptr<base::DictionaryValue> response,
-      bool supported,
-      bool allowed,
-      bool set_by_policy);
+  void SendUsageStatsConsentResponse(scoped_ptr<base::DictionaryValue> response,
+                                     bool supported,
+                                     bool allowed,
+                                     bool set_by_policy);
   void SendAsyncResult(scoped_ptr<base::DictionaryValue> response,
                        DaemonController::AsyncResult result);