New issue
Advanced search Search tips

Issue 822100 link

Starred by 1 user

Issue metadata

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


Participants' hotlists:
Autofill-UKM


Sign in to add a comment

Add IsForCrditCard to Autofill_FormSubmitted UKM.

Project Member Reported by nikunjb@chromium.org, Mar 15 2018

Issue description

Autofill_FormSubmitted Ukm metric is missing if the form is a CC form or not. This is needed for separating submission for CC form and other forms.
 
Status: Assigned (was: Untriaged)
Where is your original doc approved by privacy? Can you share the link and also mark where you are modifying?
Form Interactions UKM was one of the first UKMs added. It was before we had a process for privacy review of new metric.

It was added in crbug/709530.

I can briefly describe the metric:- 

Autofill_FormSubmitted is recorded when autofill detects form submit on a page. Following additional data is recorded with form submits -

AutofillFormSubmittedState - Enum storing one of the values, all fields were autofill, some field were autofill, no field were autofilled, form was not fillable.

MillisecondsSinceFormParsed - Time since form parse.

I am proposing to add form_type information to this. Form type is one of the 3 values CreditCard, Password and Profile.

This metric doesn't add any new data. This data is already available through other fields. e.g IsForCreditCard is available in Autofill_InteractedWithForm, Password type is present through various Password related UKM. Having this directly in FormSubmitted will simplify querying for number of form submits for each of the form type.






Thanks to help from holte@ I was able to find the initial review doc. Although this is in different format -

https://docs.google.com/document/d/1M5hkTc4ZEUwKeCqBXGQBvVktdX8zpx1loxnXv6BrCNs/edit#heading=h.bzl43651yhcs

The section is "Credit Card and Address Form Interactions"

It looks like original design envisioned adding form_type (Credit card vs Address form). Although this wasn't implemented.
Status: Fixed (was: Assigned)

Sign in to add a comment