Fix a crash in chrome.exe caused by the external tab container instance getti...

chrome/browser/external_tab_container.h

 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_EXTERNAL_TAB_CONTAINER_H_
 #define CHROME_BROWSER_EXTERNAL_TAB_CONTAINER_H_
 
 #include <atlbase.h>
 #include <atlapp.h>
 #include <atlcrack.h>
@@ skipping 23 matching lines @@
                              public views::Widget,
                              public views::KeystrokeListener,
                              public CWindowImpl<ExternalTabContainer,
                                                 CWindow,
                                                 CWinTraits<WS_POPUP |
                                                     WS_CLIPCHILDREN>> {
  public:
   BEGIN_MSG_MAP(ExternalTabContainer)
     MESSAGE_HANDLER(WM_SIZE, OnSize)
     MESSAGE_HANDLER(WM_SETFOCUS, OnSetFocus)
-    MSG_WM_DESTROY(OnDestroy)
   END_MSG_MAP()
 
   DECLARE_WND_CLASS(chrome::kExternalTabWindowClass)
 
   ExternalTabContainer(AutomationProvider* automation);
   ~ExternalTabContainer();
 
   TabContents* tab_contents() const {
     return tab_contents_;
   }
 
   bool Init(Profile* profile, HWND parent, const gfx::Rect& dimensions,
             unsigned int style);
 
+  // This function gets called from two places, which is fine.
+  // 1. OnFinalMessage
+  // 2. In the destructor.
+  bool Uninitialize(HWND window);
+
   // Overridden from TabContentsDelegate:
   virtual void OpenURLFromTab(TabContents* source,
                               const GURL& url,
                               const GURL& referrer,
                               WindowOpenDisposition disposition,
                               PageTransition::Type transition);
   virtual void NavigationStateChanged(const TabContents* source,
                                       unsigned changed_flags);
   virtual void ReplaceContents(TabContents* source, TabContents* new_contents);
   virtual void AddNewContents(TabContents* source,
@@ skipping 82 matching lines @@
   // A failed navigation like a 404 is followed in chrome with a success
   // navigation for the 404 page. We need to ignore the next navigation
   // to avoid confusing the clients of the external tab. This member variable
   // is set when we need to ignore the next load notification.
   bool ignore_next_load_notification_;
 
   DISALLOW_COPY_AND_ASSIGN(ExternalTabContainer);
 };
 
 #endif  // CHROME_BROWSER_EXTERNAL_TAB_CONTAINER_H__