Multi-tab selection: Renaming TabStripModelObserver::TabSelectedAt to ActiveTabChanged

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 54 matching lines @@
   // by |old_contents| and the newly selected tab by |new_contents|. |index| is
   // the index of |new_contents|. When using multiple selection this may be sent
   // even when the active tab has not changed. For example, if the selection is
   // extended this method is invoked to inform observers the selection has
   // changed, but |old_contents| and |new_contents| are the same.  If you only
   // care about when the active tab changes, check for when |old_contents|
   // differs from |new_contents|. |user_gesture| specifies whether or not this
   // was done by a user input event (e.g. clicking on a tab, keystroke) or as a
   // side-effect of some other function.
   //
-  // TODO(sky): consider not overloading this. Instead rename this to
-  // TabActivatedAt (or something) and have TabSelectionChanged as well.
-  // TabSelectedAt. This requires renaming everyone to use new terms instead of
-  // selection.
-  virtual void TabSelectedAt(TabContentsWrapper* old_contents,
-                             TabContentsWrapper* new_contents,
-                             int index,
-                             bool user_gesture);
+  // TODO(dpapad): Add TabSelectionChanged method for when the selected tabs
+  // change.
+  virtual void ActiveTabChanged(TabContentsWrapper* old_contents,
+                                TabContentsWrapper* new_contents,
+                                int index,
+                                bool user_gesture);
 
   // The specified TabContents at |from_index| was moved to |to_index|.
   virtual void TabMoved(TabContentsWrapper* contents,
                         int from_index,
                         int to_index);
 
   // The specified TabContents at |index| changed in some way. |contents| may
   // be an entirely different object and the old value is no longer available
   // by the time this message is delivered.
   //
@@ skipping 36 matching lines @@
 
   // Invoked when an active/selected tab at |index| is selected again (ie - the
   // active/foreground tab is clicked).
   virtual void ActiveTabClicked(int index);
 
  protected:
   virtual ~TabStripModelObserver() {}
 };
 
 #endif  // CHROME_BROWSER_TABS_TAB_STRIP_MODEL_OBSERVER_H_