Form submissions opening in a new window don't need special treatment anymore.

content/shell/browser/shell.h

Index: content/shell/browser/shell.h
diff --git a/content/shell/browser/shell.h b/content/shell/browser/shell.h
index 708b3d572e1ca5b285c9dad78f5113e56b9a9700..4068230fb90dd0c7508ce59d894dc5b01dc0598f 100644
--- a/content/shell/browser/shell.h
+++ b/content/shell/browser/shell.h
@@ -10,6 +10,7 @@
 #include <vector>
 
 #include "base/callback_forward.h"
+#include "base/memory/ref_counted.h"
 #include "base/strings/string_piece.h"
 #include "build/build_config.h"
 #include "content/public/browser/web_contents_delegate.h"
@@ -85,10 +86,11 @@ class Shell : public WebContentsDelegate,
   // Do one time initialization at application startup.
   static void Initialize();
 
-  static Shell* CreateNewWindow(BrowserContext* browser_context,
-                                const GURL& url,
-                                SiteInstance* site_instance,
-                                const gfx::Size& initial_size);
+  static Shell* CreateNewWindow(
+      BrowserContext* browser_context,
+      const GURL& url,
+      const scoped_refptr<SiteInstance>& site_instance,

[Łukasz Anforowicz, 2016/10/20 22:56:42]

scoped_refptr is better than a raw pointer, right? :-)
(otherwise I would have to pass |params.source_site_instance.get()| - yucky
"get"...)

+      const gfx::Size& initial_size);
 
   // Returns the Shell object corresponding to the given RenderViewHost.
   static Shell* FromRenderViewHost(RenderViewHost* rvh);