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

Unified Diff: remoting/host/desktop_process.h

Issue 11231060: [Chromoting] The desktop process now creates a pre-connected pipe and passes (with some help of the… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixup Created 8 years, 2 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
Index: remoting/host/desktop_process.h
diff --git a/remoting/host/desktop_process.h b/remoting/host/desktop_process.h
index 9f364719549869cce9ddfc7809f33d264b4e4c0e..e27372a5c3c0264882a0e68a16ef7d16421b200f 100644
--- a/remoting/host/desktop_process.h
+++ b/remoting/host/desktop_process.h
@@ -18,10 +18,12 @@ class ChannelProxy;
namespace remoting {
+class DesktopSessionAgent;
+
class DesktopProcess : public IPC::Listener {
public:
explicit DesktopProcess(const std::string& daemon_channel_name);
- ~DesktopProcess();
+ virtual ~DesktopProcess();
// IPC::Listener implementation.
virtual bool OnMessageReceived(const IPC::Message& message) OVERRIDE;
@@ -32,13 +34,24 @@ class DesktopProcess : public IPC::Listener {
int Run();
private:
- // IPC channel connecting the desktop process with the daemon process.
- scoped_ptr<IPC::ChannelProxy> daemon_channel_;
+ // Crashes the process in response to a daemon's request. The daemon passes
+ // the location of the code that detected the fatal error resulted in this
+ // request.
Sergey Ulanov 2012/10/24 20:12:01 Not sure I understand why we need this. Is it beca
alexeypa (please no reviews) 2012/10/24 21:41:51 It is explained at the declaration of ChromotingDa
+ void OnCrash(const std::string& function_name,
+ const std::string& file_name,
+ const int& line_number);
// Name of the IPC channel connecting the desktop process with the daemon
// process.
std::string daemon_channel_name_;
+ // IPC channel connecting the desktop process with the daemon process.
+ scoped_ptr<IPC::ChannelProxy> daemon_channel_;
+
+ // Provides screen/audio capturing and input injection services for
+ // the network process.
+ scoped_ptr<DesktopSessionAgent> desktop_agent_;
+
DISALLOW_COPY_AND_ASSIGN(DesktopProcess);
};

Powered by Google App Engine
This is Rietveld 408576698