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

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

 // Copyright (c) 2012 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.
 
 #include "chrome/browser/extensions/api/app_window/app_window_api.h"
 
 #include "base/command_line.h"
 #include "base/time.h"
 #include "base/values.h"
 #include "chrome/browser/debugger/devtools_window.h"
@@ skipping 94 matching lines @@
       // TODO(mek): use URL if no id specified?
       // Limit length of id to 256 characters.
       if (options->id->length() > 256) {
         error_ = app_window_constants::kInvalidWindowId;
         return false;
       }
 
       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())
       create_params.bounds.set_height(*options->default_height.get());
     if (options->default_left.get())
       create_params.bounds.set_x(*options->default_left.get());
     if (options->default_top.get())
       create_params.bounds.set_y(*options->default_top.get());
 
-
-    if (options->width.get() || options->height.get()) {
-      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()) {
-      if (options->left.get())
-        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->width.get())
+      create_params.bounds.set_width(*options->width.get());
+    if (options->height.get())
+      create_params.bounds.set_height(*options->height.get());
+    if (options->left.get())
+      create_params.bounds.set_x(*options->left.get());
+    if (options->top.get())
+      create_params.bounds.set_y(*options->top.get());
 
     if (options->bounds.get()) {
       app_window::Bounds* bounds = options->bounds.get();
       if (bounds->width.get())
         create_params.bounds.set_width(*bounds->width.get());
       if (bounds->height.get())
         create_params.bounds.set_height(*bounds->height.get());
       if (bounds->left.get())
         create_params.bounds.set_x(*bounds->left.get());
       if (bounds->top.get())
@@ skipping 70 matching lines @@
   if (ShellWindowRegistry::Get(profile())->HadDevToolsAttached(created_view)) {
     new DevToolsRestorer(this, created_view);
     return true;
   }
 
   SendResponse(true);
   return true;
 }
 
 }  // namespace extensions