Restore parent semantings for external tab container creation, and error chec...

chrome/browser/external_tab_container.cc

 // 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.
 
 #include "chrome/browser/external_tab_container.h"
 
 #include "app/win_util.h"
 #include "base/logging.h"
 #include "base/win_util.h"
 #include "chrome/browser/automation/automation_provider.h"
@@ skipping 31 matching lines @@
 bool ExternalTabContainer::Init(Profile* profile,
                                 HWND parent,
                                 const gfx::Rect& bounds,
                                 DWORD style) {
   if (IsWindow()) {
     NOTREACHED();
     return false;
   }
 
   set_window_style(WS_POPUP);
-  views::WidgetWin::Init(parent, bounds, true);
+  views::WidgetWin::Init(NULL, bounds, true);
+  if (!IsWindow()) {
+    NOTREACHED();
+    return false;
+  }
 
   // We don't ever remove the prop because the lifetime of this object
   // is the same as the lifetime of the window
   SetProp(GetNativeView(), kWindowObjectKey, this);
 
   views::FocusManager* focus_manager =
       views::FocusManager::GetFocusManager(GetNativeView());
   focus_manager->AddKeystrokeListener(this);
 
   tab_contents_ = new TabContents(profile, NULL, MSG_ROUTING_NONE, NULL);
@@ skipping 322 matching lines @@
     NotificationService::current()->Notify(
         NotificationType::EXTERNAL_TAB_CLOSED,
         Source<NavigationController>(&tab_contents_->controller()),
         Details<ExternalTabContainer>(this));
 
     delete tab_contents_;
     tab_contents_ = NULL;
   }
 }