Chromium Code Reviews| Index: components/autofill/content/browser/content_autofill_driver_unittest.cc |
| diff --git a/components/autofill/content/browser/content_autofill_driver_unittest.cc b/components/autofill/content/browser/content_autofill_driver_unittest.cc |
| index 90fb9df13fb233d39e859e8fd242cc774bcf16e4..93e7e26bf2fa7a0c292e24cc232a1766282c32c9 100644 |
| --- a/components/autofill/content/browser/content_autofill_driver_unittest.cc |
| +++ b/components/autofill/content/browser/content_autofill_driver_unittest.cc |
| @@ -28,7 +28,6 @@ |
| #include "content/public/common/frame_navigate_params.h" |
| #include "content/public/test/mock_render_process_host.h" |
| #include "content/public/test/test_renderer_host.h" |
| -#include "mojo/common/common_type_converters.h" |
| #include "mojo/public/cpp/bindings/binding_set.h" |
| #include "services/shell/public/cpp/interface_provider.h" |
| #include "testing/gmock/include/gmock/gmock.h" |
| @@ -162,8 +161,8 @@ class FakeAutofillAgent : public mojom::AutofillAgent { |
| } |
| void FieldTypePredictionsAvailable( |
| - mojo::Array<FormDataPredictions> forms) override { |
| - predictions_ = forms.PassStorage(); |
| + const std::vector<FormDataPredictions>& forms) override { |
| + predictions_ = std::move(forms); |
|
yzshen1
2016/07/27 15:29:53
nit: no need to std::move.
leonhsl(Using Gerrit)
2016/07/29 02:22:31
Done.
|
| CallDone(); |
| } |
| @@ -177,26 +176,26 @@ class FakeAutofillAgent : public mojom::AutofillAgent { |
| CallDone(); |
| } |
| - void FillFieldWithValue(const mojo::String& value) override { |
| - value_fill_field_ = value.To<base::string16>(); |
| + void FillFieldWithValue(const base::string16& value) override { |
| + value_fill_field_ = value; |
| CallDone(); |
| } |
| - void PreviewFieldWithValue(const mojo::String& value) override { |
| - value_preview_field_ = value.To<base::string16>(); |
| + void PreviewFieldWithValue(const base::string16& value) override { |
| + value_preview_field_ = value; |
| CallDone(); |
| } |
| - void AcceptDataListSuggestion(const mojo::String& value) override { |
| - value_accept_data_ = value.To<base::string16>(); |
| + void AcceptDataListSuggestion(const base::string16& value) override { |
| + value_accept_data_ = value; |
| CallDone(); |
| } |
| - void FillPasswordSuggestion(const mojo::String& username, |
| - const mojo::String& password) override {} |
| + void FillPasswordSuggestion(const base::string16& username, |
| + const base::string16& password) override {} |
| - void PreviewPasswordSuggestion(const mojo::String& username, |
| - const mojo::String& password) override {} |
| + void PreviewPasswordSuggestion(const base::string16& username, |
| + const base::string16& password) override {} |
| void ShowInitialPasswordAccountSuggestions( |
| int32_t key, |