CastExtension Impl for Chromoting Host

remoting/host/cast_extension.h

+// 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_CAST_EXTENSION_H_
+#define REMOTING_HOST_CAST_EXTENSION_H_
+
+#include <string>
+
+#include "base/memory/scoped_ptr.h"
+#include "net/url_request/url_request_context_getter.h"

[Wez, 2014/08/07 01:25:29]

Forward define this rather than include?

[aiguha, 2014/08/12 01:42:36]

Done.

+#include "remoting/host/host_extension.h"
+
+namespace base {
+class SingleThreadTaskRunner;
+}  // namespace base
+
+namespace remoting {
+
+namespace protocol {
+struct NetworkSettings;
+}  // namespace protocol
+
+// CastExtension extends HostExtension to enable WebRTC support.
+class CastExtension : public HostExtension {
+ public:
+  CastExtension(
+      scoped_refptr<base::SingleThreadTaskRunner> network_task_runner,
+      scoped_refptr<net::URLRequestContextGetter> url_request_context_getter,
+      const protocol::NetworkSettings& network_settings);
+  virtual ~CastExtension();
+
+  virtual std::string capability() const OVERRIDE;

[Wez, 2014/08/07 01:25:29]

nit: Suggest comment "HostExtension interface." to precede these.

[aiguha, 2014/08/12 01:42:35]

Done.

+  virtual scoped_ptr<HostExtensionSession> CreateExtensionSession(
+      ClientSessionControl* client_session_control,
+      protocol::ClientStub* client_stub) OVERRIDE;
+
+ private:
+  // Parameters passed to CastExtensionSession on creation.

[Wez, 2014/08/07 01:25:29]

nit: This comment is not necessary, since there are no other members to
distinguish these from.

[aiguha, 2014/08/12 01:42:35]

Acknowledged.

+  scoped_refptr<base::SingleThreadTaskRunner> network_task_runner_;
+  scoped_refptr<net::URLRequestContextGetter> url_request_context_getter_;
+  const protocol::NetworkSettings& network_settings_;
+};

[Wez, 2014/08/07 01:25:29]

DISALLOW_COPY_AND_ASSIGN(...)

[aiguha, 2014/08/12 01:42:35]

Done.

+
+}  // namespace remoting
+
+#endif  // REMOTING_HOST_CAST_EXTENSION_H_
+