Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1798)

Unified Diff: content/renderer/render_view_impl.cc

Issue 8669014: Fix a bug where redirect chain gets lost on process swap. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/renderer/render_view_impl.cc
diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc
index 502bb67479243ba36ee31077b9e867be424f6bb6..9557c025544305ce9086175b305bc800f915a1fa 100644
--- a/content/renderer/render_view_impl.cc
+++ b/content/renderer/render_view_impl.cc
@@ -1990,7 +1990,7 @@ WebNavigationPolicy RenderViewImpl::decidePolicyForNavigation(
// boundaries. This is currently expected to break some script calls and
// navigations, such as form submissions.
const CommandLine& command_line = *CommandLine::ForCurrentProcess();
- if (!frame->parent() && (is_content_initiated || is_redirect) &&
+ if (!frame->parent() && (is_content_initiated) &&
Matt Perry 2011/11/23 02:51:20 this change makes it so that the renderer no longe
command_line.HasSwitch(switches::kEnableStrictSiteIsolation)) {
GURL referrer(request.httpHeaderField(WebString::fromUTF8("Referer")));
OpenURL(frame, url, referrer, default_policy);

Powered by Google App Engine
This is Rietveld 408576698