Save window placement for App windows.

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

Index: chrome/browser/ui/views/frame/browser_view.cc
===================================================================
--- chrome/browser/ui/views/frame/browser_view.cc	(revision 99708)
+++ chrome/browser/ui/views/frame/browser_view.cc	(working copy)
@@ -1636,7 +1636,8 @@
   // 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);
   }
@@ -1649,7 +1650,7 @@
   *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).