In ChromeFrame if window.open is received for a pending external tab, i.e. wh...

chrome/browser/external_tab_container_win.cc

 // Copyright (c) 2010 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.
 
 #include "chrome/browser/external_tab_container_win.h"
 
 #include <string>
 
 #include "app/win_util.h"
 #include "base/i18n/rtl.h"
@@ skipping 315 matching lines @@
       automation_->Send(new AutomationMsg_NavigationStateChanged(
           0, tab_handle_, changed_flags, nav_info));
   }
 }
 
 void ExternalTabContainer::AddNewContents(TabContents* source,
                             TabContents* new_contents,
                             WindowOpenDisposition disposition,
                             const gfx::Rect& initial_pos,
                             bool user_gesture) {
-  DCHECK(automation_ != NULL);
+  if (!automation_) {
+    DCHECK(pending_);
+    LOG(ERROR) << "Invalid automation provider. Dropping new contents notify";
+    delete new_contents;

[amit, 2010/07/16 22:25:38]

we might have to do other cleanup corresponding to
RegisterRenderViewHostForAutomation etc here?

+    return;
+  }
 
   scoped_refptr<ExternalTabContainer> new_container =
       new ExternalTabContainer(NULL, NULL);
 
   // Make sure that ExternalTabContainer instance is initialized with
   // an unwrapped Profile.
   bool result = new_container->Init(
       new_contents->profile()->GetOriginalProfile(),
       NULL,
       initial_pos,
@@ skipping 617 matching lines @@
 
   layout->StartRow(0, 0);
   layout->AddView(info_bar_container);
   layout->StartRow(1, 0);
   layout->AddView(tab_contents_container_);
   SetContentsView(external_tab_view_);
   // Note that SetTabContents must be called after AddChildView is called
   tab_contents_container_->ChangeTabContents(tab_contents_);
 }