DevTools window doesn't restore its minimal size if it's shrinked to zero.

chrome/browser/ui/gtk/browser_window_gtk.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/gtk/browser_window_gtk.h"
 
 #include <gdk/gdkkeysyms.h>
 
 #include <dlfcn.h>
 #include <string>
@@ skipping 87 matching lines @@
 
 namespace {
 
 // The number of milliseconds between loading animation frames.
 const int kLoadingAnimationFrameTimeMs = 30;
 
 // Default height of dev tools pane when docked to the browser window.  This
 // matches the value in Views.
 const int kDefaultDevToolsHeight = 200;
 
-const int kMinDevToolsHeight = 50;
+const int kMinDevToolsHeight = 100;
 
 const char* kBrowserWindowKey = "__BROWSER_WINDOW_GTK__";
 
 // The frame border is only visible in restored mode and is hardcoded to 4 px
 // on each side regardless of the system window border size.
 const int kFrameBorderThickness = 4;
 // While resize areas on Windows are normally the same size as the window
 // borders, our top area is shrunk by 1 px to make it easier to move the window
 // around with our thinner top grabbable strip.  (Incidentally, our side and
 // bottom resize areas don't match the frame border thickness either -- they
@@ skipping 1173 matching lines @@
   if (devtools_contents) {
     // TabContentsViewGtk::WasShown is not called when tab contents is shown by
     // anything other than user selecting a Tab.
     // See TabContentsViewViews::OnWindowPosChanged for reference on how it
     // should be implemented.
     devtools_contents->tab_contents()->ShowContents();
   }
 
   bool should_show = old_devtools == NULL && devtools_contents != NULL;
   bool should_hide = old_devtools != NULL && devtools_contents == NULL;
+  GtkAllocation devtools_rect;
+  gtk_widget_get_allocation(devtools_container_->widget(), &devtools_rect);
   if (should_show) {
+    if (devtools_rect.height < kMinDevToolsHeight) {
+      gint split_offset = gtk_paned_get_position(GTK_PANED(contents_split_));
+      split_offset -= kMinDevToolsHeight - devtools_rect.height;
+      gtk_paned_set_position(GTK_PANED(contents_split_), split_offset);
+    }
     gtk_widget_show(devtools_container_->widget());
   } else if (should_hide) {
-    // Store split offset when hiding devtools window only.
-    gint divider_offset = gtk_paned_get_position(GTK_PANED(contents_split_));
     browser_->profile()->GetPrefs()->
-        SetInteger(prefs::kDevToolsSplitLocation, divider_offset);
+        SetInteger(prefs::kDevToolsWidgetHeight, devtools_rect.height);
     gtk_widget_hide(devtools_container_->widget());
   }
 }
 
 void BrowserWindowGtk::DestroyBrowser() {
   browser_.reset();
 }
 
 gboolean BrowserWindowGtk::OnConfigure(GtkWidget* widget,
                                        GdkEventConfigure* event) {
@@ skipping 360 matching lines @@
 
   contents_container_.reset(new TabContentsContainerGtk(status_bubble_.get()));
   devtools_container_.reset(new TabContentsContainerGtk(NULL));
   ViewIDUtil::SetID(devtools_container_->widget(), VIEW_ID_DEV_TOOLS_DOCKED);
   contents_split_ = gtk_vpaned_new();
   gtk_paned_pack1(GTK_PANED(contents_split_), contents_container_->widget(),
                   TRUE, TRUE);
   gtk_paned_pack2(GTK_PANED(contents_split_), devtools_container_->widget(),
                   FALSE, TRUE);
   gtk_box_pack_end(GTK_BOX(render_area_vbox_), contents_split_, TRUE, TRUE, 0);
-  // Restore split offset.
+
+  // split_location is obsolete property. It will never be updated.
+  // We just check it for backward compatibility.
   int split_offset = browser_->profile()->GetPrefs()->
-      GetInteger(prefs::kDevToolsSplitLocation);
-  if (split_offset != -1) {
-    if (split_offset < kMinDevToolsHeight)
-      split_offset = kMinDevToolsHeight;
-    gtk_paned_set_position(GTK_PANED(contents_split_), split_offset);
-  } else {
-    gtk_widget_set_size_request(devtools_container_->widget(), -1,
-                                kDefaultDevToolsHeight);
-  }
+    GetInteger(prefs::kDevToolsSplitLocation);

[pfeldman, 2011/06/07 14:54:06]

I'd suggest nuking the property and fixing all ports. It will simplify the code
and add one-time-inconvenience to the end user.

+
+  int devtools_height = browser_->profile()->GetPrefs()->
+      GetInteger(prefs::kDevToolsWidgetHeight);
+
+  if (devtools_height == -1 && split_offset == -1)
+    devtools_height = kDefaultDevToolsHeight;
+
+  if (devtools_height == -1)
+    devtools_height = bounds_.height() - split_offset;
+
+  devtools_height = std::max(devtools_height, kMinDevToolsHeight);
+  devtools_height = std::min(devtools_height, bounds_.height() * 2 / 3);
+  gtk_widget_set_size_request(devtools_container_->widget(), -1,
+                                devtools_height);
+
   gtk_widget_show_all(render_area_floating_container_);
   gtk_widget_hide(devtools_container_->widget());
   render_area_event_box_ = gtk_event_box_new();
   // Set a white background so during startup the user sees white in the
   // content area before we get a TabContents in place.
   gtk_widget_modify_bg(render_area_event_box_, GTK_STATE_NORMAL,
                        &gtk_util::kGdkWhite);
   gtk_container_add(GTK_CONTAINER(render_area_event_box_),
                     render_area_floating_container_);
   gtk_widget_show(render_area_event_box_);
@@ skipping 662 matching lines @@
   // are taken from the WMs' source code.
   return (wm_name == "Blackbox" ||
           wm_name == "compiz" ||
           wm_name == "Compiz" ||
           wm_name == "e16" ||  // Enlightenment DR16
           wm_name == "Metacity" ||
           wm_name == "Mutter" ||
           wm_name == "Openbox" ||
           wm_name == "Xfwm4");
 }