Add a return value to InternalCloseTabContentsAt, indicating whether or not the TabContents was closed immediately or if we're waiting for an unload handler to fire.

Adjust indices in Close other tabs handler to take into account the fact that some tabs may not close immediately

B=1295790

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 8 insertions, 3 deletions

diff --git a/chrome/browser/tabs/tab_strip_model.h b/chrome/browser/tabs/tab_strip_model.h
index aaa3edb..8a235c5 100644
--- a/chrome/browser/tabs/tab_strip_model.h
+++ b/chrome/browser/tabs/tab_strip_model.h
@@ -248,8 +248,10 @@ class TabStripModel : public NotificationObserver {
   // Closes the TabContents at the specified index. This causes the TabContents
   // to be destroyed, but it may not happen immediately (e.g. if it's a
   // WebContents).
-  void CloseTabContentsAt(int index) {
-    InternalCloseTabContentsAt(index, true);
+  // Returns true if the TabContents was closed immediately, false if we are
+  // waiting for a response from an onunload handler.
+  bool CloseTabContentsAt(int index) {
+    return InternalCloseTabContentsAt(index, true);
   }
 
   // Replaces the entire state of a the tab at index by switching in a
@@ -426,7 +428,10 @@ class TabStripModel : public NotificationObserver {
   // The boolean parameter create_historical_tab controls whether to
   // record this tab and its history for reopening recently closed
   // tabs.
-  void InternalCloseTabContentsAt(int index, bool create_historical_tab);
+  //
+  // Returns true if the TabContents was closed immediately, false if we are
+  // waiting for the result of an onunload handler.
+  bool InternalCloseTabContentsAt(int index, bool create_historical_tab);
 
   TabContents* GetContentsAt(int index) const;