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

Unified Diff: remoting/host/plugin/daemon_controller.h

Issue 9316078: Added daemon process NPAPI. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Separated out Start/SetPin and added return codes to Stop and Start. Created 8 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 | « no previous file | remoting/host/plugin/daemon_controller_linux.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/host/plugin/daemon_controller.h
diff --git a/remoting/host/plugin/daemon_controller.h b/remoting/host/plugin/daemon_controller.h
new file mode 100644
index 0000000000000000000000000000000000000000..2ff19722132e69fe9e9d37a1281b26125bef198b
--- /dev/null
+++ b/remoting/host/plugin/daemon_controller.h
@@ -0,0 +1,74 @@
+// Copyright (c) 2012 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 REMOTING_HOST_DAEMON_CONTROLLER_H_
+#define REMOTING_HOST_DAEMON_CONTROLLER_H_
+
+#include <string>
+
+namespace remoting {
+
+class DaemonController {
+ public:
+ enum State {
+ // Placeholder state for platforms on which the daemon process is not
+ // implemented. The web-app will not show the corresponding UI. This value
+ // will eventually be deprecated or removed.
+ STATE_NOT_IMPLEMENTED = -1,
+ // The daemon process is not installed. This is functionally equivalent
+ // to STATE_STOPPED, but the start method is expected to be significantly
+ // slower, and might involve user interaction. It might be appropriate to
+ // indicate this in the UI.
+ STATE_NOT_INSTALLED = 0,
+ // The daemon process is installed but not running. Call Start to start it.
+ STATE_STOPPED = 1,
+ // The daemon process is running. Call Start again to change the PIN or
+ // Stop to stop it.
+ STATE_STARTED = 2,
+ // The previous Start operation failed. This is functionally equivalent
+ // to STATE_STOPPED, but the UI should report an error in this state.
+ // This state should not persist across restarts of the NPAPI process.
+ STATE_START_FAILED = 3,
+ // The state cannot be determined. This could indicate that the plugin
+ // has not been provided with sufficient information, for example, the
+ // user for which to query state on a multi-user system.
+ STATE_UNKNOWN = 4
+ };
+
+ virtual ~DaemonController() {}
+
+ // Return the "installed/running" state of the daemon process.
+ virtual State GetState() = 0;
+
+ // Set the PIN for accessing this host, which should be expressed as a
+ // UTF8-encoded string. It is permitted to call SetPin when the daemon
+ // is already running. Returns true if successful, or false if the PIN
+ // does not satisfy complexity requirements.
+ //
+ // TODO(jamiewalch): More state-setting methods needed here. Sufficient
Wez 2012/02/07 00:39:53 nit: They're not _needed_, unless they're... well.
+ // state must be set prior to calling any other DaemonController method;
+ // this should be documented for each method.
+ virtual bool SetPin(const std::string& pin) = 0;
+
+ // Start the daemon process. Since this may require that the daemon be
+ // downloaded and installed, this may take a long time--poll GetState
+ // until the state is STATE_STARTED or STATE_START_FAILED. Start fails
+ // synchronously and returns false if sufficient state has not been set,
+ // including a valid PIN.
+ virtual bool Start() = 0;
+
+ // Stop the daemon process. It is permitted to call Stop while the daemon
+ // process is being installed, in which case the installation should be
+ // aborted if possible; if not then it is sufficient to ensure that the
+ // daemon process is not started automatically upon successful installation.
+ // Returns false if sufficient state has not been set; Stop is not permitted
+ // to fail for any other reason.
+ virtual bool Stop() = 0;
+
+ static DaemonController* Create();
+};
+
+} // namespace remoting
+
+#endif // REMOTING_HOST_DAEMON_CONTROLLER_H_
« no previous file with comments | « no previous file | remoting/host/plugin/daemon_controller_linux.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698