Commit 16a13fad authored by creis@chromium.org's avatar creis@chromium.org

Temporary fix for CWS client redirect issue to undo a regression.

This re-introduces a smaller bug fixed by r231370 until the underlying
problem in WebDataSource::replacesCurrentHistoryItem is resolved.

BUG=313572, 311271
TEST=Try to add an app in CWS when not signed in.  Go back from signin page.
R=nasko@chromium.org

Review URL: https://codereview.chromium.org/59073010

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@233454 0039d316-1c4b-4281-b951-d872f2087c98
parent e4037964
...@@ -3156,6 +3156,13 @@ void WebContentsImpl::RequestTransferURL( ...@@ -3156,6 +3156,13 @@ void WebContentsImpl::RequestTransferURL(
// Navigations in Web UI pages count as browser-initiated navigations. // Navigations in Web UI pages count as browser-initiated navigations.
params.is_renderer_initiated = false; params.is_renderer_initiated = false;
// TODO(creis): Remove this line. In the short term, it fixes a regression
// (http://crbug.com/313572) by partially reverting r231370.
// The underlying problem is that WebDataSource::replacesCurrentHistoryItem
// should be returning false when the CWS goes to the sign-in page.
// See http://crbug.com/311721.
params.should_replace_current_entry = false;
} }
new_contents = OpenURL(params); new_contents = OpenURL(params);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment