Moving host status events monitoring to a separate interface (HostStatusMonitor).

remoting/host/host_event_logger_win.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/host_event_logger.h"
 
 #include <windows.h>
 #include <string>
 #include <vector>
 
-#include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
+#include "base/memory/weak_ptr.h"
 #include "base/string16.h"
 #include "base/utf_string_conversions.h"
 #include "net/base/ip_endpoint.h"
-#include "remoting/host/chromoting_host.h"
+#include "remoting/host/host_status_monitor.h"
 #include "remoting/host/host_status_observer.h"
 #include "remoting/protocol/transport.h"
 
 #include "remoting_host_messages.h"
 
 namespace remoting {
 
 namespace {
 
 class HostEventLoggerWin : public HostEventLogger, public HostStatusObserver {
  public:
-  HostEventLoggerWin(ChromotingHost* host, const std::string& application_name);
+  HostEventLoggerWin(base::WeakPtr<HostStatusMonitor> monitor,
+                     const std::string& application_name);
 
   virtual ~HostEventLoggerWin();
 
   // HostStatusObserver implementation.  These methods will be called from the
   // network thread.
   virtual void OnClientAuthenticated(const std::string& jid) OVERRIDE;
   virtual void OnClientDisconnected(const std::string& jid) OVERRIDE;
   virtual void OnAccessDenied(const std::string& jid) OVERRIDE;
   virtual void OnClientRouteChange(
       const std::string& jid,
       const std::string& channel_name,
       const protocol::TransportRoute& route) OVERRIDE;
   virtual void OnStart(const std::string& xmpp_login) OVERRIDE;
   virtual void OnShutdown() OVERRIDE;
 
  private:
   void LogString(WORD type, DWORD event_id, const std::string& string);
   void Log(WORD type, DWORD event_id, const std::vector<std::string>& strings);
 
-  scoped_refptr<ChromotingHost> host_;
+  base::WeakPtr<HostStatusMonitor> monitor_;
 
   // The handle of the application event log.
   HANDLE event_log_;
 
   DISALLOW_COPY_AND_ASSIGN(HostEventLoggerWin);
 };
 
 } //namespace
 
-HostEventLoggerWin::HostEventLoggerWin(ChromotingHost* host,
+HostEventLoggerWin::HostEventLoggerWin(base::WeakPtr<HostStatusMonitor> monitor,
                                        const std::string& application_name)
-    : host_(host),
+    : monitor_(monitor),
       event_log_(NULL) {
   event_log_ = RegisterEventSourceW(NULL,
                                     UTF8ToUTF16(application_name).c_str());
   if (event_log_ != NULL) {
-    host_->AddStatusObserver(this);
+    monitor_->AddStatusObserver(this);
   } else {
     LOG_GETLASTERROR(ERROR) << "Failed to register the event source: "
                             << application_name;
   }
 }
 
 HostEventLoggerWin::~HostEventLoggerWin() {
   if (event_log_ != NULL) {
-    host_->RemoveStatusObserver(this);
+    if (monitor_)
+      monitor_->RemoveStatusObserver(this);
     DeregisterEventSource(event_log_);
   }
 }
 
 void HostEventLoggerWin::OnClientAuthenticated(const std::string& jid) {
   LogString(EVENTLOG_INFORMATION_TYPE, MSG_HOST_CLIENT_CONNECTED, jid);
 }
 
 void HostEventLoggerWin::OnClientDisconnected(const std::string& jid) {
   LogString(EVENTLOG_INFORMATION_TYPE, MSG_HOST_CLIENT_DISCONNECTED, jid);
@@ skipping 55 matching lines @@
 void HostEventLoggerWin::LogString(WORD type,
                                    DWORD event_id,
                                    const std::string& string) {
   std::vector<std::string> strings;
   strings.push_back(string);
   Log(type, event_id, strings);
 }
 
 // static
 scoped_ptr<HostEventLogger> HostEventLogger::Create(
-    ChromotingHost* host, const std::string& application_name) {
+    base::WeakPtr<HostStatusMonitor> monitor,
+    const std::string& application_name) {
   return scoped_ptr<HostEventLogger>(
-      new HostEventLoggerWin(host, application_name));
+      new HostEventLoggerWin(monitor, application_name));
 }
 
 }  // namespace remoting