Elides the beginning of tab titles that have common prefixes. ...

chrome/browser/ui/views/tabs/base_tab_strip.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_VIEWS_TABS_BASE_TAB_STRIP_H_
 #define CHROME_BROWSER_UI_VIEWS_TABS_BASE_TAB_STRIP_H_
 #pragma once
 
 #include <vector>
 
@@ skipping 176 matching lines @@
   // code and is not a general-purpose method.
   void RemoveAndDeleteTab(BaseTab* tab);
 
   // Resets the bounds of all non-closing tabs.
   virtual void GenerateIdealBounds() = 0;
 
   void set_ideal_bounds(int index, const gfx::Rect& bounds) {
     tab_data_[index].ideal_bounds = bounds;
   }
 
+  // Update the lengths of common title prefixes for all tabs. This needs
+  // to be done every time tabs are added/removed or when titles change.
+  virtual void UpdateCommonTitlePrefix();

[sky, 2011/02/25 17:02:07]

Why is this virtual?

Also, this class is only used on windows and chromeos. We're going to want this
feature on other platforms too (sadly). This function should be extracted into a
place that can be used on other platforms. I'm thinking you create some class
like PrefixMatcher or PrefixFinder or TabTitlePrefixBlahBlahBlah (you get the
idea) in chrome/browser/ui. The class has a single static method,
CalculatePrefixes and it takes a std::vector<string16> and calculates the
std::vector<size_t> of prefixes. This also has the benefit that you can write a
nice unit test for it:)

[sky, 2011/02/25 17:03:03]

One additional comment. Don't take my comment to mean you have to figure out the
other platforms now. Just that you should prepare for them:)

[MAD, 2011/02/25 20:09:13]

Good point... I looked for a test infrastructure for tabs and couldn't find
one... I'll follow your advice here... Thanks!

+
   // Returns the index into |tab_data_| corresponding to the specified tab, or
   // -1 if the tab isn't in |tab_data_|.
   int TabIndexOfTab(BaseTab* tab) const;
 
   // Stops any ongoing animations. If |layout| is true and an animation is
   // ongoing this does a layout.
   virtual void StopAnimating(bool layout);
 
   // Destroys the active drag controller.
   void DestroyDragController();
@@ skipping 39 matching lines @@
   // model.
   bool attaching_dragged_tab_;
 
   views::BoundsAnimator bounds_animator_;
 
   // Size we last layed out at.
   gfx::Size last_layout_size_;
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_TABS_BASE_TAB_STRIP_H_