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

Unified Diff: chrome/browser/metrics/subprocess_metrics_provider_unittest.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/metrics/subprocess_metrics_provider_unittest.cc
diff --git a/chrome/browser/metrics/subprocess_metrics_provider_unittest.cc b/chrome/browser/metrics/subprocess_metrics_provider_unittest.cc
index 5b3e641d9eae898dd621b24099ac8a9dbfb78a4f..1db5c1e2c580df3220f3a0a15b1549d6d285ba24 100644
--- a/chrome/browser/metrics/subprocess_metrics_provider_unittest.cc
+++ b/chrome/browser/metrics/subprocess_metrics_provider_unittest.cc
@@ -5,6 +5,7 @@
#include "chrome/browser/metrics/subprocess_metrics_provider.h"
#include <memory>
+#include <string>
#include "base/memory/ptr_util.h"
#include "base/metrics/histogram.h"
@@ -85,10 +86,10 @@ class SubprocessMetricsProviderTest : public testing::Test {
std::unique_ptr<base::PersistentHistogramAllocator> CreateDuplicateAllocator(
base::PersistentHistogramAllocator* allocator) {
// Just wrap around the data segment in-use by the passed allocator.
- return WrapUnique(new base::PersistentHistogramAllocator(
- WrapUnique(new base::PersistentMemoryAllocator(
- const_cast<void*>(allocator->data()), allocator->length(),
- 0, 0, "", false))));
+ return base::MakeUnique<base::PersistentHistogramAllocator>(
+ base::MakeUnique<base::PersistentMemoryAllocator>(
+ const_cast<void*>(allocator->data()), allocator->length(), 0, 0,
+ std::string(), false));
}
size_t GetSnapshotHistogramCount() {
« no previous file with comments | « chrome/browser/metrics/subprocess_metrics_provider.cc ('k') | chrome/browser/net/disk_cache_dir_policy_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698