Multi-tab: Adding new Notification when tab selection changes.

chrome/browser/tabs/tab_strip_model.h

Index: chrome/browser/tabs/tab_strip_model.h
diff --git a/chrome/browser/tabs/tab_strip_model.h b/chrome/browser/tabs/tab_strip_model.h
index e711b493a90d0bd02f7b1d5a2ee5dd4d66f52313..4cf6173e26f4c2bf0ffd7c435675b921dba5cd89 100644
--- a/chrome/browser/tabs/tab_strip_model.h
+++ b/chrome/browser/tabs/tab_strip_model.h
@@ -516,14 +516,15 @@ class TabStripModel : public NotificationObserver {
   TabContentsWrapper* GetContentsAt(int index) const;
 
   // If the TabContentsWrapper at |to_index| differs from |old_contents|
-  // notifies observers.
-  void NotifyTabSelectedIfChanged(TabContentsWrapper* old_contents,
-                                  int to_index,
-                                  bool user_gesture);
-
-  // Notifies the observers the active tab changed. |old_active_index| gives
-  // the old active index.
-  void NotifyActiveTabChanged(int old_active_index);
+  // notifies observers. If |old_contents| still exist a TabDeselected

[sky, 2011/06/07 20:32:44]

'still exist' -> 'non-null'
TabDeselected -> TabDeactivated

[dpapad, 2011/06/07 23:20:09]

Done.

+  // notification is sent right before sending ActiveTabChanged notification.
+  void NotifyIfActiveTabChanged(TabContentsWrapper* old_contents,
+                                int to_index,
+                                bool user_gesture);
+
+  // Notifies the observers only if the tab selection has changed. |model| is a
+  // snapshot of |selection_model_| before the change.
+  void NotifyIfSelectionChanged(const TabStripSelectionModel& model);
 
   // Returns the number of New Tab tabs in the TabStripModel.
   int GetNewTabCount() const;