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

Unified Diff: chrome/browser/ui/sync/one_click_signin_helper.cc

Issue 11778029: Be more consistent about respecting link click dispositions in infobars: for most infobars, we shou… (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 7 years, 8 months 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
« no previous file with comments | « chrome/browser/autofill/autofill_cc_infobar_delegate.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/sync/one_click_signin_helper.cc
===================================================================
--- chrome/browser/ui/sync/one_click_signin_helper.cc (revision 194436)
+++ chrome/browser/ui/sync/one_click_signin_helper.cc (working copy)
@@ -546,7 +546,8 @@
WindowOpenDisposition disposition) {
RecordHistogramAction(one_click_signin::HISTOGRAM_LEARN_MORE);
content::OpenURLParams params(
- GURL(chrome::kChromeSyncLearnMoreURL), content::Referrer(), disposition,
+ GURL(chrome::kChromeSyncLearnMoreURL), content::Referrer(),
+ (disposition == CURRENT_TAB) ? NEW_FOREGROUND_TAB : disposition,
content::PAGE_TRANSITION_LINK, false);
owner()->GetWebContents()->OpenURL(params);
return false;
« no previous file with comments | « chrome/browser/autofill/autofill_cc_infobar_delegate.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698