DevTools: “Dock to right” broken after turning a tab into a window of its own.

chrome/browser/debugger/devtools_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_DEBUGGER_DEVTOOLS_WINDOW_H_
 #define CHROME_BROWSER_DEBUGGER_DEVTOOLS_WINDOW_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 69 matching lines @@
   // Overridden from DevToolsClientHost.
   virtual void InspectedContentsClosing() OVERRIDE;
   virtual void ContentsReplaced(content::WebContents* new_contents) OVERRIDE;
   content::RenderViewHost* GetRenderViewHost();
 
   void Show(DevToolsToggleAction action);
 
   TabContents* tab_contents() { return tab_contents_; }
   Browser* browser() { return browser_; }  // For tests.
   DevToolsDockSide dock_side() { return dock_side_; }
-  content::DevToolsClientHost* devtools_client_host() {
-    return frontend_host_;
-  }
+  content::DevToolsClientHost* devtools_client_host() { return frontend_host_; }
+
+  // Returns preferred devtools window width for given |container_width|.

[Peter Kasting, 2012/10/26 03:22:07]

Nit: Add more detail, like "This tries to use the saved window width, or, if
none exists, 1/3 of the container width, then clamps to try and ensure both
devtools and content are at least somewhat visible."

[pfeldman, 2012/10/26 08:56:02]

Done.

+  // Called only for the case when devtools window is docked to the side.
+  int GetWidth(int container_width);
+
+  // Returns preferred devtools window height for given |container_height|.
+  // Called only for the case when devtools window is docked to bottom.
+  int GetHeight(int container_height);
+
+  // Stores preferred devtools window width for this instance.
+  void SetWidth(int width);
+
+  // Stores preferred devtools window height for this instance.
+  void SetHeight(int height);
 
  private:
   static DevToolsWindow* Create(Profile* profile,
                                 content::RenderViewHost* inspected_rvh,
                                 DevToolsDockSide dock_side,
                                 bool shared_worker_frontend);
   DevToolsWindow(TabContents* tab_contents,
                  Profile* profile,
                  content::RenderViewHost* inspected_rvh,
                  DevToolsDockSide dock_side);
@@ skipping 73 matching lines @@
   Profile* profile_;
   TabContents* inspected_tab_;
   TabContents* tab_contents_;
   Browser* browser_;
   DevToolsDockSide dock_side_;
   bool is_loaded_;
   DevToolsToggleAction action_on_load_;
   content::NotificationRegistrar registrar_;
   content::DevToolsClientHost* frontend_host_;
   scoped_ptr<DevToolsFileHelper> file_helper_;
+  int width_;
+  int height_;
   DISALLOW_COPY_AND_ASSIGN(DevToolsWindow);
 };
 
 #endif  // CHROME_BROWSER_DEBUGGER_DEVTOOLS_WINDOW_H_