New issue
Advanced search Search tips

Issue 865704 link

Starred by 1 user
Status: Fixed
Owner:
parastoog@chromium.org
Closed: Jul 20
Cc:
se...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

[Autofill] regex for phone code

Project Member Reported by parastoog@chromium.org, Jul 19 
For H&M, the heuristics detect phone country code as whole phone number. The name is user-phone-code. We need to add this to our regex for heuristics.
Project Member

Comment 1 by bugdroid1@chromium.org, Jul 20 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/a0c66d93c5077f94ea751bdee504aee528fe9aa7

commit a0c66d93c5077f94ea751bdee504aee528fe9aa7
Author: Parastoo Geranmayeh <parastoog@google.com>
Date: Fri Jul 20 03:52:49 2018

[Autofill] Adding regex for heuristics

Regex constants are added to cover phone code (as in +1).
The regex for other languages are also improved.

Bug:  865704 
Change-Id: Iab678b7419f04fb6e495eda1c35d5911b2217997
Reviewed-on: https://chromium-review.googlesource.com/1144251
Commit-Queue: Parastoo Geranmayeh <parastoog@google.com>
Reviewed-by: Sebastien Seguin-Gagnon <sebsg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#576771}
[modify] https://crrev.com/a0c66d93c5077f94ea751bdee504aee528fe9aa7/components/autofill/core/common/autofill_regex_constants.cc

Comment 2 by parastoog@chromium.org, Jul 20

Status: Fixed (was: Started)
►

Sign in to add a comment