Reduce minimum size of Panels to allow 1-text-line tight autosize.

chrome/browser/ui/panels/panel_manager.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/panels/panel_manager.h"
 
 #include <algorithm>
 
 #include "base/bind.h"
 #include "base/logging.h"
@@ skipping 32 matching lines @@
 
 // See usage below.
 #if defined(TOOLKIT_GTK)
 const int kMillisecondsBeforeCollapsingFromTitleOnlyState = 2000;
 #else
 const int kMillisecondsBeforeCollapsingFromTitleOnlyState = 0;
 #endif
 }  // namespace
 
 // static
+const int PanelManager::kPanelMinWidth;

[jennb, 2011/11/23 04:38:42]

Are these needed?

[Dmitry Titov, 2011/11/23 18:47:49]

Yes! Without these, linker often complains that it doesn't find the variable to
link with. It doesn't always inline, with rules being somewhat random-looking.
Sometimes it works fine (when inlining happens). In this case,
interactive_ui_tests can not be linked with library containing PanelManager w/o
these.

+const int PanelManager::kPanelMinHeight;
+
+// static
 PanelManager* PanelManager::GetInstance() {
   static base::LazyInstance<PanelManager> instance = LAZY_INSTANCE_INITIALIZER;
   return instance.Pointer();
 }
 
 PanelManager::PanelManager()
     : minimized_panel_count_(0),
       are_titlebars_up_(false),
       dragging_panel_index_(kInvalidPanelIndex),
       dragging_panel_original_x_(0),
@@ skipping 565 matching lines @@
 
   // Start from the bottom to avoid reshuffling.
   for (Panels::reverse_iterator iter = panels_copy.rbegin();
        iter != panels_copy.rend(); ++iter)
     (*iter)->Close();
 }
 
 bool PanelManager::is_dragging_panel() const {
   return dragging_panel_index_ != kInvalidPanelIndex;
 }