Compact Navigation prototype

chrome/browser/tabs/tab_strip_model_observer.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_TABS_TAB_STRIP_MODEL_OBSERVER_H_
 #define CHROME_BROWSER_TABS_TAB_STRIP_MODEL_OBSERVER_H_
 #pragma once
 
 class TabContentsWrapper;
 class TabStripModel;
@@ skipping 108 matching lines @@
 
   // The TabStripModel now no longer has any tabs. The implementer may
   // use this as a trigger to try and close the window containing the
   // TabStripModel, for example...
   virtual void TabStripEmpty();
 
   // Sent when the tabstrip model is about to be deleted and any reference held
   // 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() {}
 };
 
 #endif  // CHROME_BROWSER_TABS_TAB_STRIP_MODEL_OBSERVER_H_