Requery the autofill server when forms and input fields are dynamically added.

components/autofill/renderer/autofill_agent.cc

Index: components/autofill/renderer/autofill_agent.cc
diff --git a/components/autofill/renderer/autofill_agent.cc b/components/autofill/renderer/autofill_agent.cc
index c85c265efae424a9a745c801536f06334b09580f..4e53c32d8f0ee57fddac054c55dcfb97604b6445 100644
--- a/components/autofill/renderer/autofill_agent.cc
+++ b/components/autofill/renderer/autofill_agent.cc
@@ -212,11 +212,14 @@ void AutofillAgent::DidFinishDocumentLoad(WebFrame* frame) {
     form_cache_.ExtractForms(*frame, &forms);
   }
 
+  autofill::FormsSeenParam param = has_more_forms ?
+      autofill::PARTIAL_FORMS_SEEN : autofill::NO_SPECIAL_FORMS_SEEN;
+
   // Always communicate to browser process for topmost frame.
   if (!forms.empty() || !frame->parent()) {
     Send(new AutofillHostMsg_FormsSeen(routing_id(), forms,
                                        forms_seen_timestamp_,
-                                       has_more_forms));
+                                       param));
   }
 }
 
@@ -726,7 +729,8 @@ void AutofillAgent::OnGetAllForms() {
 
   // Report to AutofillManager that all forms are being sent.
   Send(new AutofillHostMsg_FormsSeen(routing_id(), forms,
-                                     forms_seen_timestamp_, false));
+                                     forms_seen_timestamp_,
+                                     NO_SPECIAL_FORMS_SEEN));
 }
 
 void AutofillAgent::OnRequestAutocompleteResult(
@@ -774,6 +778,9 @@ void AutofillAgent::OnFillFormsAndClick(
 
 void AutofillAgent::OnWhitelistedForAutocheckout() {
   is_whitelisted_for_autocheckout_ = true;
+  if (forms_have_changed_since_load_) {

[ahutter, 2013/04/02 21:34:53]

no curlies

[Dane Wallinga, 2013/04/03 00:40:41]

Done.

+    SendDynamicFormsSeen(forms_seen_timestamp_);
+  }
 }
 
 void AutofillAgent::ClickFailed() {
@@ -931,4 +938,37 @@ void AutofillAgent::HideHostAutofillUi() {
   Send(new AutofillHostMsg_HideAutofillUi(routing_id()));
 }
 
+void AutofillAgent::didAssociateFormControls(
+    const WebKit::WebVector<WebKit::WebNode>& nodes) {
+  for (size_t i = 0; i < nodes.size(); ++i) {
+    WebKit::WebNode node = nodes[i];
+    if (node.document().frame() == topmost_frame_) {
+      forms_seen_timestamp_ = base::TimeTicks::Now();
+      forms_have_changed_since_load_ = true;
+      break;
+    }
+  }
+
+  if (forms_have_changed_since_load_ && is_whitelisted_for_autocheckout_) {

[ahutter, 2013/04/02 21:34:53]

no curlies.

[ahutter, 2013/04/02 21:34:53]

no curlies

[Dane Wallinga, 2013/04/03 00:40:41]

Done.

+    SendDynamicFormsSeen(forms_seen_timestamp_);
+  }
+}
+
+void AutofillAgent::SendDynamicFormsSeen(base::TimeTicks forms_seen_timestamp) {

[Raman Kakilate, 2013/04/02 21:39:46]

forms_seen_timestamp is always a member variable forms_seen_timestamp_. We could
avoid this arg completely.

[Dane Wallinga, 2013/04/03 00:40:41]

works for me

+  forms_have_changed_since_load_ = false;
+  bool has_more_forms = false;
+  std::vector<FormData> forms;
+  form_elements_.clear();
+  has_more_forms = form_cache_.ExtractFormsAndFormElements(
+      *topmost_frame_, kRequiredAutofillFields, &forms, &form_elements_);
+  autofill::FormsSeenParam param = has_more_forms ?
+      autofill::PARTIAL_AND_DYNAMIC_FORMS_SEEN : autofill::DYNAMIC_FORMS_SEEN;
+
+  if (!forms.empty()) {
+    Send(new AutofillHostMsg_FormsSeen(routing_id(), forms,
+                                       forms_seen_timestamp,
+                                       param));
+  }
+}
+
 }  // namespace autofill