ExtensionsTest: Move initialization to SetUp and avoid potential UAF.

extensions/browser/api/bluetooth/bluetooth_event_router_unittest.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 "extensions/browser/api/bluetooth/bluetooth_event_router.h"
 
 #include <memory>
 #include <string>
 #include <utility>
 
+#include "base/memory/ptr_util.h"
 #include "base/memory/ref_counted.h"
 #include "base/message_loop/message_loop.h"
 #include "base/run_loop.h"
 #include "content/public/test/test_browser_context.h"
 #include "content/public/test/test_browser_thread.h"
 #include "device/bluetooth/bluetooth_common.h"
 #include "device/bluetooth/bluetooth_uuid.h"
 #include "device/bluetooth/test/mock_bluetooth_adapter.h"
 #include "extensions/browser/extension_registry.h"
 #include "extensions/browser/extensions_test.h"
@@ skipping 14 matching lines @@
 }  // namespace
 
 namespace extensions {
 
 namespace bluetooth = api::bluetooth;
 
 class BluetoothEventRouterTest : public ExtensionsTest {
  public:
   BluetoothEventRouterTest()
       : ui_thread_(content::BrowserThread::UI, &message_loop_),
-        mock_adapter_(new testing::StrictMock<device::MockBluetoothAdapter>()),
-        router_(new BluetoothEventRouter(browser_context())) {
+        mock_adapter_(new testing::StrictMock<device::MockBluetoothAdapter>()) {

[karandeepb, 2017/04/06 03:29:24]

Seems this is leaked. Intentional?

+  }
+
+  void SetUp() override {
+    ExtensionsTest::SetUp();
+    router_ = base::MakeUnique<BluetoothEventRouter>(browser_context());
     router_->SetAdapterForTest(mock_adapter_);
   }
 
   void TearDown() override {
     // It's important to destroy the router before the browser context keyed
     // services so it removes itself as an ExtensionRegistry observer.
     router_.reset(NULL);
     ExtensionsTest::TearDown();
   }
 
@@ skipping 59 matching lines @@
                                   testing::_, testing::_)).Times(1);
 
   router_->StartDiscoverySession(mock_adapter_, kTestExtensionId,
                                  base::Bind(&base::DoNothing),
                                  base::Bind(&base::DoNothing));
 
   EXPECT_CALL(*mock_adapter_, RemoveObserver(testing::_)).Times(1);
 }
 
 }  // namespace extensions