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

Unified Diff: components/sync/engine_impl/syncer_util_unittest.cc

Issue 2422253002: [Sync] Rewriting ".reset(new" pattern to use "= base::MakeUnique" instead. (Closed)
Patch Set: Fixing compile. Created 4 years, 2 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: components/sync/engine_impl/syncer_util_unittest.cc
diff --git a/components/sync/engine_impl/syncer_util_unittest.cc b/components/sync/engine_impl/syncer_util_unittest.cc
index 5c64a612898a8c98ef69f3cfb72733ffae28cfd9..5ccd27007f80b11550a097cd1aca6f61ee5362aa 100644
--- a/components/sync/engine_impl/syncer_util_unittest.cc
+++ b/components/sync/engine_impl/syncer_util_unittest.cc
@@ -6,6 +6,7 @@
#include <memory>
+#include "base/memory/ptr_util.h"
#include "base/message_loop/message_loop.h"
#include "base/rand_util.h"
#include "components/sync/base/unique_position.h"
@@ -21,7 +22,7 @@ class GetUpdatePositionTest : public ::testing::Test {
public:
void SetUp() override {
dir_maker_.SetUp();
- entry_factory_.reset(new TestEntryFactory(directory()));
+ entry_factory_ = base::MakeUnique<TestEntryFactory>(directory());
}
void TearDown() override { dir_maker_.TearDown(); }
« no previous file with comments | « components/sync/engine_impl/syncer_unittest.cc ('k') | components/sync/engine_impl/worker_entity_tracker.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698