[Autofill] Some forms need field by field. |
|||||
Issue descriptionThe last name and first name don't get filled, because the placeholder changes from capital letter to small letter! www.cargurus.com
,
Jan 31 2018
,
Jan 31 2018
,
Jan 31 2018
,
Feb 2 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/d265f7fc28ba25514382016dd46b0fe8422319cb commit d265f7fc28ba25514382016dd46b0fe8422319cb Author: Parastoo Geranmayeh <parastoog@google.com> Date: Fri Feb 02 00:31:30 2018 [Autofill] Handle placeholder format changes. Sometimes websites change the format of the placeholders. This makes us detect the second one as a value entered by the user, and then we miss the field. Solution: While filling the form in the renderer, compare the placeholder value with the element value case-insensitively. Fixes: www.cargurus.com, etc. Bug: 807747 Change-Id: Id97a2b9978818ed22dacfd9ee427886562bf6f41 Reviewed-on: https://chromium-review.googlesource.com/898072 Commit-Queue: Parastoo Geranmayeh <parastoog@google.com> Reviewed-by: Sebastien Seguin-Gagnon <sebsg@chromium.org> Cr-Commit-Position: refs/heads/master@{#533870} [modify] https://crrev.com/d265f7fc28ba25514382016dd46b0fe8422319cb/chrome/renderer/autofill/form_autofill_browsertest.cc [modify] https://crrev.com/d265f7fc28ba25514382016dd46b0fe8422319cb/components/autofill/content/renderer/form_autofill_util.cc
,
Feb 8 2018
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by parastoog@chromium.org
, Jan 31 2018