Make app windows restore position even if bounds are explicitly specified.

chrome/browser/extensions/api/app_window/app_window_api.cc

Index: chrome/browser/extensions/api/app_window/app_window_api.cc
diff --git a/chrome/browser/extensions/api/app_window/app_window_api.cc b/chrome/browser/extensions/api/app_window/app_window_api.cc
index 2bed9f500d9b5babe969c3658015476cf1ceae03..2d41e66c0dbb58a812b8a9b3ecfe4325a8221744 100644
--- a/chrome/browser/extensions/api/app_window/app_window_api.cc
+++ b/chrome/browser/extensions/api/app_window/app_window_api.cc
@@ -112,6 +112,8 @@ bool AppWindowCreateFunction::RunImpl() {
       create_params.window_key = *options->id;
     }
 
+    // TODO(jeremya): remove these, since they do the same thing as
+    // left/top/width/height.
     if (options->default_width.get())
       create_params.bounds.set_width(*options->default_width.get());
     if (options->default_height.get())
@@ -121,13 +123,11 @@ bool AppWindowCreateFunction::RunImpl() {
     if (options->default_top.get())
       create_params.bounds.set_y(*options->default_top.get());
 
-
     if (options->width.get() || options->height.get()) {

[Marijn Kruisselbrink, 2012/11/12 17:56:51]

nit: With the restore_size bit gone, there is no reason to keep this if wrapping
the width/height getting.

       if (options->width.get())
         create_params.bounds.set_width(*options->width.get());
       if (options->height.get())
         create_params.bounds.set_height(*options->height.get());
-      create_params.restore_size = false;
     }
 
     if (options->left.get() || options->top.get()) {

[Marijn Kruisselbrink, 2012/11/12 17:56:51]

same here, this outer if no longer has a purpose

@@ -135,7 +135,6 @@ bool AppWindowCreateFunction::RunImpl() {
         create_params.bounds.set_x(*options->left.get());
       if (options->top.get())
         create_params.bounds.set_y(*options->top.get());
-      create_params.restore_position = false;
     }
 
     if (options->frame.get()) {