Use a weak pointer to post service control events and session change notifications.

remoting/host/daemon_process.h

 // 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.
 
 #ifndef REMOTING_HOST_DAEMON_PROCESS_H_
 #define REMOTING_HOST_DAEMON_PROCESS_H_
 
 #include <list>
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
 #include "base/observer_list.h"
 #include "base/process.h"
 #include "ipc/ipc_channel.h"
 #include "ipc/ipc_channel_proxy.h"
 #include "ipc/ipc_platform_file.h"
-#include "remoting/base/stoppable.h"
 #include "remoting/host/config_file_watcher.h"
 #include "remoting/host/host_status_monitor.h"
 #include "remoting/host/worker_process_ipc_delegate.h"
 
 struct SerializedTransportRoute;
 
 namespace tracked_objects {
 class Location;
 }  // namespace tracked_objects
 
 namespace remoting {
 
 class AutoThreadTaskRunner;
 class DesktopSession;
 class HostEventLogger;
 class HostStatusObserver;
 class ScreenResolution;
 
 // This class implements core of the daemon process. It manages the networking
 // process running at lower privileges and maintains the list of desktop
 // sessions.
 class DaemonProcess
-    : public Stoppable,
-      public ConfigFileWatcher::Delegate,
+    : public ConfigFileWatcher::Delegate,
       public HostStatusMonitor,
       public WorkerProcessIpcDelegate {
  public:
   typedef std::list<DesktopSession*> DesktopSessionList;
 
   virtual ~DaemonProcess();
 
   // Creates a platform-specific implementation of the daemon process object
   // passing relevant task runners. Public methods of this class must be called
   // on the |caller_task_runner| thread. |io_task_runner| is used to handle IPC
@@ skipping 45 matching lines @@
   // Changes the screen resolution of the desktop session identified by
   // |terminal_id|.
   void SetScreenResolution(int terminal_id, const ScreenResolution& resolution);
 
   // Requests the network process to crash.
   void CrashNetworkProcess(const tracked_objects::Location& location);
 
   // Reads the host configuration and launches the network process.
   void Initialize();
 
+  void Stop();

[Wez, 2013/06/07 17:48:24]

nit: Comment explaining Stop() semantics here, plz.

[alexeypa (please no reviews), 2013/06/07 18:30:23]

Done.

+
   // Returns true if |terminal_id| is in the range of allocated IDs. I.e. it is
   // less or equal to the highest ID we have seen so far.
   bool WasTerminalIdAllocated(int terminal_id);
 
   // Handlers for the host status notifications received from the network
   // process.
   void OnAccessDenied(const std::string& jid);
   void OnClientAuthenticated(const std::string& jid);
   void OnClientConnected(const std::string& jid);
   void OnClientDisconnected(const std::string& jid);
   void OnClientRouteChange(const std::string& jid,
                            const std::string& channel_name,
                            const SerializedTransportRoute& route);
   void OnHostStarted(const std::string& xmpp_login);
   void OnHostShutdown();
 
-  // Stoppable implementation.
-  virtual void DoStop() OVERRIDE;
-
   // Creates a platform-specific desktop session and assigns a unique ID to it.
   // An implementation should validate |params| as they are received via IPC.
   virtual scoped_ptr<DesktopSession> DoCreateDesktopSession(
       int terminal_id,
       const ScreenResolution& resolution,
       bool virtual_terminal) = 0;
 
   // Requests the network process to crash.
   virtual void DoCrashNetworkProcess(
       const tracked_objects::Location& location) = 0;
@@ skipping 32 matching lines @@
 
   // The list of active desktop sessions.
   DesktopSessionList desktop_sessions_;
 
   // The highest desktop session ID that has been seen so far.
   int next_terminal_id_;
 
   // Keeps track of observers receiving host status notifications.
   ObserverList<HostStatusObserver> status_observers_;
 
+  // Invoked to ask the owner to delete |this|.
+  base::Closure stopped_callback_;
+
   // Writes host status updates to the system event log.
   scoped_ptr<HostEventLogger> host_event_logger_;
 
   base::WeakPtrFactory<DaemonProcess> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(DaemonProcess);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_HOST_DAEMON_PROCESS_H_