Index: extensions/common/permissions/api_permission_set_unittest.cc |
diff --git a/extensions/common/permissions/api_permission_set_unittest.cc b/extensions/common/permissions/api_permission_set_unittest.cc |
index b0acc83b141f0ce394de114300c1fd5d336f51ad..4101574473e3c036befcfe6e8e8942a25df2ea46 100644 |
--- a/extensions/common/permissions/api_permission_set_unittest.cc |
+++ b/extensions/common/permissions/api_permission_set_unittest.cc |
@@ -45,9 +45,9 @@ TEST(APIPermissionSetTest, CreateUnion) { |
permission = permission_info->CreateAPIPermission(); |
{ |
std::unique_ptr<base::ListValue> value(new base::ListValue()); |
- value->Append(new base::StringValue("tcp-connect:*.example.com:80")); |
- value->Append(new base::StringValue("udp-bind::8080")); |
- value->Append(new base::StringValue("udp-send-to::8888")); |
+ value->AppendString("tcp-connect:*.example.com:80"); |
+ value->AppendString("udp-bind::8080"); |
+ value->AppendString("udp-send-to::8888"); |
ASSERT_TRUE(permission->FromValue(value.get(), NULL, NULL)); |
} |
@@ -80,8 +80,8 @@ TEST(APIPermissionSetTest, CreateUnion) { |
permission = permission_info->CreateAPIPermission(); |
{ |
std::unique_ptr<base::ListValue> value(new base::ListValue()); |
- value->Append(new base::StringValue("tcp-connect:*.example.com:80")); |
- value->Append(new base::StringValue("udp-send-to::8899")); |
+ value->AppendString("tcp-connect:*.example.com:80"); |
+ value->AppendString("udp-send-to::8899"); |
ASSERT_TRUE(permission->FromValue(value.get(), NULL, NULL)); |
} |
apis2.insert(permission); |
@@ -94,10 +94,10 @@ TEST(APIPermissionSetTest, CreateUnion) { |
permission = permission_info->CreateAPIPermission(); |
{ |
std::unique_ptr<base::ListValue> value(new base::ListValue()); |
- value->Append(new base::StringValue("tcp-connect:*.example.com:80")); |
- value->Append(new base::StringValue("udp-bind::8080")); |
- value->Append(new base::StringValue("udp-send-to::8888")); |
- value->Append(new base::StringValue("udp-send-to::8899")); |
+ value->AppendString("tcp-connect:*.example.com:80"); |
+ value->AppendString("udp-bind::8080"); |
+ value->AppendString("udp-send-to::8888"); |
+ value->AppendString("udp-send-to::8899"); |
ASSERT_TRUE(permission->FromValue(value.get(), NULL, NULL)); |
} |
// Insert a new socket permission which will replace the old one. |
@@ -132,9 +132,9 @@ TEST(APIPermissionSetTest, CreateIntersection) { |
permission = permission_info->CreateAPIPermission(); |
{ |
std::unique_ptr<base::ListValue> value(new base::ListValue()); |
- value->Append(new base::StringValue("tcp-connect:*.example.com:80")); |
- value->Append(new base::StringValue("udp-bind::8080")); |
- value->Append(new base::StringValue("udp-send-to::8888")); |
+ value->AppendString("tcp-connect:*.example.com:80"); |
+ value->AppendString("udp-bind::8080"); |
+ value->AppendString("udp-send-to::8888"); |
ASSERT_TRUE(permission->FromValue(value.get(), NULL, NULL)); |
} |
apis1.insert(permission); |
@@ -160,9 +160,9 @@ TEST(APIPermissionSetTest, CreateIntersection) { |
permission = permission_info->CreateAPIPermission(); |
{ |
std::unique_ptr<base::ListValue> value(new base::ListValue()); |
- value->Append(new base::StringValue("udp-bind::8080")); |
- value->Append(new base::StringValue("udp-send-to::8888")); |
- value->Append(new base::StringValue("udp-send-to::8899")); |
+ value->AppendString("udp-bind::8080"); |
+ value->AppendString("udp-send-to::8888"); |
+ value->AppendString("udp-send-to::8899"); |
ASSERT_TRUE(permission->FromValue(value.get(), NULL, NULL)); |
} |
apis2.insert(permission); |
@@ -171,8 +171,8 @@ TEST(APIPermissionSetTest, CreateIntersection) { |
permission = permission_info->CreateAPIPermission(); |
{ |
std::unique_ptr<base::ListValue> value(new base::ListValue()); |
- value->Append(new base::StringValue("udp-bind::8080")); |
- value->Append(new base::StringValue("udp-send-to::8888")); |
+ value->AppendString("udp-bind::8080"); |
+ value->AppendString("udp-send-to::8888"); |
ASSERT_TRUE(permission->FromValue(value.get(), NULL, NULL)); |
} |
expected_apis.insert(permission); |
@@ -206,9 +206,9 @@ TEST(APIPermissionSetTest, CreateDifference) { |
permission = permission_info->CreateAPIPermission(); |
{ |
std::unique_ptr<base::ListValue> value(new base::ListValue()); |
- value->Append(new base::StringValue("tcp-connect:*.example.com:80")); |
- value->Append(new base::StringValue("udp-bind::8080")); |
- value->Append(new base::StringValue("udp-send-to::8888")); |
+ value->AppendString("tcp-connect:*.example.com:80"); |
+ value->AppendString("udp-bind::8080"); |
+ value->AppendString("udp-send-to::8888"); |
ASSERT_TRUE(permission->FromValue(value.get(), NULL, NULL)); |
} |
apis1.insert(permission); |
@@ -226,8 +226,8 @@ TEST(APIPermissionSetTest, CreateDifference) { |
permission = permission_info->CreateAPIPermission(); |
{ |
std::unique_ptr<base::ListValue> value(new base::ListValue()); |
- value->Append(new base::StringValue("tcp-connect:*.example.com:80")); |
- value->Append(new base::StringValue("udp-send-to::8899")); |
+ value->AppendString("tcp-connect:*.example.com:80"); |
+ value->AppendString("udp-send-to::8899"); |
ASSERT_TRUE(permission->FromValue(value.get(), NULL, NULL)); |
} |
apis2.insert(permission); |
@@ -236,8 +236,8 @@ TEST(APIPermissionSetTest, CreateDifference) { |
permission = permission_info->CreateAPIPermission(); |
{ |
std::unique_ptr<base::ListValue> value(new base::ListValue()); |
- value->Append(new base::StringValue("udp-bind::8080")); |
- value->Append(new base::StringValue("udp-send-to::8888")); |
+ value->AppendString("udp-bind::8080"); |
+ value->AppendString("udp-send-to::8888"); |
ASSERT_TRUE(permission->FromValue(value.get(), NULL, NULL)); |
} |
expected_apis.insert(permission); |
@@ -269,9 +269,9 @@ TEST(APIPermissionSetTest, IPC) { |
permission = permission_info->CreateAPIPermission(); |
{ |
std::unique_ptr<base::ListValue> value(new base::ListValue()); |
- value->Append(new base::StringValue("tcp-connect:*.example.com:80")); |
- value->Append(new base::StringValue("udp-bind::8080")); |
- value->Append(new base::StringValue("udp-send-to::8888")); |
+ value->AppendString("tcp-connect:*.example.com:80"); |
+ value->AppendString("udp-bind::8080"); |
+ value->AppendString("udp-send-to::8888"); |
ASSERT_TRUE(permission->FromValue(value.get(), NULL, NULL)); |
} |
apis.insert(permission); |