Chromium Code Reviews| Index: chrome/renderer/autofill/autofill_renderer_browsertest.cc |
| diff --git a/chrome/renderer/autofill/autofill_renderer_browsertest.cc b/chrome/renderer/autofill/autofill_renderer_browsertest.cc |
| index eee167a8d980523bedc3299b9a52eb838d7140ec..2ce84e801e1bcc9ad91c5df0a1cece58ae5bbd74 100644 |
| --- a/chrome/renderer/autofill/autofill_renderer_browsertest.cc |
| +++ b/chrome/renderer/autofill/autofill_renderer_browsertest.cc |
| @@ -65,13 +65,13 @@ class FakeContentAutofillDriver : public mojom::AutofillDriver { |
| // mojom::AutofillDriver: |
| void FirstUserGestureObserved() override {} |
| - void FormsSeen(mojo::Array<FormData> forms, |
| + void FormsSeen(const std::vector<FormData>& forms, |
| base::TimeTicks timestamp) override { |
| // FormsSeen() could be called multiple times and sometimes even with empty |
| // forms array for main frame, but we're interested in only the first time |
| // call. |
| if (!forms_) |
| - forms_.reset(new std::vector<FormData>(forms.PassStorage())); |
| + forms_.reset(new std::vector<FormData>(std::move(forms))); |
|
yzshen1
2016/07/26 16:45:36
nit: std::move is not necessary here because it ha
leonhsl(Using Gerrit)
2016/07/27 02:50:26
Acknowledged.
leonhsl(Using Gerrit)
2016/07/27 09:25:32
Done.
|
| } |
| void WillSubmitForm(const FormData& form, |
| @@ -103,8 +103,8 @@ class FakeContentAutofillDriver : public mojom::AutofillDriver { |
| void DidEndTextFieldEditing() override {} |
| - void SetDataList(mojo::Array<mojo::String> values, |
| - mojo::Array<mojo::String> labels) override {} |
| + void SetDataList(const std::vector<std::string>& values, |
| + const std::vector<std::string>& labels) override {} |
| // Records whether TextFieldDidChange() get called. |
| bool called_field_change_; |