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

Unified Diff: mojo/edk/system/multiprocess_message_pipe_unittest.cc

Issue 2623263005: Tag some of Mojo heap allocations for the heap profiler. (Closed)
Patch Set: Addressed comments + fixed build. Created 3 years, 11 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: mojo/edk/system/multiprocess_message_pipe_unittest.cc
diff --git a/mojo/edk/system/multiprocess_message_pipe_unittest.cc b/mojo/edk/system/multiprocess_message_pipe_unittest.cc
index 11bb4053ac625bb08e28a5ee23f79ff14a89ec96..ed5a3da84a326764ba4d33130252231f83e7374c 100644
--- a/mojo/edk/system/multiprocess_message_pipe_unittest.cc
+++ b/mojo/edk/system/multiprocess_message_pipe_unittest.cc
@@ -1312,7 +1312,7 @@ DEFINE_TEST_CLIENT_TEST_WITH_PIPE(MessagePipeStatusChangeInTransitClient,
// Wait on handle 1 using a Watcher.
{
base::RunLoop run_loop;
- Watcher watcher;
+ Watcher watcher(FROM_HERE);
watcher.Start(Handle(handles[1]), MOJO_HANDLE_SIGNAL_PEER_CLOSED,
base::Bind([] (base::RunLoop* loop, MojoResult result) {
EXPECT_EQ(MOJO_RESULT_OK, result);

Powered by Google App Engine
This is Rietveld 408576698