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

remoting/host/in_memory_host_config.cc

 // Copyright (c) 2012 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.
 
 #include "remoting/host/in_memory_host_config.h"
 
 #include "base/logging.h"
 #include "base/values.h"
 
 namespace remoting {
@@ skipping 26 matching lines @@
                                    const std::string& in_value) {
   DCHECK(CalledOnValidThread());
   values_->SetString(path, in_value);
 }
 
 void InMemoryHostConfig::SetBoolean(const std::string& path, bool in_value) {
   DCHECK(CalledOnValidThread());
   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