Conenct Chromoting plugin debug log to JS UI.

remoting/host/simple_host_process.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.
 //
 // This is an application of a minimal host process in a Chromoting
 // system. It serves the purpose of gluing different pieces together
 // to make a functional host process for testing.
 //
 // It peforms the following functionality:
 // 1. Connect to the GTalk network and register the machine as a host.
@@ skipping 16 matching lines @@
 #include "base/file_path.h"
 #include "base/logging.h"
 #include "base/mac/scoped_nsautorelease_pool.h"
 #include "base/message_loop.h"
 #include "base/path_service.h"
 #include "base/test/mock_chrome_application_mac.h"
 #include "base/threading/thread.h"
 #include "crypto/nss_util.h"
 #include "media/base/media.h"
 #include "remoting/base/constants.h"
+#include "remoting/base/logger.h"
 #include "remoting/base/tracer.h"
 #include "remoting/host/capturer_fake.h"
 #include "remoting/host/chromoting_host.h"
 #include "remoting/host/chromoting_host_context.h"
 #include "remoting/host/continue_window.h"
 #include "remoting/host/curtain.h"
 #include "remoting/host/desktop_environment.h"
 #include "remoting/host/disconnect_window.h"
 #include "remoting/host/event_executor.h"
 #include "remoting/host/heartbeat_sender.h"
@@ skipping 121 matching lines @@
     } else {
       scoped_ptr<remoting::SelfAccessVerifier> self_access_verifier(
           new remoting::SelfAccessVerifier());
       if (!self_access_verifier->Init(config))
         return 1;
       access_verifier.reset(self_access_verifier.release());
     }
 
     // Construct a chromoting host.
     scoped_refptr<ChromotingHost> host;
+    logger_.reset(new remoting::Logger());
     if (fake_) {
       remoting::Capturer* capturer =
           new remoting::CapturerFake();
       remoting::EventExecutor* event_executor =
           remoting::EventExecutor::Create(context.desktop_message_loop(),
                                           capturer);
       remoting::Curtain* curtain = remoting::Curtain::Create();
       remoting::DisconnectWindow* disconnect_window =
           remoting::DisconnectWindow::Create();
       remoting::ContinueWindow* continue_window =
           remoting::ContinueWindow::Create();
       remoting::LocalInputMonitor* local_input_monitor =
           remoting::LocalInputMonitor::Create();
       host = ChromotingHost::Create(
           &context, config,
           new DesktopEnvironment(capturer, event_executor, curtain,
                                  disconnect_window, continue_window,
                                  local_input_monitor),
-          access_verifier.release());
+          access_verifier.release(), logger_.get());

[dmac, 2011/06/26 14:34:44]

do you actually need to call get here and below? C++ doesn't automatically call
through for you?

[garykac, 2011/06/27 21:24:20]

When using it, yes. But here is tries to pass scoped_ptr<HostPluginLogger>& and
fails.

     } else {
       host = ChromotingHost::Create(&context, config,
-                                    access_verifier.release());
+                                    access_verifier.release(), logger_.get());
     }
     host->set_it2me(is_it2me_);
 
     if (protocol_config_.get()) {
       host->set_protocol_config(protocol_config_.release());
     }
 
     if (is_it2me_) {
       host->AddStatusObserver(register_request.get());
     } else {
@@ skipping 33 matching lines @@
 
 #if defined(OS_WIN)
     wstring home_path = GetEnvironmentVar(kHomeDrive);
     home_path += GetEnvironmentVar(kHomePath);
 #else
     string home_path = GetEnvironmentVar(base::env_vars::kHome);
 #endif
     return FilePath(home_path).Append(kDefaultConfigPath);
   }
 
+  scoped_ptr<remoting::Logger> logger_;
+
   FilePath config_path_;
   bool fake_;
   bool is_it2me_;
   scoped_ptr<CandidateSessionConfig> protocol_config_;
 };
 
 int main(int argc, char** argv) {
   // Needed for the Mac, so we don't leak objects when threads are created.
   base::mac::ScopedNSAutoreleasePool pool;
 
@@ skipping 46 matching lines @@
       LOG(ERROR) << "Unknown video codec: " << video_codec;
       return 1;
     }
     config->mutable_video_configs()->push_back(ChannelConfig(
         transport, remoting::protocol::kDefaultStreamVersion, codec));
     simple_host.set_protocol_config(config.release());
   }
 
   return simple_host.Run();
 }