Autofill browser_tests: set autofilled property after autofilled value is set.

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 abffbf6c06b12d618e85265cf9d99d7ceb9e7929..3fc71bced03250e97dc1feb6a70da04631e36b47 100644
--- a/chrome/renderer/autofill/form_autofill_browsertest.cc
+++ b/chrome/renderer/autofill/form_autofill_browsertest.cc
@@ -3199,15 +3199,17 @@ TEST_F(FormAutofillTest, ClearFormWithNodeContainingSelectOne) {
   WebInputElement lastname =
       web_frame->document().getElementById("lastname").to<WebInputElement>();
   lastname.setAutofilled(true);
-  WebInputElement state =
-      web_frame->document().getElementById("state").to<WebInputElement>();
-  state.setAutofilled(true);
 
   // Set the value of the select-one.
   WebSelectElement select_element =
       web_frame->document().getElementById("state").to<WebSelectElement>();
   select_element.setValue(WebString::fromUTF8("AK"));
 
+  // Set the auto-filled attribute of the select-one.
+  WebInputElement state =
+        web_frame->document().getElementById("state").to<WebInputElement>();

[Ilya Sherman, 2014/09/25 22:31:14]

Hmm... how are we even able to cast to WebInputElement?  AFAICT, that's not a
parent class for this element.

[ziran.sun, 2014/09/26 09:34:41]

Good point! Sorry, I've been careless and didn't spot this.

+  state.setAutofilled(true);

[Ilya Sherman, 2014/09/25 22:31:14]

I'd suggest combining lines 3203 through 3211 as

// Set the value and autofilled attribute of the state element.
WebSelectElement state =
    web_frame->document().getElementById("state").to<WebSelectElement>();
state.setValue(WebString::fromUTF8("AK"));
state.setAutofilled(true);

[ziran.sun, 2014/09/26 09:34:41]

Done.

+
   // Clear the form.
   EXPECT_TRUE(form_cache.ClearFormWithElement(firstname));