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

Unified Diff: content/public/common/child_process_host_delegate.h

Issue 8774040: Don't make classes derive from ChildProcessHost, and instead have them use it through composition... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 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
« content/common/child_process_host.h ('K') | « content/content_common.gypi ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/public/common/child_process_host_delegate.h
===================================================================
--- content/public/common/child_process_host_delegate.h (revision 0)
+++ content/public/common/child_process_host_delegate.h (revision 0)
@@ -0,0 +1,36 @@
+// Copyright (c) 2011 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 CONTENT_PULIC_COMMON_CHILD_PROCESS_HOST_DELEGATE_H_
+#define CONTENT_PULIC_COMMON_CHILD_PROCESS_HOST_DELEGATE_H_
+#pragma once
+
+#include <string>
+
+#include "ipc/ipc_channel.h"
+
+namespace content {
+
+// Interface that all users of ChildProcessHost need to provide.
+class ChildProcessHostDelegate : public IPC::Channel::Listener {
+ public:
+ virtual ~ChildProcessHostDelegate() {}
+
+ // Derived classes return true if it's ok to shut down the child process.
+ // Normally they would return true. The exception is if the host is in the
+ // middle of sending a request to the process, in which case the other side
+ // might think it's ok to shutdown, when really it's not.
+ virtual bool CanShutdown() = 0;
+
+ // Notifies the derived class that we told the child process to kill itself.
+ virtual void ShutdownStarted() = 0;
+
+ // Called when the child process unexpected closes the IPC channel. Delegates
+ // would normally delete the object in this case.
+ virtual void OnChildDisconnected() = 0;
+};
+
+}; // namespace content
+
+#endif // CONTENT_PULIC_COMMON_CHILD_PROCESS_HOST_DELEGATE_H_
Property changes on: content\public\common\child_process_host_delegate.h
___________________________________________________________________
Added: svn:eol-style
+ LF
« content/common/child_process_host.h ('K') | « content/content_common.gypi ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698