New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 895340 link

Starred by 3 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Task



Sign in to add a comment

Move User Events away from components/sync

Project Member Reported by mastiz@chromium.org, Oct 15

Issue description

USER_EVENTS is the only remaining datatype (excluding DeviceInfo which is special) with code living under components/sync.

For proper layering, it's generally undesirable to bundle datatype-specific code/logic in core sync, and we've done similar cleanup work for SESSIONS, in  crbug.com/883199 .

Currently, the code lives in components/sync/user_events, which we believe was chosen to avoid build dependency cycles. Presumably these arguments no longer apply, so let's find a new home for the code.

Some possible destination candidates:
components/event_logger
components/user_events
 
Labels: Hotlist-GoodFirstBug

Sign in to add a comment