Chromium Code Reviews| Index: content/renderer/render_frame_impl.cc |
| diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc |
| index 11e7198d70cf15b155ba0bac452aacadb0a4cd91..8de6942440ce87b3f937e6e5ea1130ec64bfc7cc 100644 |
| --- a/content/renderer/render_frame_impl.cc |
| +++ b/content/renderer/render_frame_impl.cc |
| @@ -5469,15 +5469,6 @@ 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. |
|
Charlie Reis
2017/01/19 22:36:35
Awesome. I love getting rid of checks like this.
|
| - if (url.SchemeIs(url::kAboutScheme) && url.path() == "blank") |
| - return info.defaultPolicy; |
| - } |
| - |
| BeginNavigation(info); |
| return blink::WebNavigationPolicyHandledByClient; |
| } else { |