Change disposition to POPUP for navigations in a new process with no user gesture to trigger popup blocker checks.
BUG=118317
TEST=N/A
Review URL: https://chromiumcodereview.appspot.com/10883031
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153542 0039d316-1c4b-4281-b951-d872f2087c98
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 30d1190..077e5b3e 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -1284,8 +1284,10 @@
// When the opener is suppressed, the original renderer cannot access the
// new window. As a result, we need to show and navigate the window here.
gfx::Rect initial_pos;
+ // TODO(cdn) Fix popup white-listing for links that open in a new process.
AddNewContents(
- new_contents, params.disposition, initial_pos, params.user_gesture);
+ new_contents, params.user_gesture ? params.disposition : NEW_POPUP,
+ initial_pos, params.user_gesture);
content::OpenURLParams open_params(params.target_url, content::Referrer(),
CURRENT_TAB,
