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

Unified Diff: mojo/public/cpp/system/tests/watcher_unittest.cc

Issue 2623263005: Tag some of Mojo heap allocations for the heap profiler. (Closed)
Patch Set: Synced 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
« no previous file with comments | « mojo/public/cpp/bindings/lib/multiplex_router.cc ('k') | mojo/public/cpp/system/watcher.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/public/cpp/system/tests/watcher_unittest.cc
diff --git a/mojo/public/cpp/system/tests/watcher_unittest.cc b/mojo/public/cpp/system/tests/watcher_unittest.cc
index ebeebf6777888a67aeeba7af1b6bf0290cd2d312..9b59240c5dcdf82b865ed07f53a5c06ee79af17d 100644
--- a/mojo/public/cpp/system/tests/watcher_unittest.cc
+++ b/mojo/public/cpp/system/tests/watcher_unittest.cc
@@ -48,7 +48,7 @@ TEST_F(WatcherTest, WatchBasic) {
bool notified = false;
base::RunLoop run_loop;
- Watcher b_watcher;
+ Watcher b_watcher(FROM_HERE);
EXPECT_EQ(MOJO_RESULT_OK,
b_watcher.Start(b.get(), MOJO_HANDLE_SIGNAL_READABLE,
OnReady([&] (MojoResult result) {
@@ -71,7 +71,7 @@ TEST_F(WatcherTest, WatchUnsatisfiable) {
CreateMessagePipe(nullptr, &a, &b);
a.reset();
- Watcher b_watcher;
+ Watcher b_watcher(FROM_HERE);
EXPECT_EQ(MOJO_RESULT_FAILED_PRECONDITION,
b_watcher.Start(b.get(), MOJO_HANDLE_SIGNAL_READABLE,
NotReached()));
@@ -84,7 +84,7 @@ TEST_F(WatcherTest, WatchInvalidHandle) {
a.reset();
b.reset();
- Watcher b_watcher;
+ Watcher b_watcher(FROM_HERE);
EXPECT_EQ(MOJO_RESULT_INVALID_ARGUMENT,
b_watcher.Start(b.get(), MOJO_HANDLE_SIGNAL_READABLE,
NotReached()));
@@ -96,7 +96,7 @@ TEST_F(WatcherTest, Cancel) {
CreateMessagePipe(nullptr, &a, &b);
base::RunLoop run_loop;
- Watcher b_watcher;
+ Watcher b_watcher(FROM_HERE);
EXPECT_EQ(MOJO_RESULT_OK,
b_watcher.Start(b.get(), MOJO_HANDLE_SIGNAL_READABLE,
NotReached()));
@@ -118,7 +118,7 @@ TEST_F(WatcherTest, CancelOnClose) {
CreateMessagePipe(nullptr, &a, &b);
base::RunLoop run_loop;
- Watcher b_watcher;
+ Watcher b_watcher(FROM_HERE);
EXPECT_EQ(MOJO_RESULT_OK,
b_watcher.Start(b.get(), MOJO_HANDLE_SIGNAL_READABLE,
OnReady([&] (MojoResult result) {
@@ -140,7 +140,7 @@ TEST_F(WatcherTest, CancelOnDestruction) {
CreateMessagePipe(nullptr, &a, &b);
base::RunLoop run_loop;
{
- Watcher b_watcher;
+ Watcher b_watcher(FROM_HERE);
EXPECT_EQ(MOJO_RESULT_OK,
b_watcher.Start(b.get(), MOJO_HANDLE_SIGNAL_READABLE,
NotReached()));
@@ -161,7 +161,7 @@ TEST_F(WatcherTest, CloseAndCancel) {
ScopedMessagePipeHandle a, b;
CreateMessagePipe(nullptr, &a, &b);
- Watcher b_watcher;
+ Watcher b_watcher(FROM_HERE);
EXPECT_EQ(MOJO_RESULT_OK,
b_watcher.Start(b.get(), MOJO_HANDLE_SIGNAL_READABLE,
OnReady([](MojoResult result) { FAIL(); })));
« no previous file with comments | « mojo/public/cpp/bindings/lib/multiplex_router.cc ('k') | mojo/public/cpp/system/watcher.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698