Fix browser launch on the login screen.

chrome/browser/chromeos/login/ui/login_web_dialog.h

Index: chrome/browser/chromeos/login/ui/login_web_dialog.h
diff --git a/chrome/browser/chromeos/login/ui/login_web_dialog.h b/chrome/browser/chromeos/login/ui/login_web_dialog.h
index c9d38cd5c2849259df8bd40c41de6cde8d114aa2..b6c20e2b9fdb6e4d15fff8f2e3c85c476ae5c1bf 100644
--- a/chrome/browser/chromeos/login/ui/login_web_dialog.h
+++ b/chrome/browser/chromeos/login/ui/login_web_dialog.h
@@ -83,6 +83,19 @@ class LoginWebDialog : public ui::WebDialogDelegate,
   virtual bool ShouldShowDialogTitle() const override;
   virtual bool HandleContextMenu(
       const content::ContextMenuParams& params) override;
+  virtual bool HandleOpenURLFromTab(

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

Is this method still needed?

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

Yes, so that we could suppress OpenURL message originating from auto-reload.
Without this method, a browser window still opens when extension is unavailable.

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

I'm not quiet sure if such behaviour is correct. Let's add somebody from
content/OWNERS to the review.

+      content::WebContents* source,
+      const content::OpenURLParams& params,
+      content::WebContents** out_new_contents) override;
+  virtual bool HandleShouldCreateWebContents(
+      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) override;
 
   // content::NotificationObserver implementation.
   virtual void Observe(int type,