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

Unified Diff: chrome/browser/password_manager_delegate_impl.cc

Issue 9873028: Make sure password manager does not display at the same time as one-click on (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 8 years, 9 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/password_manager_delegate_impl.cc
diff --git a/chrome/browser/password_manager_delegate_impl.cc b/chrome/browser/password_manager_delegate_impl.cc
index 10eda7bd831f4fd16780030538e9e2c74dc1fd48..5a3b5720831d52856ea8923d5e5d961d19eb49ff 100644
--- a/chrome/browser/password_manager_delegate_impl.cc
+++ b/chrome/browser/password_manager_delegate_impl.cc
@@ -133,10 +133,8 @@ void PasswordManagerDelegateImpl::AddSavePasswordInfoBarIfPermitted(
PasswordFormManager* form_to_save) {
// Don't show the password manager infobar if this form is for a google
// account and we are going to show the one-click singin infobar.
- // For now, one-click signin is fully implemented only on windows. When
- // the feature is finally implemented on mac and linux, will need to add
- // OS_MACOSX and OS_LINUX.
-#if defined(OS_WIN)
+ // For now, one-click signin is fully implemented only on windows.
+#if defined(ENABLE_ONE_CLICK_SIGNIN)
GURL realm(form_to_save->realm());
if (realm == GURL(GaiaUrls::GetInstance()->gaia_login_form_realm()) &&
OneClickSigninHelper::CanOffer(tab_contents_->web_contents(), true)) {
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698