Add "previewing on hover" support for password field.

chrome/renderer/autofill/form_autofill_browsertest.cc

Index: chrome/renderer/autofill/form_autofill_browsertest.cc
diff --git a/chrome/renderer/autofill/form_autofill_browsertest.cc b/chrome/renderer/autofill/form_autofill_browsertest.cc
index 419545d58668d7f6515f5c562b9e1d987d8b25ce..391354ad72180b9fd197c5ec40188b5a7678b8b0 100644
--- a/chrome/renderer/autofill/form_autofill_browsertest.cc
+++ b/chrome/renderer/autofill/form_autofill_browsertest.cc
@@ -2837,6 +2837,42 @@ TEST_F(FormAutofillTest, ClearFormWithNodeContainingSelectOne) {
   EXPECT_EQ(0, firstname.selectionEnd());
 }
 
+TEST_F(FormAutofillTest, ClearPreviewedPassword) {
+  LoadHTML("<FORM name=\"TestForm\" action=\"http://buh.com\" method=\"post\">"
+           "  <INPUT type=\"text\" id=\"username\" value=\"Wyatt\"/>"
+           "  <INPUT type=\"password\" id=\"password\" autocomplete=\"off\"/>"
+           "  <INPUT type=\"submit\" value=\"Send\"/>"
+           "</FORM>");
+
+  WebFrame* web_frame = GetMainFrame();
+  ASSERT_NE(static_cast<WebFrame*>(NULL), web_frame);
+
+  // Set the auto-filled attribute.
+  WebInputElement username =
+      web_frame->document().getElementById("username").to<WebInputElement>();
+  username.setAutofilled(true);
+  WebInputElement password =
+      web_frame->document().getElementById("password").to<WebInputElement>();
+  password.setAutofilled(true);
+
+  // Set the suggested values on password.
+  password.setSuggestedValue(ASCIIToUTF16("SuggestedPassword"));
+
+  // Clear the previewed fields.
+  EXPECT_TRUE(ClearPreviewedFormWithElement(password, REQUIRE_NONE, false,
+                                            true));
+
+  // Fields with empty suggestions are not modified.
+  EXPECT_EQ(ASCIIToUTF16("Wyatt"), username.value());
+  EXPECT_TRUE(username.suggestedValue().isEmpty());
+  EXPECT_TRUE(username.isAutofilled());
+
+  // Verify the previewed fields are cleared.
+  EXPECT_TRUE(password.value().isEmpty());
+  EXPECT_TRUE(password.suggestedValue().isEmpty());
+  EXPECT_FALSE(password.isAutofilled());

[Ilya Sherman, 2014/03/21 22:35:19]

Hmm, why is the password field no longer autofilled, given that the last
parameter to ClearPreviewedFormWithElement() is |true|, indicating that the
password field was previously autofilled?

Please fix this [or let me know why I'm wrong ;)], and also add additional test
coverage.  Specifically, please test all four combinations of the username and
password fields being previously autofilled or not.  Also, please test cases
where the username element has a non-empty suggestion vs. where it doesn't.

In addition to that, please add test coverage for the PasswordAutofillAgent
method responsible for previewing the suggestions.

[ziran.sun, 2014/03/25 18:25:26]

You are right. There is an error in the test code. I've corrected it. 

Added all four combinations of the username and
password fields being previously autofilled or not.

Added one test on empty suggestedValue for username. I didn't add non-empty
suggestedValue test since I feel that it's covered by the other 4 tests .

For PasswordAutofillAgent related test, do you mean add a browser test in
autofill_password_agent_browsertest.cc? Otherwise, there is
DidSelectAutofillSuggestion unittest covered in PasswordAutofillManagerTest
already. 

[Ilya Sherman, 2014/03/25 19:45:27]

Yes, I do mean that we should test the renderer code as well as the browser
code.  Though, you might want to hold off on working on that until you implement
the change to send both the username and password across IPC -- it ought to
simplify the renderer code, which will hopefully make it both clearer to see
what needs to be tested, and easier to test.

+}
+
 TEST_F(FormAutofillTest, ClearPreviewedFormWithElement) {
   LoadHTML("<FORM name=\"TestForm\" action=\"http://buh.com\" method=\"post\">"
            "  <INPUT type=\"text\" id=\"firstname\" value=\"Wyatt\"/>"
@@ -2879,7 +2915,8 @@ TEST_F(FormAutofillTest, ClearPreviewedFormWithElement) {
   phone.setSuggestedValue(ASCIIToUTF16("650-777-9999"));
 
   // Clear the previewed fields.
-  EXPECT_TRUE(ClearPreviewedFormWithElement(lastname, false));
+  EXPECT_TRUE(ClearPreviewedFormWithElement(lastname, REQUIRE_AUTOCOMPLETE,
+                                            false, false));
 
   // Fields with empty suggestions suggestions are not modified.
   EXPECT_EQ(ASCIIToUTF16("Wyatt"), firstname.value());
@@ -2949,7 +2986,8 @@ TEST_F(FormAutofillTest, ClearPreviewedFormWithNonEmptyInitiatingNode) {
   phone.setSuggestedValue(ASCIIToUTF16("650-777-9999"));
 
   // Clear the previewed fields.
-  EXPECT_TRUE(ClearPreviewedFormWithElement(firstname, false));
+  EXPECT_TRUE(ClearPreviewedFormWithElement(firstname, REQUIRE_AUTOCOMPLETE,
+                                            false, false));
 
   // Fields with non-empty values are restored.
   EXPECT_EQ(ASCIIToUTF16("W"), firstname.value());
@@ -3016,7 +3054,8 @@ TEST_F(FormAutofillTest, ClearPreviewedFormWithAutofilledInitiatingNode) {
   phone.setSuggestedValue(ASCIIToUTF16("650-777-9999"));
 
   // Clear the previewed fields.
-  EXPECT_TRUE(ClearPreviewedFormWithElement(firstname, true));
+  EXPECT_TRUE(ClearPreviewedFormWithElement(firstname, REQUIRE_AUTOCOMPLETE,
+                                            true, false));
 
   // Fields with non-empty values are restored.
   EXPECT_EQ(ASCIIToUTF16("W"), firstname.value());