Fix DaemonControllerLinux::SetConfigAndStart() to reload config automatically.

remoting/host/in_memory_host_config.cc

Index: remoting/host/in_memory_host_config.cc
diff --git a/remoting/host/in_memory_host_config.cc b/remoting/host/in_memory_host_config.cc
index 5e95127b27bd0c7be8afe500d20df6150425161b..b2f2991b124332007c228a053d7d48843331f63e 100644
--- a/remoting/host/in_memory_host_config.cc
+++ b/remoting/host/in_memory_host_config.cc
@@ -44,4 +44,23 @@ void InMemoryHostConfig::SetBoolean(const std::string& path, bool in_value) {
   values_->SetBoolean(path, in_value);
 }
 
+bool InMemoryHostConfig::CopyFrom(const base::DictionaryValue* dictionary) {
+  bool result = true;
+
+  for (DictionaryValue::key_iterator key(dictionary->begin_keys());
+       key != dictionary->end_keys(); ++key) {
+    std::string str_value;
+    bool bool_value;
+    if (dictionary->GetString(*key, &str_value)) {
+      SetString(*key, str_value);
+    } else if (dictionary->GetBoolean(*key, &bool_value)) {

[Jamie, 2012/08/13 23:44:31]

The code this is based on didn't handle Boolean values. This is undoubtedly
better, but was there a bug in the old code we should be aware of?

[Sergey Ulanov, 2012/08/14 18:00:37]

We use boolean only for oauth_use_official_client_id parameter. That parameter
is specified when host is started, but not when the config is updated. It was
breaking the code we had for linux, but on mac only update was affected by this
issue, so it was still working fine on mac.

+      SetBoolean(*key, bool_value);
+    } else {
+      result = false;
+    }
+  }
+
+  return result;
+}
+
 }  // namespace remoting