Fix browser launch on the login screen.

ui/web_dialogs/web_dialog_delegate.h

 // 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.
 
 #ifndef UI_WEB_DIALOGS_WEB_DIALOG_DELEGATE_H_
 #define UI_WEB_DIALOGS_WEB_DIALOG_DELEGATE_H_
 
 #include <string>
 #include <vector>
 
 #include "base/strings/string16.h"
+#include "content/public/browser/web_contents_delegate.h"
 #include "ui/base/ui_base_types.h"
 #include "ui/base/window_open_disposition.h"
 #include "ui/web_dialogs/web_dialogs_export.h"
 
 class GURL;
 
 namespace content {
 class RenderViewHost;
 class WebContents;
 class WebUI;
@@ skipping 99 matching lines @@
   // WebContent where the operation originated. |disposition| controls how the
   // new tab should be opened. |initial_pos| is the position of the window if a
   // new window is created. |user_gesture| is true if the operation was started
   // by a user gesture. Return false to use the default handler.
   virtual bool HandleAddNewContents(content::WebContents* source,
                                     content::WebContents* new_contents,
                                     WindowOpenDisposition disposition,
                                     const gfx::Rect& initial_pos,
                                     bool user_gesture);
 
+  // A callback to control whether a WebContents will be created. Returns
+  // false to disallow the creation. Return true to use the default handler.
+  virtual bool HandleShouldCreateWebContents(

[dzhioev (left Google), 2014/12/22 18:30:13]

I don't think we need to pass all the arguments of ShouldCreateWebContents to
HandleShouldCreateWebContents. For our purpose we don't need any parameters at
all.

[Ivan Podogov, 2014/12/23 06:54:51]

True, but I'm not sure if it is a good idea to leave no parameters at all, and
which we should keep otherwise.

[dzhioev (left Google), 2014/12/23 14:34:25]

Let's drop all of them. It is not hard to add parameters later, if somebody need
them.

[Ivan Podogov, 2014/12/23 14:41:37]

Done.

+      content::WebContents* web_contents,
+      int route_id,
+      int main_frame_route_id,
+      WindowContainerType window_container_type,
+      const base::string16& frame_name,
+      const GURL& target_url,
+      const std::string& partition_id,
+      content::SessionStorageNamespace* session_storage_namespace);
+
   // Stores the dialog bounds.
   virtual void StoreDialogSize(const gfx::Size& dialog_size) {}
 
   virtual ~WebDialogDelegate() {}
 };
 
 }  // namespace ui
 
 #endif  // UI_WEB_DIALOGS_WEB_DIALOG_DELEGATE_H_