Fix browser launch on the login screen.

ui/web_dialogs/web_dialog_delegate.h

Index: ui/web_dialogs/web_dialog_delegate.h
diff --git a/ui/web_dialogs/web_dialog_delegate.h b/ui/web_dialogs/web_dialog_delegate.h
index ccfd26ef98287f8e292653cf30d97d2d59d672b5..b1fb3afdd17b2f785bb9640e2c831bd9b8b76667 100644
--- a/ui/web_dialogs/web_dialog_delegate.h
+++ b/ui/web_dialogs/web_dialog_delegate.h
@@ -9,6 +9,7 @@
 #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"
@@ -128,6 +129,18 @@ class WEB_DIALOGS_EXPORT WebDialogDelegate {
                                     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) {}