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

Unified Diff: content/renderer/gpu/frame_swap_message_queue_unittest.cc

Issue 2253053003: Re-write many calls to WrapUnique() with MakeUnique() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 4 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
« no previous file with comments | « content/renderer/gpu/frame_swap_message_queue.cc ('k') | content/renderer/history_controller.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/gpu/frame_swap_message_queue_unittest.cc
diff --git a/content/renderer/gpu/frame_swap_message_queue_unittest.cc b/content/renderer/gpu/frame_swap_message_queue_unittest.cc
index 05016405906a28553bef7fea6e899f47b9085cb5..8488b559b55c4a849ee2d8d3dc4b837a36d6e5c3 100644
--- a/content/renderer/gpu/frame_swap_message_queue_unittest.cc
+++ b/content/renderer/gpu/frame_swap_message_queue_unittest.cc
@@ -65,7 +65,7 @@ class FrameSwapMessageQueueTest : public testing::Test {
}
std::unique_ptr<IPC::Message> CloneMessage(const IPC::Message& other) {
- return base::WrapUnique(new IPC::Message(other));
+ return base::MakeUnique<IPC::Message>(other);
}
void TestDidNotSwap(cc::SwapPromise::DidNotSwapReason reason);
@@ -264,24 +264,24 @@ class NotifiesDeletionMessage : public IPC::Message {
TEST_F(FrameSwapMessageQueueTest, TestDeletesNextSwapMessage) {
bool message_deleted = false;
- QueueNextSwapMessage(base::WrapUnique(
- new NotifiesDeletionMessage(&message_deleted, first_message_)));
+ QueueNextSwapMessage(base::MakeUnique<NotifiesDeletionMessage>(
+ &message_deleted, first_message_));
queue_ = NULL;
ASSERT_TRUE(message_deleted);
}
TEST_F(FrameSwapMessageQueueTest, TestDeletesVisualStateMessage) {
bool message_deleted = false;
- QueueVisualStateMessage(1, base::WrapUnique(new NotifiesDeletionMessage(
- &message_deleted, first_message_)));
+ QueueVisualStateMessage(1, base::MakeUnique<NotifiesDeletionMessage>(
+ &message_deleted, first_message_));
queue_ = NULL;
ASSERT_TRUE(message_deleted);
}
TEST_F(FrameSwapMessageQueueTest, TestDeletesQueuedVisualStateMessage) {
bool message_deleted = false;
- QueueVisualStateMessage(1, base::WrapUnique(new NotifiesDeletionMessage(
- &message_deleted, first_message_)));
+ QueueVisualStateMessage(1, base::MakeUnique<NotifiesDeletionMessage>(
+ &message_deleted, first_message_));
queue_->DidActivate(1);
queue_->DidSwap(1);
queue_ = NULL;
« no previous file with comments | « content/renderer/gpu/frame_swap_message_queue.cc ('k') | content/renderer/history_controller.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698