DevTools: respect minimum width/height when resizing.

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

 // Copyright 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.
 
 #include "chrome/browser/ui/views/frame/browser_view.h"
 
 #include <algorithm>
 
 #include "base/auto_reset.h"
 #include "base/command_line.h"
@@ skipping 2211 matching lines @@
 }
 
 void BrowserView::ShowDevToolsContainer() {
   if (!devtools_focus_tracker_.get()) {
     // Install devtools focus tracker when dev tools window is shown for the
     // first time.
     devtools_focus_tracker_.reset(
         new views::ExternalFocusTracker(devtools_container_,
                                         GetFocusManager()));
   }
+
+  gfx::Size minDevtoolsSize(devtools_window_->GetMinimumWidth(),

[pfeldman, 2013/03/19 10:42:06]

min_devtools_size

[dgozman, 2013/03/21 09:55:26]

Done.

+      devtools_window_->GetMinimumHeight());
+  devtools_container_->SetPreferredSize(minDevtoolsSize);

[sky, 2013/03/19 14:40:50]

This changes the preferred size, are you sure you want that?

[dgozman, 2013/03/21 09:55:26]

I did not found another way to set minimum size for WebView, and subclassing
looks unnecessary.

+
   devtools_container_->SetVisible(true);
   devtools_dock_side_ = devtools_window_->dock_side();
   bool dock_to_right = devtools_dock_side_ == DEVTOOLS_DOCK_SIDE_RIGHT;
   contents_split_->set_orientation(
       dock_to_right ? views::SingleSplitView::HORIZONTAL_SPLIT
                     : views::SingleSplitView::VERTICAL_SPLIT);
   UpdateDevToolsSplitPosition();
   contents_split_->InvalidateLayout();
   Layout();
 }
@@ skipping 468 matching lines @@
 
   Browser* modal_browser =
       chrome::FindBrowserWithWebContents(active_dialog->web_contents());
   if (modal_browser && (browser_ != modal_browser)) {
     modal_browser->window()->FlashFrame(true);
     modal_browser->window()->Activate();
   }
 
   AppModalDialogQueue::GetInstance()->ActivateModalDialog();
 }