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

Unified Diff: mojo/shell/child_process_host.h

Issue 775343004: Move //mojo/shell to //shell (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 6 years 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/shell/child_process.cc ('k') | mojo/shell/child_process_host.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/shell/child_process_host.h
diff --git a/mojo/shell/child_process_host.h b/mojo/shell/child_process_host.h
deleted file mode 100644
index 6e94b9616b15e1571f3b0136775cbc356b9e4835..0000000000000000000000000000000000000000
--- a/mojo/shell/child_process_host.h
+++ /dev/null
@@ -1,84 +0,0 @@
-// Copyright 2014 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.
-
-#ifndef MOJO_SHELL_CHILD_PROCESS_HOST_H_
-#define MOJO_SHELL_CHILD_PROCESS_HOST_H_
-
-#include "base/macros.h"
-#include "base/process/process_handle.h"
-#include "mojo/edk/embedder/platform_channel_pair.h"
-#include "mojo/edk/embedder/scoped_platform_handle.h"
-#include "mojo/shell/child_process.h" // For |ChildProcess::Type|.
-
-namespace mojo {
-namespace shell {
-
-class Context;
-
-// (Base) class for a "child process host". Handles launching and connecting a
-// platform-specific "pipe" to the child, and supports joining the child
-// process. Intended for use as a base class, but may be used on its own in
-// simple cases.
-//
-// This class is not thread-safe. It should be created/used/destroyed on a
-// single thread.
-//
-// Note: Does not currently work on Windows before Vista.
-class ChildProcessHost {
- public:
- class Delegate {
- public:
- virtual void WillStart() = 0;
- virtual void DidStart(bool success) = 0;
- };
-
- ChildProcessHost(Context* context,
- Delegate* delegate,
- ChildProcess::Type type);
- virtual ~ChildProcessHost();
-
- // |Start()|s the child process; calls the delegate's |DidStart()| (on the
- // thread on which |Start()| was called) when the child has been started (or
- // failed to start). After calling |Start()|, this object must not be
- // destroyed until |DidStart()| has been called.
- // TODO(vtl): Consider using weak pointers and removing this requirement.
- void Start();
-
- // Waits for the child process to terminate, and returns its exit code.
- // Note: If |Start()| has been called, this must not be called until the
- // callback has been called.
- int Join();
-
- embedder::ScopedPlatformHandle* platform_channel() {
- return &platform_channel_;
- }
-
- protected:
- Context* context() const {
- return context_;
- }
-
- private:
- bool DoLaunch();
- void DidLaunch(bool success);
-
- Context* const context_;
- Delegate* const delegate_;
- const ChildProcess::Type type_;
-
- base::ProcessHandle child_process_handle_;
-
- embedder::PlatformChannelPair platform_channel_pair_;
-
- // Platform-specific "pipe" to the child process. Valid immediately after
- // creation.
- embedder::ScopedPlatformHandle platform_channel_;
-
- DISALLOW_COPY_AND_ASSIGN(ChildProcessHost);
-};
-
-} // namespace shell
-} // namespace mojo
-
-#endif // MOJO_SHELL_CHILD_PROCESS_HOST_H_
« no previous file with comments | « mojo/shell/child_process.cc ('k') | mojo/shell/child_process_host.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698