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

Unified Diff: components/autofill/content/browser/autofill_driver_impl.cc

Issue 79103002: Abstracted AcceptPasswordAutofillSuggestion IPC out of core Autofill code. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Review fixes. Created 7 years, 1 month 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/browser/autofill_driver_impl.cc
diff --git a/components/autofill/content/browser/autofill_driver_impl.cc b/components/autofill/content/browser/autofill_driver_impl.cc
index 5e21fb1addb1b05b3f583d690c356863d0f1c92e..ad2454ec8b73c3d054d7c118663cef0d238e7c0f 100644
--- a/components/autofill/content/browser/autofill_driver_impl.cc
+++ b/components/autofill/content/browser/autofill_driver_impl.cc
@@ -61,7 +61,7 @@ AutofillDriverImpl::AutofillDriverImpl(
: content::WebContentsObserver(web_contents),
autofill_manager_(new AutofillManager(
this, delegate, app_locale, enable_download_manager)),
- autofill_external_delegate_(web_contents, autofill_manager_.get(), this) {
+ autofill_external_delegate_(autofill_manager_.get(), this) {
autofill_manager_->SetExternalDelegate(&autofill_external_delegate_);
}
@@ -139,6 +139,16 @@ void AutofillDriverImpl::RendererShouldAcceptDataListSuggestion(
value));
}
+void AutofillDriverImpl::RendererShouldAcceptPasswordAutofillSuggestion(
+ const base::string16& username) {
+ if (!RendererIsAvailable())
+ return;
+ content::RenderViewHost* host = web_contents()->GetRenderViewHost();
+ host->Send(
+ new AutofillMsg_AcceptPasswordAutofillSuggestion(host->GetRoutingID(),
+ username));
+}
+
void AutofillDriverImpl::RendererShouldClearFilledForm() {
if (!RendererIsAvailable())
return;

Powered by Google App Engine
This is Rietveld 408576698