Nukes the 3 arg TabMoved variant in favor of the 4 arg variant....

chrome/browser/tabs/tab_strip_model.h

 // Copyright (c) 2006-2008 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_H_
 #define CHROME_BROWSER_TABS_TAB_STRIP_MODEL_H_
 
 #include <vector>
 
 #include "base/observer_list.h"
@@ skipping 54 matching lines @@
 
   // The selected TabContents changed from |old_contents| to |new_contents| at
   // |index|. |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.
   virtual void TabSelectedAt(TabContents* old_contents,
                              TabContents* new_contents,
                              int index,
                              bool user_gesture) { }
 
-  // The specified TabContents at |from_index| was moved to |to_index|.
-  // TODO(sky): nuke and convert all to 4 arg variant.
-  virtual void TabMoved(TabContents* contents,
-                        int from_index,
-                        int to_index) { }
-
   // The specified TabContents at |from_index| was moved to |to_index|. If
   // the pinned state of the tab is changing |pinned_state_changed| is true.
   virtual void TabMoved(TabContents* contents,
                         int from_index,
                         int to_index,
-                        bool pinned_state_changed) {
-    TabMoved(contents, from_index, to_index);
-  }
+                        bool pinned_state_changed) { }
 
   // 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.
   //
   // If only the loading state was updated, the loading_only flag should be
   // specified. The tab model will update only the throbber, loading status,
   // and crashed state.
   //
   // If other things change, set this flag to false to update all state,
@@ skipping 534 matching lines @@
   typedef ObserverList<TabStripModelObserver> TabStripModelObservers;
   TabStripModelObservers observers_;
 
   // A scoped container for notification registries.
   NotificationRegistrar registrar_;
 
   DISALLOW_COPY_AND_ASSIGN(TabStripModel);
 };
 
 #endif  // CHROME_BROWSER_TABS_TAB_STRIP_MODEL_H_