Move maximize/fullscreen/restore to shell

ui/aura_shell/property_util.h

+// Copyright (c) 2011 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_AURA_SHELL_PROPERTY_UTIL_H_
+#define UI_AURA_SHELL_PROPERTY_UTIL_H_
+#pragma once
+
+namespace aura {
+class Window;
+}
+
+namespace gfx {
+class Rect;
+}
+
+namespace aura_shell {
+
+// Sets the restore bounds property on |window|. It deletes

[sky, 2011/11/01 03:48:25]

'It deletes' -> Deletes

[oshima, 2011/11/01 15:44:53]

Done.

+// existing bounds value if exists.
+void SetRestoreBounds(aura::Window* window, const gfx::Rect&);
+
+// Returns the restore bounds property on |window|.

[sky, 2011/11/01 03:48:25]

Returning a pointer makes ownership a bit dicey. Could this return a gfx::Rect
by value instead?

[oshima, 2011/11/01 15:44:53]

It's pointer because the caller needs to know if the property exists.
It's possible to use gfx::Rect(), but I don't think that's good idea either.

I added comments to make it more clear. Let me know what you think.

+const gfx::Rect* GetRestoreBounds(aura::Window* window);
+
+// Deletes and clears the restore bounds roperty on |window|.
+void ClearRestoreBounds(aura::Window* window);
+
+}
+
+#endif  // UI_AURA_SHELL_PROPERTY_UTIL_H_