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

Unified Diff: mojo/edk/system/core.h

Issue 1675603002: [mojo-edk] Simplify multiprocess pipe bootstrap (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix some callers to work with sync APIs Created 4 years, 10 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/edk/embedder/embedder.cc ('k') | mojo/edk/system/core.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/edk/system/core.h
diff --git a/mojo/edk/system/core.h b/mojo/edk/system/core.h
index 8e9af8eddb5dfaf691d59dad3a24df1cb34c5bfa..603cba9082c8e8ed6d7af494f5b7d0e256da2481 100644
--- a/mojo/edk/system/core.h
+++ b/mojo/edk/system/core.h
@@ -52,26 +52,19 @@ class MOJO_SYSTEM_IMPL_EXPORT Core {
// Called in a child process exactly once during early initialization.
void InitChild(ScopedPlatformHandle platform_handle);
- // This creates a message pipe endpoint connected to an endpoint in a remote
+ // Creates a message pipe endpoint connected to an endpoint in a remote
// embedder. |platform_handle| is used as a channel to negotiate the
- // connection. This is only here to facilitate legacy embedder code. See
- // mojo::edk::CreateMessagePipe in mojo/edk/embedder/embedder.h.
- void CreateMessagePipe(
- ScopedPlatformHandle platform_handle,
- const base::Callback<void(ScopedMessagePipeHandle)>& callback);
+ // connection.
+ ScopedMessagePipeHandle CreateMessagePipe(
+ ScopedPlatformHandle platform_handle);
// Creates a message pipe endpoint associated with |token|, which a child
// holding the token can later locate and connect to.
- void CreateParentMessagePipe(
- const std::string& token,
- const base::Callback<void(ScopedMessagePipeHandle)>& callback);
-
- // Creates a message pipe endpoint associated with |token|, which will be
- // passed to the parent in order to find an associated remote port and connect
- // to it.
- void CreateChildMessagePipe(
- const std::string& token,
- const base::Callback<void(ScopedMessagePipeHandle)>& callback);
+ ScopedMessagePipeHandle CreateParentMessagePipe(const std::string& token);
+
+ // Creates a message pipe endpoint and connects it to a pipe the parent has
+ // associated with |token|.
+ ScopedMessagePipeHandle CreateChildMessagePipe(const std::string& token);
MojoHandle AddDispatcher(scoped_refptr<Dispatcher> dispatcher);
« no previous file with comments | « mojo/edk/embedder/embedder.cc ('k') | mojo/edk/system/core.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698