WeakPtr destruction order cleanup: media edition.

media/base/serial_runner.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 MEDIA_BASE_SERIAL_RUNNER_H_
 #define MEDIA_BASE_SERIAL_RUNNER_H_
 
 #include <queue>
 
 #include "base/callback.h"
@@ skipping 53 matching lines @@
       const Queue& bound_fns, const PipelineStatusCB& done_cb);
 
  private:
   friend struct base::DefaultDeleter<SerialRunner>;
 
   SerialRunner(const Queue& bound_fns, const PipelineStatusCB& done_cb);
   ~SerialRunner();
 
   void RunNextInSeries(PipelineStatus last_status);
 
-  base::WeakPtrFactory<SerialRunner> weak_this_;
   scoped_refptr<base::SingleThreadTaskRunner> task_runner_;
   Queue bound_fns_;
   PipelineStatusCB done_cb_;
 
+  // NOTE: Weak pointers must be invalidated before all other member variables.
+  base::WeakPtrFactory<SerialRunner> weak_factory_;
+
   DISALLOW_COPY_AND_ASSIGN(SerialRunner);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_SERIAL_RUNNER_H_