PlzNavigate: don't send unexpected form urls to the browser

content/renderer/render_frame_impl.cc

Index: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index be46fd8b8128c5eb43ff8e6d8c75d1861c037bce..b67de19e5879156475894a2edac5549ac1ad6b74 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -5398,6 +5398,15 @@ WebNavigationPolicy RenderFrameImpl::decidePolicyForNavigation(
       info.urlRequest.checkForBrowserSideNavigation() &&
       ShouldMakeNetworkRequestForURL(url)) {
     if (info.defaultPolicy == blink::WebNavigationPolicyCurrentTab) {
+      if (RenderThreadImpl::current() &&
+          RenderThreadImpl::current()->layout_test_mode()) {
+        // Layout tests sometimes attempt to load urls of the form
+        // about:blank?foo which the browser doesn't expect and will convert to
+        // about:blank. Don't send these to the browser.
+        if (url.SchemeIs(url::kAboutScheme) && url.path() == "blank")

[nasko, 2017/01/09 23:57:51]

Ugh, is "blank: really parsed as the "path" part of an URL?!

[clamy, 2017/01/10 12:13:52]

Yeah I know. I had to print all parts of the url to find what it was parsed to.
Very not intuitive.

+          return info.defaultPolicy;
+      }
+
       BeginNavigation(info);
       return blink::WebNavigationPolicyHandledByClient;
     } else {