Fix WeakPtrFactory member placement in ipc

ipc/mojo/ipc_channel_mojo_host.cc

 // Copyright 2014 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 "ipc/mojo/ipc_channel_mojo_host.h"
 
 #include "base/bind.h"
 #include "base/message_loop/message_loop.h"
 #include "ipc/mojo/ipc_channel_mojo.h"
 
@@ skipping 13 matching lines @@
   virtual void OnChannelCreated(base::WeakPtr<ChannelMojo> channel) override;
   virtual scoped_refptr<base::TaskRunner> GetIOTaskRunner() override;
 
   // Returns an weak ptr of ChannelDelegate instead of Delegate
   base::WeakPtr<ChannelDelegate> GetWeakPtr();
   void OnClientLaunched(base::ProcessHandle process);
   void DeleteThisSoon();
 
  private:
   scoped_refptr<base::TaskRunner> io_task_runner_;
+  base::WeakPtr<ChannelMojo> channel_;
   base::WeakPtrFactory<ChannelDelegate> weak_factory_;
-  base::WeakPtr<ChannelMojo> channel_;
 
   DISALLOW_COPY_AND_ASSIGN(ChannelDelegate);
 };
 
 ChannelMojoHost::ChannelDelegate::ChannelDelegate(
     scoped_refptr<base::TaskRunner> io_task_runner)
     : io_task_runner_(io_task_runner), weak_factory_(this) {
 }
 
 ChannelMojoHost::ChannelDelegate::~ChannelDelegate() {
@@ skipping 31 matching lines @@
       FROM_HERE,
       base::Bind(&base::DeletePointer<ChannelMojoHost::ChannelDelegate>,
                  base::Unretained(this)));
 }
 
 //
 // ChannelMojoHost
 //
 
 ChannelMojoHost::ChannelMojoHost(scoped_refptr<base::TaskRunner> io_task_runner)
-    : weak_factory_(this),
-      io_task_runner_(io_task_runner),
-      channel_delegate_(new ChannelDelegate(io_task_runner)) {
+    : io_task_runner_(io_task_runner),
+      channel_delegate_(new ChannelDelegate(io_task_runner)),
+      weak_factory_(this) {
 }
 
 ChannelMojoHost::~ChannelMojoHost() {
 }
 
 void ChannelMojoHost::OnClientLaunched(base::ProcessHandle process) {
   if (io_task_runner_ == base::MessageLoop::current()->message_loop_proxy()) {
     channel_delegate_->OnClientLaunched(process);
   } else {
     io_task_runner_->PostTask(FROM_HERE,
                               base::Bind(&ChannelDelegate::OnClientLaunched,
                                          channel_delegate_->GetWeakPtr(),
                                          process));
   }
 }
 
 ChannelMojo::Delegate* ChannelMojoHost::channel_delegate() const {
   return channel_delegate_.get();
 }
 
 void ChannelMojoHost::DelegateDeleter::operator()(
     ChannelMojoHost::ChannelDelegate* ptr) const {
   ptr->DeleteThisSoon();
 }
 
 }  // namespace IPC