Host extensions

remoting/host/host_extension.h

Index: remoting/host/host_extension.h
diff --git a/remoting/host/host_extension.h b/remoting/host/host_extension.h
new file mode 100644
index 0000000000000000000000000000000000000000..e8fbc84771a15da2aff67c4359b2c3f4be14271e
--- /dev/null
+++ b/remoting/host/host_extension.h
@@ -0,0 +1,40 @@
+// 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 REMOTING_HOST_HOST_EXTENSION_H_
+#define REMOTING_HOST_HOST_EXTENSION_H_
+
+#include <string>
+
+#include "base/memory/scoped_ptr.h"
+
+namespace remoting {
+
+class ClientSession;
+class HostExtensionSession;
+
+// Extends |ChromotingHost| with new functionality, and can use extension
+// messages to communicate with the client.
+class HostExtension {
+ public:
+  virtual ~HostExtension() {}
+
+  // Returns a space-separated list of capabilities provided by this extension.
+  // Capabilities are used to inform the client of the availability of an

[Wez, 2014/05/28 23:03:41]

nit: "... may be used ..." - we don't require this; it's just a tool for
extensions that need to be checked for?

[dcaiafa, 2014/05/29 00:03:16]

Done.

+  // extension. They are merged into the capabilities the host reports to the
+  // client.
+  virtual std::string GetCapabilities() = 0;
+
+  // Creates an extension session, which can handle extension messages for a
+  // client session.
+  // If NULL is returned, then the extension will not be available to the client
+  // session.

[Wez, 2014/05/28 23:03:41]

nit: I'd suggest "NULL may be returned if |client_session| cannot support this
extension."

[dcaiafa, 2014/05/29 00:03:16]

Done.

+  // |client_session| must outlive the resulting |HostExtensionSession|.
+  virtual scoped_ptr<HostExtensionSession> CreateExtensionSession(
+      ClientSession* client_session) = 0;
+};
+
+}  // namespace remoting
+
+#endif  // REMOTING_HOST_HOST_EXTENSION_H_