First part of remembering platform app window geometry.

chrome/browser/ui/extensions/shell_window.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_EXTENSIONS_SHELL_WINDOW_H_
 #define CHROME_BROWSER_UI_EXTENSIONS_SHELL_WINDOW_H_
 
 #include "base/memory/scoped_ptr.h"
 #include "chrome/browser/extensions/extension_function_dispatcher.h"
 #include "chrome/browser/sessions/session_id.h"
@@ skipping 34 matching lines @@
       FRAME_CHROME, // Chrome-style window frame.
       FRAME_NONE, // Frameless window.
     };
 
     CreateParams();
 
     Frame frame;
     // Specify the initial bounds of the window. If empty, the window will be a
     // default size.
     gfx::Rect bounds;
+    // Specify if bounds should be restored from a previous time

[asargent_no_longer_on_chrome, 2012/08/28 18:49:41]

nit: period at end of sentence

[Marijn Kruisselbrink, 2012/08/28 22:14:58]

Done.

+    bool restorePosition;
+    bool restoreSize;
+
     gfx::Size minimum_size;
     gfx::Size maximum_size;
+
+    std::string window_key;
   };
 
   static ShellWindow* Create(Profile* profile,
                              const extensions::Extension* extension,
                              const GURL& url,
                              const CreateParams& params);
 
   const SessionID& session_id() const { return session_id_; }
   const extensions::Extension* extension() const { return extension_; }
   const TabContents* tab_contents() const { return contents_.get(); }
   content::WebContents* web_contents() const { return web_contents_; }
   Profile* profile() const { return profile_; }
 
   BaseWindow* GetBaseWindow();
   gfx::NativeWindow GetNativeWindow() {
     return GetBaseWindow()->GetNativeWindow();
   }
 
   // NativeShellWindows should call this to determine what the window's title
   // is on startup and from within UpdateWindowTitle().
   virtual string16 GetTitle() const;
 
   // Call to notify ShellRegistry and delete the window. Subclasses should
   // invoke this method instead of using "delete this".
   void OnNativeClose();
 
+  // Save the window position in the prefs.
+  virtual void SaveWindowPosition();
  protected:
   ShellWindow(Profile* profile,
               const extensions::Extension* extension);
   virtual ~ShellWindow();
 
  private:
   // PlatformAppBrowserTest needs access to web_contents()
   friend class extensions::PlatformAppBrowserTest;
 
   // Instantiates a platform-specific ShellWindow subclass (one implementation
@@ skipping 47 matching lines @@
   // Message handlers.
   void OnRequest(const ExtensionHostMsg_Request_Params& params);
 
   // Helper method to add a message to the renderer's DevTools console.
   void AddMessageToDevToolsConsole(content::ConsoleMessageLevel level,
                                    const std::string& message);
 
   virtual void UpdateDraggableRegions(
     const std::vector<extensions::DraggableRegion>& regions);
 
+  // Read window geometry from storage

[asargent_no_longer_on_chrome, 2012/08/28 18:49:41]

nit: period at end of sentence

[Marijn Kruisselbrink, 2012/08/28 22:14:58]

oops, that entire method here is just a leftover from an earlier attempt

+  void GeometryFromStorage(bool restorePosition,
+                           bool restoreSize,
+                           scoped_ptr<base::Value> value);
+
   Profile* profile_;  // weak pointer - owned by ProfileManager.
   // weak pointer - owned by ExtensionService.
   const extensions::Extension* extension_;
 
+  // used for saving/restoring window geometry

[asargent_no_longer_on_chrome, 2012/08/28 18:49:41]

nit: make full sentence, with period at end

[Marijn Kruisselbrink, 2012/08/28 22:14:58]

Done.

+  std::string window_key_;
+
   const SessionID session_id_;
   scoped_ptr<TabContents> contents_;
   // web_contents_ is owned by contents_.
   content::WebContents* web_contents_;
   content::NotificationRegistrar registrar_;
   ExtensionFunctionDispatcher extension_function_dispatcher_;
 
   scoped_ptr<NativeShellWindow> native_window_;
 
   DISALLOW_COPY_AND_ASSIGN(ShellWindow);
 };
 
 #endif  // CHROME_BROWSER_UI_EXTENSIONS_SHELL_WINDOW_H_