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

Unified Diff: components/autofill/content/renderer/autofill_agent.cc

Issue 2180093002: [Autofill] Switch on use_new_wrapper_types mode. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address nit Created 4 years, 5 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/autofill_agent.cc
diff --git a/components/autofill/content/renderer/autofill_agent.cc b/components/autofill/content/renderer/autofill_agent.cc
index a0908e15a3356355328311dc95c46df2ab973222..b2c81f9982574fc9cd51545260ce31039c8b74ae 100644
--- a/components/autofill/content/renderer/autofill_agent.cc
+++ b/components/autofill/content/renderer/autofill_agent.cc
@@ -42,7 +42,6 @@
#include "content/public/common/url_constants.h"
#include "content/public/renderer/render_frame.h"
#include "content/public/renderer/render_view.h"
-#include "mojo/common/common_type_converters.h"
#include "net/cert/cert_status_flags.h"
#include "services/shell/public/cpp/interface_provider.h"
#include "services/shell/public/cpp/interface_registry.h"
@@ -514,7 +513,7 @@ void AutofillAgent::OnPing() {
}
void AutofillAgent::FieldTypePredictionsAvailable(
- mojo::Array<FormDataPredictions> forms) {
+ const std::vector<FormDataPredictions>& forms) {
for (const auto& form : forms) {
form_cache_.ShowPredictions(form);
}
@@ -541,35 +540,35 @@ void AutofillAgent::ClearPreviewedForm() {
}
}
-void AutofillAgent::FillFieldWithValue(const mojo::String& value) {
+void AutofillAgent::FillFieldWithValue(const base::string16& value) {
WebInputElement* input_element = toWebInputElement(&element_);
if (input_element) {
- DoFillFieldWithValue(value.To<base::string16>(), input_element);
+ DoFillFieldWithValue(value, input_element);
input_element->setAutofilled(true);
}
}
-void AutofillAgent::PreviewFieldWithValue(const mojo::String& value) {
+void AutofillAgent::PreviewFieldWithValue(const base::string16& value) {
WebInputElement* input_element = toWebInputElement(&element_);
if (input_element)
- DoPreviewFieldWithValue(value.To<base::string16>(), input_element);
+ DoPreviewFieldWithValue(value, input_element);
}
-void AutofillAgent::AcceptDataListSuggestion(const mojo::String& value) {
- DoAcceptDataListSuggestion(value.To<base::string16>());
+void AutofillAgent::AcceptDataListSuggestion(const base::string16& value) {
+ DoAcceptDataListSuggestion(value);
}
-void AutofillAgent::FillPasswordSuggestion(const mojo::String& username,
- const mojo::String& password) {
- bool handled = password_autofill_agent_->FillSuggestion(
- element_, username.To<base::string16>(), password.To<base::string16>());
+void AutofillAgent::FillPasswordSuggestion(const base::string16& username,
+ const base::string16& password) {
+ bool handled =
+ password_autofill_agent_->FillSuggestion(element_, username, password);
DCHECK(handled);
}
-void AutofillAgent::PreviewPasswordSuggestion(const mojo::String& username,
- const mojo::String& password) {
- bool handled = password_autofill_agent_->PreviewSuggestion(
- element_, username.To<base::string16>(), password.To<base::string16>());
+void AutofillAgent::PreviewPasswordSuggestion(const base::string16& username,
+ const base::string16& password) {
+ bool handled =
+ password_autofill_agent_->PreviewSuggestion(element_, username, password);
DCHECK(handled);
}
@@ -735,9 +734,7 @@ void AutofillAgent::QueryAutofillSuggestions(
is_popup_possibly_visible_ = true;
- GetAutofillDriver()->SetDataList(
- mojo::Array<mojo::String>::From(data_list_values),
- mojo::Array<mojo::String>::From(data_list_labels));
+ GetAutofillDriver()->SetDataList(data_list_values, data_list_labels);
GetAutofillDriver()->QueryFormFieldAutofill(
autofill_query_id_, form, field,
render_frame()->GetRenderView()->ElementBoundsInWindow(element_));
@@ -767,7 +764,7 @@ void AutofillAgent::ProcessForms() {
// Always communicate to browser process for topmost frame.
if (!forms.empty() || !frame->parent()) {
- GetAutofillDriver()->FormsSeen(std::move(forms), forms_seen_timestamp);
+ GetAutofillDriver()->FormsSeen(forms, forms_seen_timestamp);
}
}

Powered by Google App Engine
This is Rietveld 408576698