New issue
Advanced search Search tips

Issue 813186 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac
Pri: 1
Type: Bug

Blocking:
issue 760294



Sign in to add a comment

Data saver network properties should be cleared after ~1 month

Project Member Reported by tbansal@chromium.org, Feb 16 2018

Issue description

Currently, the data saver network properties are limited to at most 10 in count. 

We should clear the individual properties that have not been used in the last month. This came up in the privacy review.
 
 
Blocking: 760294
Project Member

Comment 2 by bugdroid1@chromium.org, Feb 24 2018

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

commit 825e3d69e414822d3e15dd2c4035087c2c2578e4
Author: Tarun Bansal <tbansal@chromium.org>
Date: Sat Feb 24 04:53:26 2018

Remove network properties entries that are older than 30 days

Network properties entries that are older than 30 days are removed
from the persisted prefs.

Bug:  813186 
Change-Id: I7e4b306eeebff5c0eadd5f03079679cf1e89bbc7
Reviewed-on: https://chromium-review.googlesource.com/926082
Reviewed-by: Ryan Sturm <ryansturm@chromium.org>
Commit-Queue: Tarun Bansal <tbansal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#538991}
[modify] https://crrev.com/825e3d69e414822d3e15dd2c4035087c2c2578e4/components/data_reduction_proxy/core/browser/data_reduction_proxy_config_service_client_unittest.cc
[modify] https://crrev.com/825e3d69e414822d3e15dd2c4035087c2c2578e4/components/data_reduction_proxy/core/browser/data_reduction_proxy_config_unittest.cc
[modify] https://crrev.com/825e3d69e414822d3e15dd2c4035087c2c2578e4/components/data_reduction_proxy/core/browser/data_reduction_proxy_configurator_unittest.cc
[modify] https://crrev.com/825e3d69e414822d3e15dd2c4035087c2c2578e4/components/data_reduction_proxy/core/browser/data_reduction_proxy_io_data.cc
[modify] https://crrev.com/825e3d69e414822d3e15dd2c4035087c2c2578e4/components/data_reduction_proxy/core/browser/data_reduction_proxy_settings_unittest.cc
[modify] https://crrev.com/825e3d69e414822d3e15dd2c4035087c2c2578e4/components/data_reduction_proxy/core/browser/network_properties_manager.cc
[modify] https://crrev.com/825e3d69e414822d3e15dd2c4035087c2c2578e4/components/data_reduction_proxy/core/browser/network_properties_manager.h
[modify] https://crrev.com/825e3d69e414822d3e15dd2c4035087c2c2578e4/components/data_reduction_proxy/core/browser/network_properties_manager_unittest.cc
[modify] https://crrev.com/825e3d69e414822d3e15dd2c4035087c2c2578e4/components/data_reduction_proxy/core/common/data_reduction_proxy_event_store_unittest.cc

Status: Fixed (was: Started)

Sign in to add a comment