Index: ios/chrome/browser/ui/settings/block_popups_collection_view_controller.mm |
diff --git a/ios/chrome/browser/ui/settings/block_popups_collection_view_controller.mm b/ios/chrome/browser/ui/settings/block_popups_collection_view_controller.mm |
index db548f635bed35228e6876b996dfba87d017434c..65d3cb70ea2502dad327cb19fa3d574d923f8e73 100644 |
--- a/ios/chrome/browser/ui/settings/block_popups_collection_view_controller.mm |
+++ b/ios/chrome/browser/ui/settings/block_popups_collection_view_controller.mm |
@@ -8,6 +8,7 @@ |
#include "base/logging.h" |
#import "base/mac/foundation_util.h" |
#import "base/mac/scoped_nsobject.h" |
+#include "base/memory/ptr_util.h" |
#include "base/strings/sys_string_conversions.h" |
#include "base/values.h" |
#include "components/content_settings/core/browser/host_content_settings_map.h" |
@@ -282,7 +283,7 @@ typedef NS_ENUM(NSInteger, ItemType) { |
if (entries[i].secondary_pattern == ContentSettingsPattern::Wildcard() && |
entries[i].setting == CONTENT_SETTING_ALLOW) { |
_exceptions.Append( |
- new base::Value(entries[i].primary_pattern.ToString())); |
+ base::MakeUnique<base::Value>(entries[i].primary_pattern.ToString())); |
jdoerrie
2017/04/12 12:37:46
Consider using AppendString and removing the added
vabr (Chromium)
2017/04/12 12:55:04
Done.
|
} else { |
LOG(ERROR) << "Secondary content settings patterns are not " |
<< "supported by the content settings UI"; |