Ellipsize long entries in bookmark menus and the back/forward menus....

chrome/browser/gtk/back_forward_menu_model_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/back_forward_menu_model_gtk.h"
 
+#include "app/l10n_util.h"
 #include "base/string_util.h"
 #include "chrome/browser/gtk/back_forward_button_gtk.h"
 
+// Might want to vary this by locale.
+static const size_t kMaxChars = 50;
+
 BackForwardMenuModelGtk::BackForwardMenuModelGtk(Browser* browser,
                                                  ModelType model_type,
                                                  BackForwardButtonGtk* button)
     : BackForwardMenuModel(browser, model_type),
       button_(button) {
 }
 
 int BackForwardMenuModelGtk::GetItemCount() const {
   return GetTotalItemCount();
 }
 
 bool BackForwardMenuModelGtk::IsItemSeparator(int command_id) const {
   return IsSeparator(command_id);
 }
 
 std::string BackForwardMenuModelGtk::GetLabel(int command_id) const {
-  return UTF16ToUTF8(GetItemLabel(command_id));
+  std::wstring item_label_wstr = UTF16ToWideHack(GetItemLabel(command_id));
+  // This breaks on word boundaries. Ideally we would break on character
+  // boundaries.
+  item_label_wstr = l10n_util::TruncateString(item_label_wstr, kMaxChars);

[tony, 2009/08/27 23:01:04]

It looks like there's an width-chars property on GtkMenuItem.  Can we use that
instead?

[Evan Stade, 2009/08/27 23:02:27]

The "width-chars" style property

  "width-chars"              gint                  : Read
The minimum desired width of the menu item in characters.

+  return WideToUTF8(item_label_wstr);
 }
 
 bool BackForwardMenuModelGtk::HasIcon(int command_id) const {
   return ItemHasIcon(command_id);
 }
 
 const SkBitmap* BackForwardMenuModelGtk::GetIcon(int command_id) const {
   return &GetItemIcon(command_id);
 }
 
 bool BackForwardMenuModelGtk::IsCommandEnabled(int command_id) const {
   return ItemHasCommand(command_id);
 }
 
 void BackForwardMenuModelGtk::ExecuteCommand(int command_id) {
   ExecuteCommandById(command_id);
 }
 
 void BackForwardMenuModelGtk::StoppedShowing() {
   if (button_)
     button_->StoppedShowingMenu();
 }