bluetooth: Browser-side implementation of getCharacteristic

device/bluetooth/test/mock_bluetooth_gatt_service.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 "device/bluetooth/test/mock_bluetooth_gatt_service.h"
 
 #include "device/bluetooth/test/mock_bluetooth_device.h"
 
 using testing::Return;
 using testing::_;
@@ skipping 15 matching lines @@
       .WillByDefault(Return(std::vector<BluetoothGattCharacteristic*>()));
   ON_CALL(*this, GetIncludedServices())
       .WillByDefault(Return(std::vector<BluetoothGattService*>()));
   ON_CALL(*this, GetCharacteristic(_))
       .WillByDefault(Return(static_cast<BluetoothGattCharacteristic*>(NULL)));
 }
 
 MockBluetoothGattService::~MockBluetoothGattService() {
 }
 
+void MockBluetoothGattService::AddMockCharacteristic(
+    scoped_ptr<MockBluetoothGattCharacteristic> mock_characteristic) {
+  mock_characteristics_.push_back(mock_characteristic.Pass());
+}
+
+std::vector<BluetoothGattCharacteristic*>
+MockBluetoothGattService::GetMockCharacteristics() const {
+  std::vector<BluetoothGattCharacteristic*> characteristics;
+  for (BluetoothGattCharacteristic* characteristic : mock_characteristics_) {
+    characteristics.push_back(characteristic);

[armansito, 2015/06/09 19:23:49]

Why is this copying necessary? Can't you change the return type to const-ref and
return |mock_characteristics_|?

[ortuno, 2015/06/09 20:46:42]

The compiler complains when trying to convert from ScopedVector<T> to
std::vector<T*>.

[armansito, 2015/06/10 02:40:47]

If this is a test only method, why not just return a "const
ScopedVector<BluetoothGattCharacteristic>&"?

+  }
+  return characteristics;
+}
+
 }  // namespace device