ui/platform_window: Introduce PlatformWindow for windowing primitives.

ui/platform_window/platform_window.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 UI_PLATFORM_WINDOW_PLATFORM_WINDOW_H_
+#define UI_PLATFORM_WINDOW_PLATFORM_WINDOW_H_
+
+#include "base/memory/scoped_ptr.h"
+#include "ui/platform_window/platform_window_export.h"
+
+namespace gfx {
+class Rect;
+}
+
+namespace ui {
+
+class PlatformWindowDelegate;
+
+class PLATFORM_WINDOW_EXPORT PlatformWindow {

[spang, 2014/07/14 17:02:29]

I added some comments here in my version (and on each method). Might be nice to
do this here as well.

[sadrul, 2014/07/14 17:36:20]

Done.

+ public:
+  virtual ~PlatformWindow() {}
+
+  virtual void Show() = 0;
+  virtual void Hide() = 0;
+  virtual void Close() = 0;
+
+  virtual void SetBounds(const gfx::Rect& bounds) = 0;

[spang, 2014/07/14 17:02:29]

Should we have GetBounds?

Then WindowTreeHost can implement GetBounds as platform_window_->GetBounds().

[sadrul, 2014/07/14 17:14:35]

I considered doing this, but maybe it makes sense to let WindowTreeHost maintain
the bounds since it will get notified of the bounds from the
PlatformWindowDelegate::OnBoundsChanged() callback. But then again, it would be
necessary for all delegates to do the same thing ... I will add GetBounds().

[sadrul, 2014/07/14 17:36:20]

Done.

+
+  virtual void SetCapture() = 0;
+  virtual void ReleaseCapture() = 0;
+
+  virtual void ToggleFullscreen() = 0;
+  virtual void Maximize() = 0;
+  virtual void Minimize() = 0;
+  virtual void Restore() = 0;
+};
+
+}  // namespace ui
+
+#endif  // UI_PLATFORM_WINDOW_PLATFORM_WINDOW_H_