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

remoting/host/client_session.h

Index: remoting/host/client_session.h
diff --git a/remoting/host/client_session.h b/remoting/host/client_session.h
index f9fdd3bd5c25de4e63bc9839a8b1d2cfe2ba76cf..90de054e168de7e862c81087c748d01a341a4f69 100644
--- a/remoting/host/client_session.h
+++ b/remoting/host/client_session.h
@@ -208,11 +208,6 @@ class ClientSession
   protocol::InputFilter auth_input_filter_;
   protocol::ClipboardFilter auth_clipboard_filter_;
 
-  // Factory for weak pointers to the client clipboard stub.
-  // This must appear after |clipboard_echo_filter_|, so that it won't outlive
-  // it.
-  base::WeakPtrFactory<protocol::ClipboardStub> client_clipboard_factory_;

[Wez, 2014/09/12 18:21:52]

This doesn't need to move, since it's not a WeakPtrFactory to |this|.

-
   // The maximum duration of this session.
   // There is no maximum if this value is <= 0.
   base::TimeDelta max_duration_;
@@ -263,6 +258,11 @@ class ClientSession
   bool lossless_video_encode_;
   bool lossless_video_color_;
 
+  // Factory for weak pointers to the client clipboard stub.
+  // This must appear after |clipboard_echo_filter_|, so that it won't outlive
+  // it.
+  base::WeakPtrFactory<protocol::ClipboardStub> client_clipboard_factory_;
+
   DISALLOW_COPY_AND_ASSIGN(ClientSession);
 };