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

remoting/host/win/host_service.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_WIN_HOST_SERVICE_H_
 #define REMOTING_HOST_WIN_HOST_SERVICE_H_
 
 #include <windows.h>
 
 #include <list>
@@ skipping 98 matching lines @@
 
   // The action routine to be executed.
   int (HostService::*run_routine_)();
 
   // The service status handle.
   SERVICE_STATUS_HANDLE service_status_handle_;
 
   // 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>;
 
   DISALLOW_COPY_AND_ASSIGN(HostService);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_HOST_WIN_HOST_SERVICE_H_