Fix a crash in chrome.exe caused by the external tab container instance getting destroyed without

going through the proper window shutdown sequence when the external host crashes.

The fix is to ensure that we clean up correctly when the external tab instance goes away. Will add a unit test in a subsequent CB.

Bug=1688967

Review URL: http://codereview.chromium.org/39086

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

1 files changed, 13 insertions, 6 deletions

diff --git a/chrome/browser/external_tab_container.cc b/chrome/browser/external_tab_container.cc
index 216d171..e637a16 100644
--- a/chrome/browser/external_tab_container.cc
+++ b/chrome/browser/external_tab_container.cc
@@ -35,6 +35,7 @@ ExternalTabContainer::ExternalTabContainer(
 }
 
 ExternalTabContainer::~ExternalTabContainer() {
+  Uninitialize(m_hWnd);
 }
 
 bool ExternalTabContainer::Init(Profile* profile, HWND parent,
@@ -120,16 +121,18 @@ bool ExternalTabContainer::Init(Profile* profile, HWND parent,
     SetParent(parent);
 
   ::ShowWindow(tab_contents_->GetNativeView(), SW_SHOWNA);
-
   return true;
 }
 
-void ExternalTabContainer::OnDestroy() {
-  views::FocusManager* focus_manager =
-      views::FocusManager::GetFocusManager(GetHWND());
-  if (focus_manager) {
-    focus_manager->RemoveKeystrokeListener(this);
+bool ExternalTabContainer::Uninitialize(HWND window) {
+  if (::IsWindow(window)) {
+    views::FocusManager* focus_manager =
+        views::FocusManager::GetFocusManager(window);
+    if (focus_manager) {
+      focus_manager->RemoveKeystrokeListener(this);
+    }
   }
+
   root_view_.RemoveAllChildViews(true);
   if (tab_contents_) {
     NavigationController* controller = tab_contents_->controller();
@@ -139,14 +142,18 @@ void ExternalTabContainer::OnDestroy() {
         NotificationType::EXTERNAL_TAB_CLOSED,
         Source<NavigationController>(controller),
         Details<ExternalTabContainer>(this));
+
     tab_contents_->set_delegate(NULL);
     tab_contents_->CloseContents();
     // WARNING: tab_contents_ has likely been deleted.
     tab_contents_ = NULL;
   }
+
+  return true;
 }
 
 void ExternalTabContainer::OnFinalMessage(HWND window) {
+  Uninitialize(window);
   delete this;
 }