Fix browser launch on the login screen.

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

 // Copyright 2014 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 CHROME_BROWSER_CHROMEOS_LOGIN_UI_LOGIN_WEB_DIALOG_H_
 #define CHROME_BROWSER_CHROMEOS_LOGIN_UI_LOGIN_WEB_DIALOG_H_
 
 #include <string>
 
 #include "base/compiler_specific.h"
@@ skipping 65 matching lines @@
   virtual void OnDialogShown(
       content::WebUI* webui,
       content::RenderViewHost* render_view_host) override;
   // NOTE: This function deletes this object at the end.
   virtual void OnDialogClosed(const std::string& json_retval) override;
   virtual void OnCloseContents(
       content::WebContents* source, bool* out_close_dialog) override;
   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,
                        const content::NotificationSource& source,
                        const content::NotificationDetails& details) override;
 
  private:
   content::BrowserContext* browser_context_;
   gfx::NativeWindow parent_window_;
   // Notifications receiver.
   Delegate* delegate_;
 
   base::string16 title_;
   GURL url_;
   content::NotificationRegistrar notification_registrar_;
   bool is_open_;
 
   // Dialog display size.
   int width_;
   int height_;
 
   DISALLOW_COPY_AND_ASSIGN(LoginWebDialog);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_LOGIN_UI_LOGIN_WEB_DIALOG_H_