Move maximize/fullscreen/restore to shell

ui/aura_shell/property_util.h

Index: ui/aura_shell/property_util.h
diff --git a/ui/aura_shell/property_util.h b/ui/aura_shell/property_util.h
new file mode 100644
index 0000000000000000000000000000000000000000..604684b880721cc6d56d07b5021373abfdded228
--- /dev/null
+++ b/ui/aura_shell/property_util.h
@@ -0,0 +1,31 @@
+// 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_