New issue
Advanced search Search tips

Issue 792473 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Dec 18
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , iOS , Mac
Pri: 3
Type: Bug


Show other hotlists

Hotlists containing this issue:
Autofill-Fixit


Sign in to add a comment

[Autofill] Move base::Feature definitions to common/autofill_features.{h,cc}

Project Member Reported by ma...@chromium.org, Dec 6 2017

Issue description

autofill_experiments is becoming quite big and has non trivial logic. Split off the Feature definitions in common/autofill_features.
 

Comment 1 by ma...@chromium.org, May 1 2018

Status: Untriaged (was: Available)
Status: Available (was: Untriaged)
Cc: jsaul@google.com
Owner: annelim@google.com
Status: Fixed (was: Available)
Closing as already Fixed.  annelim@ handled this in August via https://chromium-review.googlesource.com/c/chromium/src/+/1176353 and https://chromium-review.googlesource.com/c/chromium/src/+/1197105.  autofill_experiments.h/cc appear to have just 2 base::Features now, but they were both added after her cleanup CL, so they seem to be there on purpose.

Sign in to add a comment