Fix remaining WeakPtrFactory ordering problems

chrome/browser/speech/extension_api/tts_extension_apitest.cc

 // 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.
 
 #include <vector>
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/memory/weak_ptr.h"
 #include "base/message_loop/message_loop.h"
@@ skipping 27 matching lines @@
 
 namespace {
 int g_saved_utterance_id;
 }
 
 namespace extensions {
 
 class MockTtsPlatformImpl : public TtsPlatformImpl {
  public:
   MockTtsPlatformImpl()
-      : ptr_factory_(this),
-        should_fake_get_voices_(false) {}
+      : should_fake_get_voices_(false),
+        ptr_factory_(this) {}
 
   virtual bool PlatformImplAvailable() {
     return true;
   }
 
   MOCK_METHOD5(Speak,
                bool(int utterance_id,
                     const std::string& utterance,
                     const std::string& lang,
                     const VoiceData& voice,
@@ skipping 95 matching lines @@
               ptr_factory_.GetWeakPtr(),
               true, utterance_id, event_type, char_index, message),
           base::TimeDelta::FromMilliseconds(100));
       return;
     }
 
     controller->OnTtsEvent(utterance_id, event_type, char_index, message);
   }
 
  private:
+  bool should_fake_get_voices_;
   base::WeakPtrFactory<MockTtsPlatformImpl> ptr_factory_;
-  bool should_fake_get_voices_;
 };
 
 class FakeNetworkOnlineStateForTest : public net::NetworkChangeNotifier {
  public:
   explicit FakeNetworkOnlineStateForTest(bool online) : online_(online) {}
   ~FakeNetworkOnlineStateForTest() override {}
 
   ConnectionType GetCurrentConnectionType() const override {
     return online_ ?
         net::NetworkChangeNotifier::CONNECTION_ETHERNET :
@@ skipping 262 matching lines @@
   // Test should fail when offline.
   ASSERT_FALSE(RunExtensionTest("tts_engine/network_speech_engine"));
 }
 
 // http://crbug.com/122474
 IN_PROC_BROWSER_TEST_F(TtsApiTest, EngineApi) {
   ASSERT_TRUE(RunExtensionTest("tts_engine/engine_api")) << message_;
 }
 
 }  // namespace extensions