Bluetooth: remove private members from BluetoothAdapter

chrome/browser/extensions/api/bluetooth/bluetooth_apitest.cc

Index: chrome/browser/extensions/api/bluetooth/bluetooth_apitest.cc
diff --git a/chrome/browser/extensions/api/bluetooth/bluetooth_apitest.cc b/chrome/browser/extensions/api/bluetooth/bluetooth_apitest.cc
index 63828ff93c40e4901696867369e9e06dd2cc9a11..27bd5256ff40dacca62a4c5dfa1c68e7a466b550 100644
--- a/chrome/browser/extensions/api/bluetooth/bluetooth_apitest.cc
+++ b/chrome/browser/extensions/api/bluetooth/bluetooth_apitest.cc
@@ -56,8 +56,7 @@ class BluetoothApiTest : public ExtensionApiTest {
 
   void SetUpMockAdapter() {
     // The browser will clean this up when it is torn down
-    mock_adapter_ = new testing::StrictMock<MockBluetoothAdapter>(
-        kAdapterAddress, kName);
+    mock_adapter_ = new testing::StrictMock<MockBluetoothAdapter>;

[youngki, 2013/04/02 12:30:31]

I think using an empty parenthesis is prevalent (i.e. new
testing::StrictMock<MockBluetoothAdapter>(); )

https://code.google.com/p/chromium/codesearch#search/&q=testing::StrictMock%3...

https://code.google.com/p/chromium/codesearch#search/&q=testing::StrictMock%3...

[keybuk, 2013/04/02 19:03:09]

Done.

     event_router()->SetAdapterForTest(mock_adapter_);
 
     device1_.reset(new testing::NiceMock<MockBluetoothDevice>(
@@ -143,6 +142,10 @@ static void CallConnectToServiceCallback(
 }  // namespace
 
 IN_PROC_BROWSER_TEST_F(BluetoothApiTest, OnAdapterStateChanged) {
+  EXPECT_CALL(*mock_adapter_, address())
+      .WillOnce(testing::Return(kAdapterAddress));
+  EXPECT_CALL(*mock_adapter_, name())
+      .WillOnce(testing::Return(kName));
   EXPECT_CALL(*mock_adapter_, IsPresent())
       .WillOnce(testing::Return(false));
   EXPECT_CALL(*mock_adapter_, IsPowered())
@@ -311,6 +314,10 @@ IN_PROC_BROWSER_TEST_F(BluetoothApiTest, DiscoveryCallback) {
 }
 
 IN_PROC_BROWSER_TEST_F(BluetoothApiTest, DiscoveryInProgress) {
+  EXPECT_CALL(*mock_adapter_, address())
+      .WillOnce(testing::Return(kAdapterAddress));
+  EXPECT_CALL(*mock_adapter_, name())
+      .WillOnce(testing::Return(kName));
   EXPECT_CALL(*mock_adapter_, IsPresent())
       .WillOnce(testing::Return(true));
   EXPECT_CALL(*mock_adapter_, IsPowered())
@@ -362,6 +369,10 @@ IN_PROC_BROWSER_TEST_F(BluetoothApiTest, Events) {
   ASSERT_TRUE(LoadExtension(test_data_dir_.AppendASCII("bluetooth/events")));
   EXPECT_TRUE(listener.WaitUntilSatisfied());
 
+  EXPECT_CALL(*mock_adapter_, address())
+      .WillOnce(testing::Return(kAdapterAddress));
+  EXPECT_CALL(*mock_adapter_, name())
+      .WillOnce(testing::Return(kName));
   EXPECT_CALL(*mock_adapter_, IsPresent())
       .WillOnce(testing::Return(false));
   EXPECT_CALL(*mock_adapter_, IsPowered())
@@ -370,6 +381,10 @@ IN_PROC_BROWSER_TEST_F(BluetoothApiTest, Events) {
       .WillOnce(testing::Return(false));
   event_router()->AdapterPoweredChanged(mock_adapter_, false);
 
+  EXPECT_CALL(*mock_adapter_, address())
+      .WillOnce(testing::Return(kAdapterAddress));
+  EXPECT_CALL(*mock_adapter_, name())
+      .WillOnce(testing::Return(kName));
   EXPECT_CALL(*mock_adapter_, IsPresent())
       .WillOnce(testing::Return(true));
   EXPECT_CALL(*mock_adapter_, IsPowered())
@@ -378,6 +393,10 @@ IN_PROC_BROWSER_TEST_F(BluetoothApiTest, Events) {
       .WillOnce(testing::Return(true));
   event_router()->AdapterPresentChanged(mock_adapter_, true);
 
+  EXPECT_CALL(*mock_adapter_, address())
+      .WillOnce(testing::Return(kAdapterAddress));
+  EXPECT_CALL(*mock_adapter_, name())
+      .WillOnce(testing::Return(kName));
   EXPECT_CALL(*mock_adapter_, IsPresent())
       .WillOnce(testing::Return(true));
   EXPECT_CALL(*mock_adapter_, IsPowered())