Update TvSettingsUIInteracted atom

This CL removes the string field from the TvSettingsUIInteracted atom to
eliminate the risk of logging sensitive information by accident. This
change should be safe as no such log has been uploaded to Google server
yet.

Besides the change to the atom, this CL also made some corresponding
change to the enum file so fill in the information that was collected
using a string.
(This CL is a mirror of cl/310419842 to Android repo.)

Metrics council review bug: 151768952
Bug: 150979527
Test: the Android tree still builds normally

Change-Id: I8a9d8c95ec06eaa2cbe097efe652e18d0e7baeac
This commit is contained in:
Zhen Sun
2020-05-07 11:49:11 -07:00
parent 2404bc617a
commit fcaca1bc15
2 changed files with 18 additions and 3 deletions

View File

@@ -9461,9 +9461,6 @@ message TvSettingsUIInteracted {
/** The ID of the entry that the users actioned on */
optional android.app.tvsettings.ItemId item_id = 2;
/** Additional information (e.g., navigation direction on page focused) */
optional string additional_info = 3;
}
/**

View File

@@ -44,6 +44,24 @@ enum Action {
/** Denotes that a toggle is clicked by a user. */
TOGGLE_INTERACTED = 3;
/**
* Denotes that a TvSettings page is being focused in the forward direction
* into the settings tree.
*/
PAGE_FOCUSED_FORWARD = 4;
/**
* Denotes that a TvSettings page is being focused in the backward direction
* up the settings tree.
*/
PAGE_FOCUSED_BACKWARD = 5;
/** Denotes that a toggle is turned on by a user. */
TOGGLED_ON = 6;
/** Denotes that a toggle is turned off by a user. */
TOGGLED_OFF = 7;
}
/**