New issue
Advanced search Search tips

Issue 818153 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug



Sign in to add a comment

Media Engagement: clear data from MEI DB when expired URLs are the latest history entry

Project Member Reported by mlamouri@chromium.org, Mar 2 2018

Issue description

Chrome's history DB entries older than 90 days are cleared. MEI doesn't stay in sync with the history DB but if example.com's last entry in the history is removed, MEI should also clear its DB. It would otherwise leave traces of the website's usage.

We should land this in M67 to make sure we do not miss some entries.
 
Status: Started (was: Assigned)
Project Member

Comment 2 by bugdroid1@chromium.org, Mar 27 2018

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

commit 17135cd8286064d2f9f5c2659fbfed918654e4f1
Author: Becca Hughes <beccahughes@chromium.org>
Date: Tue Mar 27 20:30:02 2018

Media Engagement: Clear data from MEI when urls are expired

History is expired automatically after 90 days. This clears data from
MEI if history is expired and there is no more history remaining on that
origin.

BUG= 818153 

Change-Id: I7dd309d5abddb398ff0ec452147d25a86ce827b6
Reviewed-on: https://chromium-review.googlesource.com/960142
Commit-Queue: Becca Hughes <beccahughes@chromium.org>
Reviewed-by: Mounir Lamouri <mlamouri@chromium.org>
Cr-Commit-Position: refs/heads/master@{#546237}
[modify] https://crrev.com/17135cd8286064d2f9f5c2659fbfed918654e4f1/chrome/browser/media/media_engagement_service.cc
[modify] https://crrev.com/17135cd8286064d2f9f5c2659fbfed918654e4f1/chrome/browser/media/media_engagement_service.h
[modify] https://crrev.com/17135cd8286064d2f9f5c2659fbfed918654e4f1/chrome/browser/media/media_engagement_service_unittest.cc
[modify] https://crrev.com/17135cd8286064d2f9f5c2659fbfed918654e4f1/tools/metrics/histograms/enums.xml

Status: Fixed (was: Started)

Sign in to add a comment