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

Unified Diff: chrome/browser/android/data_usage/data_use_matcher.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/android/data_usage/data_use_matcher.cc
diff --git a/chrome/browser/android/data_usage/data_use_matcher.cc b/chrome/browser/android/data_usage/data_use_matcher.cc
index b00c9e7ba55ef2b2a0ca2f4c9a3d8536be30aac3..297511318688072b502d72d7d8cf9be3bcb9f371 100644
--- a/chrome/browser/android/data_usage/data_use_matcher.cc
+++ b/chrome/browser/android/data_usage/data_use_matcher.cc
@@ -80,8 +80,8 @@ void DataUseMatcher::RegisterURLRegexes(
if (expiration <= now_ticks)
continue; // skip expired matching rules.
DCHECK(!labels.at(i).empty());
- matching_rules_.push_back(base::WrapUnique(new MatchingRule(
- app_package_name, std::move(pattern), labels.at(i), expiration)));
+ matching_rules_.push_back(base::MakeUnique<MatchingRule>(
+ app_package_name, std::move(pattern), labels.at(i), expiration));
removed_matching_rule_labels.erase(labels.at(i));
}

Powered by Google App Engine
This is Rietveld 408576698