Add SwappedOut to TabContentsDelegate to allow us to correctly delete TabContents.

content/browser/tab_contents/tab_contents.cc

Index: content/browser/tab_contents/tab_contents.cc
diff --git a/content/browser/tab_contents/tab_contents.cc b/content/browser/tab_contents/tab_contents.cc
index 03c0fdd9c18c14403df5d371351297b9bfeca2c1..b25507a9910709cd1bf9488805d11aa98ac0cca6 100644
--- a/content/browser/tab_contents/tab_contents.cc
+++ b/content/browser/tab_contents/tab_contents.cc
@@ -1645,6 +1645,11 @@ void TabContents::Close(RenderViewHost* rvh) {
     delegate()->CloseContents(this);
 }
 
+void TabContents::SwappedOut(RenderViewHost* rvh) {
+  if (delegate() && rvh == render_view_host())

[Charlie Reis, 2011/11/11 22:23:52]

You're seeing the current RenderViewHost send a swapped out message?  That
sounds like a race bug that we should be handling differently.  The current RVH
should never be swapped out.

+    delegate()->SwappedOut(this);
+}
+
 void TabContents::RequestMove(const gfx::Rect& new_bounds) {
   if (delegate() && delegate()->IsPopupOrPanel(this))
     delegate()->MoveContents(this, new_bounds);