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

Unified Diff: chrome/renderer/autofill/password_autofill_agent_browsertest.cc

Issue 614023002: [Password manager] Relplace the FormFieldData vector from autofill::FormData with named fields… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Incorporated Vaclav's review comments. Created 6 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: chrome/renderer/autofill/password_autofill_agent_browsertest.cc
diff --git a/chrome/renderer/autofill/password_autofill_agent_browsertest.cc b/chrome/renderer/autofill/password_autofill_agent_browsertest.cc
index 0c0a81bb768bfcdaaa06a0b860fd850385dfc632..9462bf9b60b929cf52eeb29d36adfb0358ee17e8 100644
--- a/chrome/renderer/autofill/password_autofill_agent_browsertest.cc
+++ b/chrome/renderer/autofill/password_autofill_agent_browsertest.cc
@@ -201,13 +201,13 @@ class PasswordAutofillAgentTest : public ChromeRenderViewTest {
FormFieldData username_field;
username_field.name = ASCIIToUTF16(kUsernameName);
username_field.value = username1_;
- fill_data_.basic_data.fields.push_back(username_field);
+ fill_data_.username_field = username_field;
FormFieldData password_field;
password_field.name = ASCIIToUTF16(kPasswordName);
password_field.value = password1_;
password_field.form_control_type = "password";
- fill_data_.basic_data.fields.push_back(password_field);
+ fill_data_.password_field = password_field;
PasswordAndRealm password2;
password2.password = password2_;
@@ -225,7 +225,7 @@ class PasswordAutofillAgentTest : public ChromeRenderViewTest {
// We need to set the origin so it matches the frame URL and the action so
// it matches the form action, otherwise we won't autocomplete.
UpdateOriginForHTML(kFormHTML);
- fill_data_.basic_data.action = GURL("http://www.bidule.com");
+ fill_data_.action = GURL("http://www.bidule.com");
LoadHTML(kFormHTML);
@@ -241,7 +241,7 @@ class PasswordAutofillAgentTest : public ChromeRenderViewTest {
void UpdateOriginForHTML(const std::string& html) {
std::string origin = "data:text/html;charset=utf-8," + html;
- fill_data_.basic_data.origin = GURL(origin);
+ fill_data_.origin = GURL(origin);
}
void UpdateUsernameAndPasswordElements() {
@@ -545,7 +545,7 @@ TEST_F(PasswordAutofillAgentTest, InitialAutocompleteForEmptyAction) {
// Set the expected form origin and action URLs.
UpdateOriginForHTML(kEmptyActionFormHTML);
- fill_data_.basic_data.action = fill_data_.basic_data.origin;
+ fill_data_.action = fill_data_.origin;
// Simulate the browser sending back the login info, it triggers the
// autocomplete.
@@ -905,8 +905,8 @@ TEST_F(PasswordAutofillAgentTest, IframeNoFillTest) {
LoadHTML(page_html.c_str());
// Set the expected form origin and action URLs.
- fill_data_.basic_data.origin = GURL(origin);
- fill_data_.basic_data.action = GURL(origin);
+ fill_data_.origin = GURL(origin);
+ fill_data_.action = GURL(origin);
SimulateOnFillPasswordForm(fill_data_);

Powered by Google App Engine
This is Rietveld 408576698