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

Unified Diff: device/bluetooth/bluetooth_advertisement_unittest.cc

Issue 2254833003: Re-write many calls to WrapUnique() with MakeUnique() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 4 years, 4 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
« no previous file with comments | « no previous file | device/bluetooth/bluetooth_device.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: device/bluetooth/bluetooth_advertisement_unittest.cc
diff --git a/device/bluetooth/bluetooth_advertisement_unittest.cc b/device/bluetooth/bluetooth_advertisement_unittest.cc
index 9edc31145a13d6fd7e462212d727754e5de5c5e9..a0ef62c7ba52469a9a8bf357878312f382f87ed7 100644
--- a/device/bluetooth/bluetooth_advertisement_unittest.cc
+++ b/device/bluetooth/bluetooth_advertisement_unittest.cc
@@ -34,7 +34,7 @@ TEST(BluetoothAdvertisementTest, DataMembersAreAssignedCorrectly) {
ASSERT_FALSE(data.service_uuids().get());
// Assign Service UUID.
data.set_service_uuids(
- base::WrapUnique(new BluetoothAdvertisement::UUIDList(uuids)));
+ base::MakeUnique<BluetoothAdvertisement::UUIDList>(uuids));
// Retrieve Service UUID.
ASSERT_EQ(*data.service_uuids(), uuids);
// Retrieve again.
@@ -43,8 +43,9 @@ TEST(BluetoothAdvertisementTest, DataMembersAreAssignedCorrectly) {
// Try without assigning Manufacturer Data.
ASSERT_FALSE(data.manufacturer_data().get());
// Assign Manufacturer Data.
- data.set_manufacturer_data(base::WrapUnique(
- new BluetoothAdvertisement::ManufacturerData(manufacturer_data)));
+ data.set_manufacturer_data(
+ base::MakeUnique<BluetoothAdvertisement::ManufacturerData>(
+ manufacturer_data));
// Retrieve Manufacturer Data.
ASSERT_EQ(*data.manufacturer_data(), manufacturer_data);
// Retrieve again.
@@ -54,7 +55,7 @@ TEST(BluetoothAdvertisementTest, DataMembersAreAssignedCorrectly) {
ASSERT_FALSE(data.solicit_uuids().get());
// Assign Solicit UUIDs.
data.set_solicit_uuids(
- base::WrapUnique(new BluetoothAdvertisement::UUIDList(uuids)));
+ base::MakeUnique<BluetoothAdvertisement::UUIDList>(uuids));
// Retrieve Solicit UUIDs.
ASSERT_EQ(*data.solicit_uuids(), uuids);
// Retieve again.
@@ -64,7 +65,7 @@ TEST(BluetoothAdvertisementTest, DataMembersAreAssignedCorrectly) {
ASSERT_FALSE(data.service_data().get());
// Assign Service Data.
data.set_service_data(
- base::WrapUnique(new BluetoothAdvertisement::ServiceData(service_data)));
+ base::MakeUnique<BluetoothAdvertisement::ServiceData>(service_data));
// Retrieve Service Data.
ASSERT_EQ(*data.service_data(), service_data);
// Retrieve again.
« no previous file with comments | « no previous file | device/bluetooth/bluetooth_device.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698