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

Unified Diff: components/autofill/core/browser/autofill_manager.cc

Issue 17572015: Begin abstracting sending of IPC from autofill core code. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 7 years, 6 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/core/browser/autofill_manager.cc
diff --git a/components/autofill/core/browser/autofill_manager.cc b/components/autofill/core/browser/autofill_manager.cc
index 338de9eb17c628c82bc4bfecc75df40e7b1b0f55..7acb75f1f766bfc1a46b0e8fa8682e86eab52f64 100644
--- a/components/autofill/core/browser/autofill_manager.cc
+++ b/components/autofill/core/browser/autofill_manager.cc
@@ -534,8 +534,7 @@ void AutofillManager::OnFillAutofillFormData(int query_id,
}
}
- host->Send(new AutofillMsg_FormDataFilled(host->GetRoutingID(), query_id,
- result));
+ driver_->SendFormDataToRenderer(query_id, result);
return;
}
@@ -579,8 +578,7 @@ void AutofillManager::OnFillAutofillFormData(int query_id,
if (autofilled_form_signatures_.size() > kMaxRecentFormSignaturesToRemember)
autofilled_form_signatures_.pop_back();
- host->Send(new AutofillMsg_FormDataFilled(
- host->GetRoutingID(), query_id, result));
+ driver_->SendFormDataToRenderer(query_id, result);
}
void AutofillManager::OnShowAutofillDialog() {
@@ -955,10 +953,10 @@ bool AutofillManager::GetHost(RenderViewHost** host) const {
return false;
}
- *host = driver_->GetWebContents()->GetRenderViewHost();
- if (!*host)
+ if (!driver_->RendererIsAvailable())
return false;
+ *host = driver_->GetWebContents()->GetRenderViewHost();
return true;
}
« no previous file with comments | « components/autofill/core/browser/autofill_driver.h ('k') | components/autofill/core/browser/autofill_manager_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698