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

Unified Diff: components/password_manager/content/browser/content_password_manager_driver.cc

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: Get unit tests compiling. 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 side-by-side diff with in-line comments
Download patch
Index: components/password_manager/content/browser/content_password_manager_driver.cc
diff --git a/components/password_manager/content/browser/content_password_manager_driver.cc b/components/password_manager/content/browser/content_password_manager_driver.cc
index c95a292aaf79ca6f0dd790c51f754e7a03855440..dc7f50311e01c5f67ff47e6cdba57c48c637c66d 100644
--- a/components/password_manager/content/browser/content_password_manager_driver.cc
+++ b/components/password_manager/content/browser/content_password_manager_driver.cc
@@ -23,7 +23,8 @@ ContentPasswordManagerDriver::ContentPasswordManagerDriver(
PasswordManagerClient* client)
: WebContentsObserver(web_contents),
password_manager_(client),
- password_generation_manager_(client) {
+ password_generation_manager_(client),
+ password_autofill_manager_(client, GetAutofillManager()->delegate()) {
DCHECK(web_contents);
}
@@ -49,6 +50,17 @@ void ContentPasswordManagerDriver::AccountCreationFormsFound(
forms));
}
+void ContentPasswordManagerDriver::AcceptPasswordAutofillSuggestion(
+ const base::string16& username,
+ const base::string16& password) {
+ DCHECK(web_contents());
+ content::RenderViewHost* host = web_contents()->GetRenderViewHost();
+ host->Send(
+ new AutofillMsg_AcceptPasswordAutofillSuggestion(host->GetRoutingID(),
+ username,
+ password));
+}
+
bool ContentPasswordManagerDriver::DidLastPageLoadEncounterSSLErrors() {
DCHECK(web_contents());
content::NavigationEntry* entry =
@@ -75,6 +87,11 @@ PasswordManager* ContentPasswordManagerDriver::GetPasswordManager() {
return &password_manager_;
}
+PasswordAutofillManager*
+ContentPasswordManagerDriver::GetPasswordAutofillManager() {
+ return &password_autofill_manager_;
+}
+
void ContentPasswordManagerDriver::DidNavigateMainFrame(
const content::LoadCommittedDetails& details,
const content::FrameNavigateParams& params) {
@@ -94,6 +111,12 @@ bool ContentPasswordManagerDriver::OnMessageReceived(
IPC_MESSAGE_FORWARD(AutofillHostMsg_PasswordFormSubmitted,
&password_manager_,
PasswordManager::OnPasswordFormSubmitted)
+ IPC_MESSAGE_FORWARD(AutofillHostMsg_ShowPasswordSuggestions,
+ &password_manager_,
+ PasswordManager::OnShowPasswordSuggestions)
+ IPC_MESSAGE_FORWARD(AutofillHostMsg_AddPasswordFormMapping,
+ &password_manager_,
+ PasswordManager::OnAddPasswordFormMapping)
IPC_MESSAGE_UNHANDLED(handled = false)
IPC_END_MESSAGE_MAP()

Powered by Google App Engine
This is Rietveld 408576698