Only consider PasswordFromManager which HasCompletedMatching when provisionally saving passwords

chrome/test/data/password/form_utils.js

+function createForm() {

[engedy, 2014/07/11 17:05:36]

Now that we are exporting this into a util, I think it might make sense to add
comments above this and rename it to something more specific (e.g.,
createSimplePasswordForm()).

[vabr (Chromium), 2014/07/11 18:15:48]

Done.

+  var form = document.createElement('form');
+  form.setAttribute('action', 'done.html');
+
+  var username_label = document.createElement('label');
+  username_label.setAttribute('for', 'username');

[engedy, 2014/07/11 17:05:36]

We could use the short form here (and in 1 more place):

username_label.htmlFor = 'username';

[vabr (Chromium), 2014/07/11 18:15:48]

Done.

+  username_label.innerText = 'Username: ';
+  var username = document.createElement('input');
+  username.type = 'text';
+  username.name = 'username';
+  username.id = 'username';
+
+  var password_label = document.createElement('label');
+  password_label.innerText = 'Password: ';
+  password_label.setAttribute('for', 'password');
+  var password = document.createElement('input');
+  password.type = 'password';
+  password.name = 'password';
+  password.id = 'password';
+
+  var submit = document.createElement('input');
+  submit.type = 'submit';
+  submit.id = 'submit-button';
+  submit.value = 'Submit';
+
+  form.appendChild(username_label);
+  form.appendChild(username);
+  form.appendChild(password_label);
+  form.appendChild(password);
+  form.appendChild(submit);
+
+  return form;
+}
+
+function addForm(form) {

[engedy, 2014/07/11 17:05:36]

I think this might be suitable for inlining after below change.

[vabr (Chromium), 2014/07/11 18:15:48]

Done.

+  document.getElementsByTagName('body')[0].appendChild(form);

[engedy, 2014/07/11 17:05:36]

We could just use the short form here:

document.body.appendChild(form);

[vabr (Chromium), 2014/07/11 18:15:48]

Done.

+}