The host sends simple log entries to the server.

remoting/host/plugin/host_script_object.cc

 // Copyright (c) 2011 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/plugin/host_script_object.h"
 
 #include "base/bind.h"
 #include "base/message_loop.h"
 #include "base/message_loop_proxy.h"
 #include "base/sys_string_conversions.h"
@@ skipping 74 matching lines @@
       parent_(parent),
       state_(kDisconnected),
       np_thread_id_(base::PlatformThread::CurrentId()),
       plugin_message_loop_proxy_(
           new PluginMessageLoopProxy(plugin_thread_delegate)),
       host_context_(plugin_message_loop_proxy_),
       failed_login_attempts_(0),
       disconnected_event_(true, false),
       am_currently_logging_(false),
       nat_traversal_enabled_(false),
-      policy_received_(false) {
+      policy_received_(false),
+      enable_log_to_server_(false) {
 }
 
 HostNPScriptObject::~HostNPScriptObject() {
   CHECK_EQ(base::PlatformThread::CurrentId(), np_thread_id_);
 
   // Shutdown DesktopEnvironment first so that it doesn't try to post
   // tasks on the UI thread while we are stopping the host.
   if (desktop_environment_.get()) {
     desktop_environment_->Shutdown();
   }
@@ skipping 399 matching lines @@
   desktop_environment_.reset(DesktopEnvironment::Create(&host_context_));
   if (desktop_environment_.get() == NULL) {
     SetState(kError);
     return;
   }
 
   // Beyond this point nothing can fail, so save the config and request.
   host_config_ = host_config;
   register_request_.reset(register_request.release());
 
+  if (enable_log_to_server_) {

[Sergey Ulanov, 2011/11/22 02:01:43]

nit: please remove brackets for consistency with other code in this file.

[simonmorris, 2011/11/22 23:27:35]

Merged with another if, so braces are now needed.

+    log_to_server_.reset(new LogToServer(host_context_.network_message_loop()));
+  }
+
   // Create the Host.
   LOG(INFO) << "NAT state: " << nat_traversal_enabled_;
   host_ = ChromotingHost::Create(
       &host_context_, host_config_, desktop_environment_.get(),
       access_verifier.release(), nat_traversal_enabled_);
   host_->AddStatusObserver(this);
   host_->AddStatusObserver(register_request_.get());
+  if (log_to_server_.get()) {

[Sergey Ulanov, 2011/11/22 02:01:43]

nit: I think you can merge this if and the previous one together.

[simonmorris, 2011/11/22 23:27:35]

Done.

+    host_->AddStatusObserver(log_to_server_.get());
+  }
   host_->set_it2me(true);
 
   {
     base::AutoLock auto_lock(ui_strings_lock_);
     host_->SetUiStrings(ui_strings_);
   }
 
   // Start the Host.
   host_->Start();
 
@@ skipping 306 matching lines @@
                                                uint32_t argCount) {
   NPVariant np_result;
   bool is_good = g_npnetscape_funcs->invokeDefault(plugin_, func, args,
                                                    argCount, &np_result);
   if (is_good)
       g_npnetscape_funcs->releasevariantvalue(&np_result);
   return is_good;
 }
 
 }  // namespace remoting