Fix a toolbar width TODO.

chrome/browser/gtk/browser_toolbar_view_gtk.cc

 // Copyright (c) 2009 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/gtk/browser_toolbar_view_gtk.h"
 
 #include "base/logging.h"
 #include "base/base_paths_linux.h"
 #include "base/path_service.h"
 #include "chrome/app/chrome_dll_resource.h"
@@ skipping 36 matching lines @@
 }
 
 BrowserToolbarGtk::~BrowserToolbarGtk() {
 }
 
 void BrowserToolbarGtk::Init(Profile* profile) {
   show_home_button_.Init(prefs::kShowHomeButton, profile->GetPrefs(), this);
 
   toolbar_ = gtk_hbox_new(FALSE, 0);
   gtk_container_set_border_width(GTK_CONTAINER(toolbar_), 4);
-  // TODO(evanm): this setting of the x-size to 0 makes it so the window
-  // can be resized arbitrarily small.  We should figure out what we want
-  // with respect to resizing before engineering around it, though.
-  gtk_widget_set_size_request(toolbar_, 0, kToolbarHeight);
+  gtk_widget_set_size_request(toolbar_, -1, kToolbarHeight);

[tony, 2009/03/05 01:11:13]

Nit: Maybe add a comment explaining -1?

 
   toolbar_tooltips_ = gtk_tooltips_new();
 
   back_.reset(BuildBackForwardButton(IDR_BACK, IDR_BACK_P, IDR_BACK_H,
               IDR_BACK_D,
               l10n_util::GetString(IDS_TOOLTIP_BACK)));
   forward_.reset(BuildBackForwardButton(IDR_FORWARD, IDR_FORWARD_P,
                  IDR_FORWARD_H, IDR_FORWARD_D,
                  l10n_util::GetString(IDS_TOOLTIP_FORWARD)));
 
@@ skipping 318 matching lines @@
     app_menu_.reset(new MenuGtk(this, GetStandardAppMenu()));
   }
 
   app_menu_->Popup(app_menu_button_->widget(), button_press_event);
 }
 
 CustomDrawButton* BrowserToolbarGtk::MakeHomeButton() {
   return BuildToolbarButton(IDR_HOME, IDR_HOME_P, IDR_HOME_H, 0,
                             l10n_util::GetString(IDS_TOOLTIP_HOME));
 }