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

Side by Side Diff: chrome/browser/password_manager/chrome_password_manager_client.h

Issue 1151373006: Update Confirmation UI for saved password change (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Clean up Created 5 years, 6 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_PASSWORD_MANAGER_CHROME_PASSWORD_MANAGER_CLIENT_H_ 5 #ifndef CHROME_BROWSER_PASSWORD_MANAGER_CHROME_PASSWORD_MANAGER_CLIENT_H_
6 #define CHROME_BROWSER_PASSWORD_MANAGER_CHROME_PASSWORD_MANAGER_CLIENT_H_ 6 #define CHROME_BROWSER_PASSWORD_MANAGER_CHROME_PASSWORD_MANAGER_CLIENT_H_
7 7
8 #include "base/basictypes.h" 8 #include "base/basictypes.h"
9 #include "base/compiler_specific.h" 9 #include "base/compiler_specific.h"
10 #include "base/prefs/pref_member.h" 10 #include "base/prefs/pref_member.h"
(...skipping 76 matching lines...) Expand 10 before | Expand all | Expand 10 after
87 content::WebContents* contents, 87 content::WebContents* contents,
88 autofill::AutofillClient* autofill_client); 88 autofill::AutofillClient* autofill_client);
89 89
90 // Observer for PasswordGenerationPopup events. Used for testing. 90 // Observer for PasswordGenerationPopup events. Used for testing.
91 void SetTestObserver(autofill::PasswordGenerationPopupObserver* observer); 91 void SetTestObserver(autofill::PasswordGenerationPopupObserver* observer);
92 92
93 // Returns true if the bubble UI is enabled, and false if we're still using 93 // Returns true if the bubble UI is enabled, and false if we're still using
94 // the sad old Infobar UI. 94 // the sad old Infobar UI.
95 static bool IsTheHotNewBubbleUIEnabled(); 95 static bool IsTheHotNewBubbleUIEnabled();
96 96
97 // Returns true if the UI for confirmation of update password is enabled.
vabr (Chromium) 2015/06/03 16:33:26 nit: update password -> password update
vabr (Chromium) 2015/06/03 16:33:26 Also, this should be marked as "override" and go u
dvadym 2015/06/19 15:27:29 Done.
dvadym 2015/06/19 15:27:29 Done.
98 bool IsPasswordUpdateUIEnabled() const;
99
97 // Returns true if the password manager should be enabled during sync signin. 100 // Returns true if the password manager should be enabled during sync signin.
98 static bool EnabledForSyncSignin(); 101 static bool EnabledForSyncSignin();
99 102
100 protected: 103 protected:
101 // Callable for tests. 104 // Callable for tests.
102 ChromePasswordManagerClient(content::WebContents* web_contents, 105 ChromePasswordManagerClient(content::WebContents* web_contents,
103 autofill::AutofillClient* autofill_client); 106 autofill::AutofillClient* autofill_client);
104 107
105 private: 108 private:
106 enum AutofillForSyncCredentialsState { 109 enum AutofillForSyncCredentialsState {
(...skipping 67 matching lines...) Expand 10 before | Expand all | Expand 10 after
174 bool sync_credential_was_filtered_; 177 bool sync_credential_was_filtered_;
175 178
176 // Set to false to disable password saving (will no longer ask if you 179 // Set to false to disable password saving (will no longer ask if you
177 // want to save passwords but will continue to fill passwords). 180 // want to save passwords but will continue to fill passwords).
178 BooleanPrefMember saving_passwords_enabled_; 181 BooleanPrefMember saving_passwords_enabled_;
179 182
180 DISALLOW_COPY_AND_ASSIGN(ChromePasswordManagerClient); 183 DISALLOW_COPY_AND_ASSIGN(ChromePasswordManagerClient);
181 }; 184 };
182 185
183 #endif // CHROME_BROWSER_PASSWORD_MANAGER_CHROME_PASSWORD_MANAGER_CLIENT_H_ 186 #endif // CHROME_BROWSER_PASSWORD_MANAGER_CHROME_PASSWORD_MANAGER_CLIENT_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698