Cross-platform plumbing for resize-to-client

remoting/host/desktop_resizer_linux.cc

Index: remoting/host/desktop_resizer_linux.cc
diff --git a/remoting/host/desktop_resizer_linux.cc b/remoting/host/desktop_resizer_linux.cc
new file mode 100644
index 0000000000000000000000000000000000000000..66fe413ea208d152437e82cf24d79f9b6fbafd6e
--- /dev/null
+++ b/remoting/host/desktop_resizer_linux.cc
@@ -0,0 +1,45 @@
+// 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.
+
+#include "remoting/host/desktop_resizer.h"
+
+#include "base/logging.h"
+
+namespace remoting {
+
+namespace {
+class DesktopResizerLinux : public DesktopResizer {
+ public:
+  DesktopResizerLinux() {
+  }
+

[tfarina, 2012/09/24 22:52:07]

this is missing a virtual dtor.

[Jamie, 2012/09/24 23:11:20]

The base class has one. Is it necessary to re-declare it here? I couldn't find
anything in the style guide requiring that.

+  virtual SkISize GetCurrentSize() OVERRIDE {
+    NOTIMPLEMENTED();
+    return SkISize::Make(0, 0);
+  }
+
+  virtual std::list<SkISize> GetSupportedSizes(
+      const SkISize& preferred) OVERRIDE {
+    NOTIMPLEMENTED();
+    return std::list<SkISize>();
+  }
+
+  virtual void SetSize(const SkISize& size) OVERRIDE {
+    NOTIMPLEMENTED();
+  }
+
+  virtual void RestoreSize(const SkISize& original) OVERRIDE {
+    NOTIMPLEMENTED();
+  }
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(DesktopResizerLinux);
+};
+}  // namespace
+
+scoped_ptr<DesktopResizer> DesktopResizer::Create() {
+  return scoped_ptr<DesktopResizer>(new DesktopResizerLinux);
+}
+
+}  // namespace remoting