Autofill: Search for standalone label tags while looking through divs.

Autofill: Search for standalone label tags while looking through divs.

Some websites use labels, but do not wrap inputs in labels and/or do not
add the proper for= attribute.

Also add a failing test case, which this CL improves upon.
BUG=465587
Committed: https://crrev.com/69ca66410b63d9fd7ce1ef620541783233349362
Cr-Commit-Position: refs/heads/master@{#320860}

[Lei Zhang, 2015-03-14 00:01:49]

Patchset #1 (id:1) has been deleted

[Lei Zhang, 2015-03-14 00:09:40]

thestig@chromium.org changed reviewers:
+ estade@chromium.org

[Evan Stade, 2015-03-16 23:35:36]

https://codereview.chromium.org/1003163003/diff/40001/components/autofill/con...
File components/autofill/content/renderer/form_autofill_util.cc (right):

https://codereview.chromium.org/1003163003/diff/40001/components/autofill/con...
components/autofill/content/renderer/form_autofill_util.cc:428: // is to ignore
<label for="valid_id"> since those are associated with other
isn't it pretty easy to do this? just check
WebLabelElement::correspondingControl()

[Lei Zhang, 2015-03-17 00:23:37]

https://codereview.chromium.org/1003163003/diff/40001/components/autofill/con...
File components/autofill/content/renderer/form_autofill_util.cc (right):

https://codereview.chromium.org/1003163003/diff/40001/components/autofill/con...
components/autofill/content/renderer/form_autofill_util.cc:428: // is to ignore
<label for="valid_id"> since those are associated with other
On 2015/03/16 23:35:36, Evan Stade wrote:
> isn't it pretty easy to do this? just check
> WebLabelElement::correspondingControl()

Yes, I just didn't get to it, so I just wrote a note-to-self.

[Evan Stade, 2015-03-17 01:42:30]

lgtm

[Lei Zhang, 2015-03-17 02:15:15]

The CQ bit was checked by thestig@chromium.org

[commit-bot: I haz the power, 2015-03-17 02:15:36]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1003163003/60001

[commit-bot: I haz the power, 2015-03-17 04:28:46]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2015-03-17 04:29:26]

Patchset 3 (id:??) landed as
https://crrev.com/69ca66410b63d9fd7ce1ef620541783233349362
Cr-Commit-Position: refs/heads/master@{#320860}