Maintaining the proper order of initialization WeakPtrFactory in "src/remoting"

remoting/host/win/host_service.h

Index: remoting/host/win/host_service.h
diff --git a/remoting/host/win/host_service.h b/remoting/host/win/host_service.h
index 855f37c14e79013ac50f642d0c2b803c2d179c35..ccd92f7bbebe27a9a287a59912445df02eb62e4e 100644
--- a/remoting/host/win/host_service.h
+++ b/remoting/host/win/host_service.h
@@ -116,9 +116,10 @@ class HostService : public WtsTerminalMonitor {
   // A waitable event that is used to wait until the service is stopped.
   base::WaitableEvent stopped_event_;
 
+  base::WeakPtr<HostService> weak_ptr_;

[Wez, 2014/09/19 18:23:03]

nit: The WeakPtrFactory comment below still applies to this, so consider putting
this between the comment and the weak_factory_ declaration.

+
   // Used to post session change notifications and control events.
   base::WeakPtrFactory<HostService> weak_factory_;
-  base::WeakPtr<HostService> weak_ptr_;
 
   // Singleton.
   friend struct DefaultSingletonTraits<HostService>;