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

Unified Diff: device/bluetooth/bluetooth_remote_gatt_characteristic_android.cc

Issue 2606823002: Remove base::ScopedPtrHashMap from device/. (Closed)
Patch Set: one last fix Created 4 years 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/bluetooth_remote_gatt_characteristic_android.cc
diff --git a/device/bluetooth/bluetooth_remote_gatt_characteristic_android.cc b/device/bluetooth/bluetooth_remote_gatt_characteristic_android.cc
index d40c0066584448ba65a61e67aab176efba12cb88..5bcd437d6a55f842da6a810c929651ed96f78fdf 100644
--- a/device/bluetooth/bluetooth_remote_gatt_characteristic_android.cc
+++ b/device/bluetooth/bluetooth_remote_gatt_characteristic_android.cc
@@ -113,7 +113,7 @@ BluetoothRemoteGattCharacteristicAndroid::GetDescriptors() const {
EnsureDescriptorsCreated();
std::vector<BluetoothRemoteGattDescriptor*> descriptors;
for (const auto& map_iter : descriptors_)
- descriptors.push_back(map_iter.second);
+ descriptors.push_back(map_iter.second.get());
return descriptors;
}
@@ -124,7 +124,7 @@ BluetoothRemoteGattCharacteristicAndroid::GetDescriptor(
const auto& iter = descriptors_.find(identifier);
if (iter == descriptors_.end())
return nullptr;
- return iter->second;
+ return iter->second.get();
}
void BluetoothRemoteGattCharacteristicAndroid::ReadRemoteCharacteristic(
@@ -241,12 +241,11 @@ void BluetoothRemoteGattCharacteristicAndroid::CreateGattRemoteDescriptor(
std::string instanceIdString =
base::android::ConvertJavaStringToUTF8(env, instanceId);
- DCHECK(!descriptors_.contains(instanceIdString));
+ DCHECK(descriptors_.find(instanceIdString) == descriptors_.end());
Reilly Grant (use Gerrit) 2016/12/28 22:48:01 DCHECK(!base::ContainsKey(descriptors_, instanceId
Avi (use Gerrit) 2017/01/02 19:42:26 Done.
- descriptors_.set(instanceIdString,
- BluetoothRemoteGattDescriptorAndroid::Create(
- instanceIdString, bluetooth_gatt_descriptor_wrapper,
- chrome_bluetooth_device));
+ descriptors_[instanceIdString] = BluetoothRemoteGattDescriptorAndroid::Create(
+ instanceIdString, bluetooth_gatt_descriptor_wrapper,
+ chrome_bluetooth_device);
}
void BluetoothRemoteGattCharacteristicAndroid::SubscribeToNotifications(

Powered by Google App Engine
This is Rietveld 408576698