Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5002)

Unified Diff: chrome/common/extensions/api/app_window.idl

Issue 11362003: Make app windows restore position even if bounds are explicitly specified. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: re-rebase Created 8 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/ui/extensions/shell_window.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/common/extensions/api/app_window.idl
diff --git a/chrome/common/extensions/api/app_window.idl b/chrome/common/extensions/api/app_window.idl
index 31e8727d575fded71a7e31de9046daf815f58621..b83984ba9cb7692d1cf630f599dba90083b973fd 100644
--- a/chrome/common/extensions/api/app_window.idl
+++ b/chrome/common/extensions/api/app_window.idl
@@ -9,17 +9,21 @@ namespace app.window {
// with the same id (and no explicit size or position) is later opened.
DOMString? id;
- // Default width of the window.
- long? defaultWidth;
+ // Default width of the window. (Deprecated; regular bounds act like this
+ // now.)
+ [nodoc] long? defaultWidth;
- // Default height of the window.
- long? defaultHeight;
+ // Default height of the window. (Deprecated; regular bounds act like this
+ // now.)
+ [nodoc] long? defaultHeight;
- // Default X coordinate of the window.
- long? defaultLeft;
+ // Default X coordinate of the window. (Deprecated; regular bounds act like
+ // this now.)
+ [nodoc] long? defaultLeft;
- // Default Y coordinate of the window.
- long? defaultTop;
+ // Default Y coordinate of the window. (Deprecated; regular bounds act like
+ // this now.)
+ [nodoc] long? defaultTop;
// Width of the window. (Deprecated; use 'bounds'.)
[nodoc] long? width;
@@ -122,10 +126,11 @@ namespace app.window {
//
// Yet another option is to give the window a (unique) id. This id is then
// used to remember the size and position of the window whenever it is
- // moved or resized. This size and position is then used on subsequent
- // opening of a window with the same id. The defaultLeft/defaultTop and
- // defaultWidth/defaultHeight properties can be used to specify a position
- // and size when no geometry has been stored for the window yet.
+ // moved or resized. This size and position is then used instead of the
+ // specified bounds on subsequent opening of a window with the same id. If
+ // you need to open a window with an id at a location other than the
+ // remembered default, you can create it hidden, move it to the desired
+ // location, then show it.
//
// You can also combine these various options, explicitly specifying for
// example the size while having the position be remembered or other
« no previous file with comments | « chrome/browser/ui/extensions/shell_window.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698