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

Unified Diff: chrome/browser/browsing_data/browsing_data_remover.cc

Issue 2334613003: Re-write many calls to WrapUnique() with MakeUnique() (Closed)
Patch Set: Changes from review by sky Created 4 years, 3 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: chrome/browser/browsing_data/browsing_data_remover.cc
diff --git a/chrome/browser/browsing_data/browsing_data_remover.cc b/chrome/browser/browsing_data/browsing_data_remover.cc
index 7d457d09845a431819820f0d66511585048b3134..d2153987defa68a7b42a13b084ac31b325804b5b 100644
--- a/chrome/browser/browsing_data/browsing_data_remover.cc
+++ b/chrome/browser/browsing_data/browsing_data_remover.cc
@@ -407,8 +407,8 @@ void BrowsingDataRemover::RemoveInternal(
// Remove() and RemoveAndReply() pass a null pointer to indicate no filter.
// No filter is equivalent to one that |IsEmptyBlacklist()|.
if (!filter_builder) {
- filter_builder = base::WrapUnique(new RegistrableDomainFilterBuilder(
- RegistrableDomainFilterBuilder::BLACKLIST));
+ filter_builder = base::MakeUnique<RegistrableDomainFilterBuilder>(
+ RegistrableDomainFilterBuilder::BLACKLIST);
DCHECK(filter_builder->IsEmptyBlacklist());
}

Powered by Google App Engine
This is Rietveld 408576698