Remote assistance on Chrome OS Part IV - It2MeHost

remoting/host/continue_window_chromeos.cc

+// 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.
+
+#include "remoting/host/continue_window.h"
+
+namespace remoting {
+
+// A place holder implementation for the ContinueWindow on
+// ChromeOS. Remote assistance on Chrome OS is currently under a flag so it is
+// secure to leave it unimplemented for now.
+class ContinueWindowAura : public ContinueWindow {

[Wez, 2014/10/17 17:58:00]

This should be inside an anonymous namespace.

[kelvinp, 2014/10/20 00:21:17]

Done.

+ public:
+  ContinueWindowAura();
+  virtual ~ContinueWindowAura();
+
+ protected:
+  // ContinueWindow overrides.

[Wez, 2014/10/17 17:58:00]

nit: s/overrides/interface

[kelvinp, 2014/10/20 00:21:17]

Done.

+  virtual void ShowUi() OVERRIDE;
+  virtual void HideUi() OVERRIDE;
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(ContinueWindowAura);
+};
+
+ContinueWindowAura::ContinueWindowAura() {
+}
+
+ContinueWindowAura::~ContinueWindowAura() {
+}
+
+void ContinueWindowAura::ShowUi() {
+  // TODO(kelvinp): Implement this on Chrome OS.

[Wez, 2014/10/17 17:58:00]

Is there a bug # for that work?

[kelvinp, 2014/10/20 00:21:17]

Done.

+}
+
+void ContinueWindowAura::HideUi() {
+  // TODO(kelvinp): Implement this on Chrome OS.
+}
+
+// static
+scoped_ptr<HostWindow> HostWindow::CreateContinueWindow() {
+  return scoped_ptr<HostWindow>(new ContinueWindowAura());

[Wez, 2014/10/17 17:58:00]

make_scoped_ptr(new ...)

[kelvinp, 2014/10/20 00:21:17]

Done.

+}
+
+}  // namespace remoting