Fix the content_browsertests everywhere maybe.

chrome/browser/ui/views/chrome_web_dialog_view.cc

 // Copyright (c) 2012 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/ui/browser_dialogs.h"
 #include "chrome/browser/ui/webui/chrome_web_contents_handler.h"
 #include "ui/views/controls/webview/web_dialog_view.h"
 #include "ui/views/widget/widget.h"
 
 namespace chrome {
 
 // Declared in browser_dialogs.h so that others don't need to depend on our .h.
 gfx::NativeWindow ShowWebDialog(gfx::NativeWindow parent,
                                 content::BrowserContext* context,
                                 ui::WebDialogDelegate* delegate) {
-  views::Widget* widget = views::Widget::CreateWindowWithParent(
-      new views::WebDialogView(context, delegate, new ChromeWebContentsHandler),
-      parent);
+  views::Widget* widget = NULL;
+  if (parent) {
+    widget = views::Widget::CreateWindowWithParent(
+        new views::WebDialogView(context,
+                                 delegate,
+                                 new ChromeWebContentsHandler),
+        parent);
+  } else {
+    // We shouldn't be called with a NULL parent, but sometimes are.
+    widget = views::Widget::CreateWindow(

[Ben Goodger (Google), 2013/01/15 18:19:52]

This is confusing at first glance due to similarity to CreateWindowWithParent, I
wonder if CreateWindow should be called CreateTopLevelWindow?

Fine for that to be a different CL.

+        new views::WebDialogView(context,
+                                 delegate,
+                                 new ChromeWebContentsHandler));
+  }
+
   widget->Show();
   return widget->GetNativeWindow();
 }
 
 }  // namespace chrome