New issue
Advanced search Search tips

Issue 807747 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 3
Type: Bug



Sign in to add a comment

[Autofill] Some forms need field by field.

Project Member Reported by parastoog@chromium.org, Jan 31 2018

Issue description

The last name and first name don't get filled, because the placeholder changes from capital letter to small letter!

www.cargurus.com

 
Description: Show this description
Description: Show this description
Description: Show this description
Description: Show this description
Project Member

Comment 5 by bugdroid1@chromium.org, 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

Status: Fixed (was: Started)

Sign in to add a comment