Remove undisplayed tabs from the views hierarchy because event delivery doesn't respect z-order.

chrome/browser/ui/views/tab_contents/tab_contents_container.cc

Index: chrome/browser/ui/views/tab_contents/tab_contents_container.cc
diff --git a/chrome/browser/ui/views/tab_contents/tab_contents_container.cc b/chrome/browser/ui/views/tab_contents/tab_contents_container.cc
index eb5b1adce10e781e88775744be3dd49dd5876f3c..1830fb223b8d7a8f2277d9dbe9b9311e868926c1 100644
--- a/chrome/browser/ui/views/tab_contents/tab_contents_container.cc
+++ b/chrome/browser/ui/views/tab_contents/tab_contents_container.cc
@@ -38,6 +38,9 @@ void TabContentsContainer::ChangeTabContents(TabContents* contents) {
   if (tab_contents_) {
 #if !defined(TOUCH_UI)
     native_container_->DetachContents(tab_contents_);
+#else
+    views::View *v = static_cast<TabContentsViewViews*>(tab_contents_->view());

[sky, 2010/12/10 18:20:07]

nit: View* v.

+    RemoveChildView(v);

[sky, 2010/12/10 16:37:21]

All these ifdefs are making this class unreadable and hard to follow. Remove all
the methods in tab_contents_container.cc that are ifdefd, putting the
implementations in tab_contents_container_touch.cc and
tab_contents_container_views.cc (yes, name sucks).

 #endif
     tab_contents_->WasHidden();
     RemoveObservers();