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

Side by Side Diff: chrome/browser/ui/passwords/manage_passwords_ui_controller.h

Issue 1311833004: [Password Manager] Implement ShouldShowMultipleAccountUpdateUI in ManagePasswordUIController (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove check Created 5 years, 3 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_UI_PASSWORDS_MANAGE_PASSWORDS_UI_CONTROLLER_H_ 5 #ifndef CHROME_BROWSER_UI_PASSWORDS_MANAGE_PASSWORDS_UI_CONTROLLER_H_
6 #define CHROME_BROWSER_UI_PASSWORDS_MANAGE_PASSWORDS_UI_CONTROLLER_H_ 6 #define CHROME_BROWSER_UI_PASSWORDS_MANAGE_PASSWORDS_UI_CONTROLLER_H_
7 7
8 #include "base/memory/scoped_vector.h" 8 #include "base/memory/scoped_vector.h"
9 #include "base/timer/elapsed_timer.h" 9 #include "base/timer/elapsed_timer.h"
10 #include "chrome/browser/ui/passwords/manage_passwords_state.h" 10 #include "chrome/browser/ui/passwords/manage_passwords_state.h"
(...skipping 134 matching lines...) Expand 10 before | Expand all | Expand 10 after
145 145
146 // Current federated forms. 146 // Current federated forms.
147 const std::vector<const autofill::PasswordForm*>& GetFederatedForms() const { 147 const std::vector<const autofill::PasswordForm*>& GetFederatedForms() const {
148 return passwords_data_.federated_credentials_forms(); 148 return passwords_data_.federated_credentials_forms();
149 } 149 }
150 150
151 // True if the password for previously stored account was overridden, i.e. in 151 // True if the password for previously stored account was overridden, i.e. in
152 // newly submitted form the password is different from stored one. 152 // newly submitted form the password is different from stored one.
153 bool PasswordOverridden() const; 153 bool PasswordOverridden() const;
154 154
155 // Returns true if the multiple account selection prompt for account update
156 // should be presented.
157 bool ShouldShowMultipleAccountUpdateUI() const;
158
155 protected: 159 protected:
156 explicit ManagePasswordsUIController( 160 explicit ManagePasswordsUIController(
157 content::WebContents* web_contents); 161 content::WebContents* web_contents);
158 162
159 // The pieces of saving and blacklisting passwords that interact with 163 // The pieces of saving and blacklisting passwords that interact with
160 // FormManager, split off into internal functions for testing/mocking. 164 // FormManager, split off into internal functions for testing/mocking.
161 virtual void SavePasswordInternal(); 165 virtual void SavePasswordInternal();
162 virtual void UpdatePasswordInternal( 166 virtual void UpdatePasswordInternal(
163 const autofill::PasswordForm& password_form); 167 const autofill::PasswordForm& password_form);
164 virtual void NeverSavePasswordInternal(); 168 virtual void NeverSavePasswordInternal();
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
205 scoped_ptr<base::ElapsedTimer> timer_; 209 scoped_ptr<base::ElapsedTimer> timer_;
206 210
207 // Contains true if the bubble is to be popped up in the next call to 211 // Contains true if the bubble is to be popped up in the next call to
208 // UpdateBubbleAndIconVisibility(). 212 // UpdateBubbleAndIconVisibility().
209 bool should_pop_up_bubble_; 213 bool should_pop_up_bubble_;
210 214
211 DISALLOW_COPY_AND_ASSIGN(ManagePasswordsUIController); 215 DISALLOW_COPY_AND_ASSIGN(ManagePasswordsUIController);
212 }; 216 };
213 217
214 #endif // CHROME_BROWSER_UI_PASSWORDS_MANAGE_PASSWORDS_UI_CONTROLLER_H_ 218 #endif // CHROME_BROWSER_UI_PASSWORDS_MANAGE_PASSWORDS_UI_CONTROLLER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698