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

Unified Diff: ipc/mojo_event.cc

Issue 2754143005: Use WaitableEvents to wake up sync IPC waiting (Closed)
Patch Set: . Created 3 years, 9 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 | « ipc/mojo_event.h ('k') | mojo/public/cpp/bindings/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ipc/mojo_event.cc
diff --git a/ipc/mojo_event.cc b/ipc/mojo_event.cc
deleted file mode 100644
index 5d802b8bff9721224756d53f670bdbab26af6dcb..0000000000000000000000000000000000000000
--- a/ipc/mojo_event.cc
+++ /dev/null
@@ -1,39 +0,0 @@
-// Copyright 2016 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "ipc/mojo_event.h"
-
-namespace IPC {
-
-MojoEvent::MojoEvent() {
- mojo::MessagePipe pipe;
- signal_handle_ = std::move(pipe.handle0);
- wait_handle_ = std::move(pipe.handle1);
-}
-
-MojoEvent::~MojoEvent() {}
-
-void MojoEvent::Signal() {
- base::AutoLock lock(lock_);
- if (is_signaled_)
- return;
- is_signaled_ = true;
- MojoResult rv = mojo::WriteMessageRaw(
- signal_handle_.get(), nullptr, 0, nullptr, 0,
- MOJO_WRITE_MESSAGE_FLAG_NONE);
- CHECK_EQ(rv, MOJO_RESULT_OK);
-}
-
-void MojoEvent::Reset() {
- base::AutoLock lock(lock_);
- if (!is_signaled_)
- return;
- is_signaled_ = false;
- MojoResult rv = mojo::ReadMessageRaw(
- wait_handle_.get(), nullptr, nullptr, nullptr, nullptr,
- MOJO_READ_MESSAGE_FLAG_NONE);
- CHECK_EQ(rv, MOJO_RESULT_OK);
-}
-
-} // namespace IPC
« no previous file with comments | « ipc/mojo_event.h ('k') | mojo/public/cpp/bindings/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698