Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(92)

Unified Diff: device/bluetooth/test/bluetooth_test_bluez.cc

Issue 2102093003: Implement BluetoothDeviceBlueZ::GetServiceRecords. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: device/bluetooth/test/bluetooth_test_bluez.cc
diff --git a/device/bluetooth/test/bluetooth_test_bluez.cc b/device/bluetooth/test/bluetooth_test_bluez.cc
index 45f2beff136bd9ae20ac218f452042401a358453..bc6cae2eac0799f25f2cbb9f2c45fda68e3419e8 100644
--- a/device/bluetooth/test/bluetooth_test_bluez.cc
+++ b/device/bluetooth/test/bluetooth_test_bluez.cc
@@ -67,11 +67,6 @@ void BluetoothTestBlueZ::SetUp() {
std::unique_ptr<bluez::BluezDBusManagerSetter> dbus_setter =
bluez::BluezDBusManager::GetSetterForTesting();
fake_bluetooth_device_client_ = new bluez::FakeBluetoothDeviceClient;
- // TODO(rkc): This is a big hacky. Creating a device client creates three
- // devices by default. For now, the easiest path is to just clear them, but
- // a better way will be to only create them as needed. This will require
- // looking at a lot of tests but should be done eventually.
- fake_bluetooth_device_client_->RemoveAllDevices();
dbus_setter->SetBluetoothDeviceClient(
std::unique_ptr<bluez::BluetoothDeviceClient>(
fake_bluetooth_device_client_));

Powered by Google App Engine
This is Rietveld 408576698