Commit 709d7f88 authored by jochen@chromium.org's avatar jochen@chromium.org

Disallow background tabs without matching user gesture for createWindow()

BUG=431335
R=mkwst@chromium.org
TEST=content_shell doesn't do tabs, so no tests

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

git-svn-id: svn://svn.chromium.org/blink/trunk@185430 bbb929c8-8fbe-4397-9dbb-9b2b20218538
parent a9352c05
......@@ -229,6 +229,8 @@ Page* ChromeClientImpl::createWindow(LocalFrame* frame, const FrameLoadRequest&
WebNavigationPolicy policy = static_cast<WebNavigationPolicy>(navigationPolicy);
if (policy == WebNavigationPolicyIgnore)
policy = getNavigationPolicy();
else if (policy == WebNavigationPolicyNewBackgroundTab && getNavigationPolicy() != WebNavigationPolicyNewBackgroundTab)
policy = WebNavigationPolicyNewForegroundTab;
ASSERT(frame->document());
Fullscreen::fullyExitFullscreen(*frame->document());
......@@ -301,6 +303,8 @@ void ChromeClientImpl::show(NavigationPolicy navigationPolicy)
WebNavigationPolicy policy = static_cast<WebNavigationPolicy>(navigationPolicy);
if (policy == WebNavigationPolicyIgnore)
policy = getNavigationPolicy();
else if (policy == WebNavigationPolicyNewBackgroundTab && getNavigationPolicy() != WebNavigationPolicyNewBackgroundTab)
policy = WebNavigationPolicyNewForegroundTab;
m_webView->client()->show(policy);
}
......
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