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

Side by Side Diff: components/autofill/core/browser/password_autofill_manager.h

Issue 184103016: Autofill: Refactoring to support fetching password after a username is selected (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 COMPONENTS_AUTOFILL_CORE_BROWSER_PASSWORD_AUTOFILL_MANAGER_H_ 5 #ifndef COMPONENTS_AUTOFILL_CORE_BROWSER_PASSWORD_AUTOFILL_MANAGER_H_
6 #define COMPONENTS_AUTOFILL_CORE_BROWSER_PASSWORD_AUTOFILL_MANAGER_H_ 6 #define COMPONENTS_AUTOFILL_CORE_BROWSER_PASSWORD_AUTOFILL_MANAGER_H_
7 7
8 // This file was contains some repeated code from 8 // This file was contains some repeated code from
9 // chrome/renderer/autofill/password_autofill_manager because as we move to the 9 // chrome/renderer/autofill/password_autofill_manager because as we move to the
10 // new Autofill UI we needs these functions in both the browser and renderer. 10 // new Autofill UI we needs these functions in both the browser and renderer.
11 // Once the move is completed the repeated code in the renderer half should be 11 // Once the move is completed the repeated code in the renderer half should be
12 // removed. 12 // removed.
13 // http://crbug.com/51644 13 // http://crbug.com/51644
14 14
15 #include <map> 15 #include <map>
16 16
17 #include "components/autofill/core/common/password_form_fill_data.h" 17 #include "components/autofill/core/common/password_form_fill_data.h"
18 #include "components/password_manager/core/browser/password_manager_driver.h"
18 19
19 namespace autofill { 20 namespace autofill {
20 21
21 class AutofillDriver; 22 class AutofillDriver;
22 23
23 // This class is responsible for filling password forms. 24 // This class is responsible for filling password forms.
24 class PasswordAutofillManager { 25 class PasswordAutofillManager {
25 public: 26 public:
26 explicit PasswordAutofillManager(AutofillDriver* autofill_driver); 27 explicit PasswordAutofillManager(PasswordManagerDriver* driver);
27 virtual ~PasswordAutofillManager(); 28 virtual ~PasswordAutofillManager();
28 29
29 // Fills the password associated with user name |username|. Returns true if 30 // Fills the password associated with user name |username|. Returns true if
30 // the username and password fields were filled, false otherwise. 31 // the username and password fields were filled, false otherwise.
31 bool DidAcceptAutofillSuggestion(const FormFieldData& field, 32 bool DidAcceptAutofillSuggestion(const FormFieldData& field,
32 const base::string16& username); 33 const base::string16& username);
33 34
34 // Invoked when a password mapping is added. 35 // Invoked when a password mapping is added.
35 void AddPasswordFormMapping( 36 void AddPasswordFormMapping(
36 const FormFieldData& username_element, 37 const FormFieldData& username_element,
37 const PasswordFormFillData& password); 38 const PasswordFormFillData& password);
38 39
39 // Invoked to clear any page specific cached values. 40 // Invoked to clear any page specific cached values.
40 void Reset(); 41 void Reset();
41 42
42 private: 43 private:
43 // TODO(csharp): Modify the AutofillExternalDeletegate code so that it can 44 // TODO(csharp): Modify the AutofillExternalDeletegate code so that it can
44 // figure out if a entry is a password one without using this mapping. 45 // figure out if a entry is a password one without using this mapping.
45 // crbug.com/118601 46 // crbug.com/118601
46 typedef std::map<FormFieldData, 47 typedef std::map<FormFieldData,
47 PasswordFormFillData> 48 PasswordFormFillData>
48 LoginToPasswordInfoMap; 49 LoginToPasswordInfoMap;
49 50
50 // Returns true if |current_username| matches a username for one of the 51 // Returns true if |current_username| matches a username for one of the
51 // login mappings in |password|. 52 // login mappings in |password|.
52 bool WillFillUserNameAndPassword( 53 bool GetPasswordForUsername(
53 const base::string16& current_username, 54 const base::string16& current_username,
54 const PasswordFormFillData& password); 55 const PasswordFormFillData& fill_data,
56 base::string16* password);
55 57
56 // Finds login information for a |node| that was previously filled. 58 // Finds login information for a |node| that was previously filled.
57 bool FindLoginInfo(const FormFieldData& field, 59 bool FindLoginInfo(const FormFieldData& field,
58 PasswordFormFillData* found_password); 60 PasswordFormFillData* found_password);
59 61
60 // The logins we have filled so far with their associated info. 62 // The logins we have filled so far with their associated info.
61 LoginToPasswordInfoMap login_to_password_info_; 63 LoginToPasswordInfoMap login_to_password_info_;
62 64
63 // Provides driver-level context to the shared code of the component. Must 65 // Provides driver-level context to the shared code of the component. Must
64 // outlive |this|. 66 // outlive |this|.
65 AutofillDriver* const autofill_driver_; // weak 67 PasswordManagerDriver* const driver_; // weak
66 68
67 DISALLOW_COPY_AND_ASSIGN(PasswordAutofillManager); 69 DISALLOW_COPY_AND_ASSIGN(PasswordAutofillManager);
68 }; 70 };
69 71
70 } // namespace autofill 72 } // namespace autofill
71 73
72 #endif // COMPONENTS_AUTOFILL_CORE_BROWSER_PASSWORD_AUTOFILL_MANAGER_H_ 74 #endif // COMPONENTS_AUTOFILL_CORE_BROWSER_PASSWORD_AUTOFILL_MANAGER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698