Compact Navigation prototype

chrome/browser/tabs/tab_strip_model_observer.h

Index: chrome/browser/tabs/tab_strip_model_observer.h
diff --git a/chrome/browser/tabs/tab_strip_model_observer.h b/chrome/browser/tabs/tab_strip_model_observer.h
index 5757d88d185608617887ab52058f675b35f13b2b..0aa1b15900200733ea6c125b4a4790616529f698 100644
--- a/chrome/browser/tabs/tab_strip_model_observer.h
+++ b/chrome/browser/tabs/tab_strip_model_observer.h
@@ -126,6 +126,10 @@ class TabStripModelObserver {
   // must be dropped.
   virtual void TabStripModelDeleted();
 
+  // Invoked when an active/selected tab at |index| is selected again (ie - the
+  // active/foreground tab is clicked).
+  virtual void TabReselected(int index);

[sky, 2011/05/03 18:38:32]

The implementation that triggers calling this only triggers the call if the tab
is already active and clicked. Also, reselected makes it sound like the
selection is changing, which isn't the case. How about ActiveTabClicked.

[SteveT, 2011/05/06 18:48:43]

"ActiveTabClicked" was actually one of the names I considered when first writing
this. :) And I guess 'Reselected' might be a bit confusing now that the tab
selection feature has landed. Will rename as per your suggestion.

I've renamed the corresponding methods from "ReselectTab" to "ClickActiveTab"
(ex - in base_tab.cc). Not sure if I'm a big fan of the wording, so let me know
if you have a suggestion.

+
  protected:
   virtual ~TabStripModelObserver() {}
 };