Enable password saving for dynamic password forms

components/autofill/content/renderer/autofill_agent.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/autofill/content/renderer/autofill_agent.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/message_loop/message_loop.h"
 #include "base/strings/string_split.h"
@@ skipping 589 matching lines @@
   base::string16 substring = value;
   substring = substring.substr(0, node->maxLength());
 
   node->setEditingValue(substring);
 }
 
 void AutofillAgent::HideAutofillUI() {
   Send(new AutofillHostMsg_HideAutofillUI(routing_id()));
 }
 
-// TODO(isherman): Decide if we want to support autofill with AJAX.

[Ilya Sherman, 2013/09/14 02:18:07]

Please keep this TODO, though feel free to tweak it to make it clear that this
is referring to non-password Autofill.

[guohui, 2013/09/16 19:51:42]

Done.

 void AutofillAgent::didAssociateFormControls(
     const WebKit::WebVector<WebKit::WebNode>& nodes) {
+  for (size_t i = 0; i < nodes.size(); ++i) {
+    WebKit::WebFrame* frame = nodes[i].document().frame();
+    // Only monitors dynamic forms created in the top frame.

[Ilya Sherman, 2013/09/14 02:18:07]

nit: Would be nice to document *why* that's important/desirable.

[guohui, 2013/09/16 19:51:42]

Done.

+    if (!frame->parent()) {
+      password_autofill_agent_->OnDynamicFormsSeen(frame);
+      return;
+    }
+  }
 }
 
 }  // namespace autofill