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

Issue 851439 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Nov 21
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , iOS , Chrome , Mac
Pri: 2
Type: Task

Blocking:
issue 892076



Sign in to add a comment

Add UserConsents garbage collection

Project Member Reported by vitaliii@chromium.org, Jun 11 2018

Issue description

Currently they are never deleted.
 
Status: Started (was: Assigned)
Labels: -Type-Bug Type-Task
sync-triage ping: any updates?
sync-triage-ping, any updates?
Blocking: 892076
Blocking: -851433
Owner: ----
Status: Available (was: Started)
Long time ago I had discussions with msramek@ and sabineb@ and they were not sure whether we need to garbage collection for consents. Thus, just delete the todo from the code and close this bug.
Labels: OS-Android OS-Chrome OS-iOS OS-Linux OS-Mac OS-Windows
Owner: treib@chromium.org
Status: Started (was: Available)
Project Member

Comment 9 by bugdroid1@chromium.org, Nov 21

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

commit d037907cfb9d4b4daad51b09c0a21c1a9fd1f11c
Author: Marc Treib <treib@chromium.org>
Date: Wed Nov 21 15:28:19 2018

Remove obsolete TODOs from the user event and consent sync bridges

Bug:  851439 
Change-Id: Id955ccd91aa234c9a81566c0c76568e30d9d54ac
Reviewed-on: https://chromium-review.googlesource.com/c/1346396
Reviewed-by: vitaliii <vitaliii@chromium.org>
Commit-Queue: Marc Treib <treib@chromium.org>
Cr-Commit-Position: refs/heads/master@{#610065}
[modify] https://crrev.com/d037907cfb9d4b4daad51b09c0a21c1a9fd1f11c/components/consent_auditor/consent_sync_bridge_impl.cc
[modify] https://crrev.com/d037907cfb9d4b4daad51b09c0a21c1a9fd1f11c/components/sync/user_events/user_event_sync_bridge.cc

Status: Fixed (was: Started)

Sign in to add a comment