Save window placement for App windows.

chrome/browser/ui/views/frame/browser_view.cc

 // 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.
 
 #include "chrome/browser/ui/views/frame/browser_view.h"
 
 #if defined(TOOLKIT_USES_GTK)
 #include <gtk/gtk.h>
 #endif
 
@@ skipping 1618 matching lines @@
 
 std::wstring BrowserView::GetWindowName() const {
   return UTF8ToWide(browser_->GetWindowPlacementKey());
 }
 
 void BrowserView::SaveWindowPlacement(const gfx::Rect& bounds,
                                       ui::WindowShowState show_state) {
   // If IsFullscreen() is true, we've just changed into fullscreen mode, and
   // we're catching the going-into-fullscreen sizing and positioning calls,
   // which we want to ignore.
-  if (!IsFullscreen() && browser_->ShouldSaveWindowPlacement()) {
+  if (!IsFullscreen() &&
+      (browser_->ShouldSaveWindowPlacement() || browser_->is_app())) {

[Ben Goodger (Google), 2011/11/07 20:47:21]

BTW, upon further consideration I realize this is not the right place to have
this check. It's not really view-state, and as such should not live in
BrowserView, rather it should live in Browser::ShouldSaveWindowPlacement. Can
you fix this? Thanks :-)

     WidgetDelegate::SaveWindowPlacement(bounds, show_state);
     browser_->SaveWindowPlacement(bounds, show_state);
   }
 }
 
 bool BrowserView::GetSavedWindowPlacement(
     gfx::Rect* bounds,
     ui::WindowShowState* show_state) const {
   *bounds = browser_->GetSavedWindowBounds();
   *show_state = browser_->GetSavedWindowShowState();
 
   if ((browser_->is_type_popup() || browser_->is_type_panel())
-      && !browser_->is_devtools()) {
+      && !browser_->is_devtools() && !browser_->is_app()) {
     // We are a popup window. The value passed in |bounds| represents two
     // pieces of information:
     // - the position of the window, in screen coordinates (outer position).
     // - the size of the content area (inner size).
     // We need to use these values to determine the appropriate size and
     // position of the resulting window.
     if (IsToolbarVisible()) {
       // If we're showing the toolbar, we need to adjust |*bounds| to include
       // its desired height, since the toolbar is considered part of the
       // window's client area as far as GetWindowBoundsForClientBounds is
@@ skipping 1011 matching lines @@
 BrowserWindow* BrowserWindow::CreateBrowserWindow(Browser* browser) {
   // Create the view and the frame. The frame will attach itself via the view
   // so we don't need to do anything with the pointer.
   BrowserView* view = new BrowserView(browser);
   (new BrowserFrame(view))->InitBrowserFrame();
   view->GetWidget()->non_client_view()->SetAccessibleName(
       l10n_util::GetStringUTF16(IDS_PRODUCT_NAME));
   return view;
 }
 #endif