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

Unified Diff: mojo/common/common_custom_types_unittest.cc

Issue 2065793002: Return a unique_ptr from BinaryValue::CreateWithCopiedBuffer. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix Android and CrOS 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: mojo/common/common_custom_types_unittest.cc
diff --git a/mojo/common/common_custom_types_unittest.cc b/mojo/common/common_custom_types_unittest.cc
index d0e115b50ee1d649a640dfad08a02cabf288b7db..370c2d451689973e20a773a74a69622c76b0c98b 100644
--- a/mojo/common/common_custom_types_unittest.cc
+++ b/mojo/common/common_custom_types_unittest.cc
@@ -164,9 +164,7 @@ TEST_F(CommonCustomTypesTest, Value) {
dict.SetString("string", "some string");
dict.SetBoolean("nested.bool", true);
dict.SetInteger("nested.int", 9);
- dict.Set(
- "some_binary",
- base::WrapUnique(base::BinaryValue::CreateWithCopiedBuffer("mojo", 4)));
+ dict.Set("some_binary", base::BinaryValue::CreateWithCopiedBuffer("mojo", 4));
{
std::unique_ptr<base::ListValue> dict_list(new base::ListValue());
dict_list->AppendString("string");
@@ -187,8 +185,7 @@ TEST_F(CommonCustomTypesTest, Value) {
list.AppendString("string");
list.AppendDouble(42.1);
list.AppendBoolean(true);
- list.Append(
- base::WrapUnique(base::BinaryValue::CreateWithCopiedBuffer("mojo", 4)));
+ list.Append(base::BinaryValue::CreateWithCopiedBuffer("mojo", 4));
{
std::unique_ptr<base::DictionaryValue> list_dict(
new base::DictionaryValue());

Powered by Google App Engine
This is Rietveld 408576698