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

Unified Diff: components/autofill/content/renderer/password_autofill_agent.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: Actually fix compile failure. Created 6 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
Index: components/autofill/content/renderer/password_autofill_agent.h
diff --git a/components/autofill/content/renderer/password_autofill_agent.h b/components/autofill/content/renderer/password_autofill_agent.h
index 582f331b3d5fb15dcdb2ecf0a006b7cc661866b8..2e45d46b5ef74ef01a11e4d59e2e43a59820b717 100644
--- a/components/autofill/content/renderer/password_autofill_agent.h
+++ b/components/autofill/content/renderer/password_autofill_agent.h
@@ -38,10 +38,12 @@ class PasswordAutofillAgent : public content::RenderViewObserver {
bool TextFieldHandlingKeyDown(const blink::WebInputElement& element,
const blink::WebKeyboardEvent& event);
- // Fills the password associated with user name |username|. Returns true if
- // the username and password fields were filled, false otherwise.
- bool DidAcceptAutofillSuggestion(const blink::WebNode& node,
- const blink::WebString& username);
+ // Fills the username and password fields of this form with the given values.
+ // Returns true if the fields were filled, false otherwise.
+ bool AcceptSuggestion(const blink::WebNode& node,
+ const blink::WebString& username,
+ const blink::WebString& password);
+
// A no-op. Password forms are not previewed, so they do not need to be
// cleared when the selection changes. However, this method returns
// true when |node| is fillable by password Autofill.

Powered by Google App Engine
This is Rietveld 408576698